-
Notifications
You must be signed in to change notification settings - Fork 192
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Allow response files to be special files #1197
base: main
Are you sure you want to change the base?
Conversation
Thank you @stephank. Could you please add a test for this (with a command like in the PR description) to the test suite? |
I added a test case and checked the before and after. I'm not sure if this sort of thing works on Windows, I don't have a machine to test. |
@swift-ci test |
@swift-ci test Windows platform |
That CI error is a little weird:
But that line doesn't have that import; it appears to be in Regardless, when I wrote the comment about Windows, I was thinking file descriptors, but I suppose the whole |
Makes sense for this particular test, I think. |
swiftlang/sourcekit-lsp#640 should help with the LSP issue. |
Please test with following PRs: @swift-ci please test Windows platform |
I think this repository does not support cross-repo testing. :( |
Unfortunately, the CI currently is build only; please do not merge this without testing on Windows, I am worried that this might regress the test suite. |
Oh hey, Windows is green. But I'm looking at the logs, and can't really find a test run for swift-driver in there, is that correct? |
Yeah, that is the problem - the CI doesn't actually run the test suite yet. That is something that would be nice to have, but needs work. In the mean time, tests need to be run locally. |
In NixOS, we wrap toolchain executables to inject search paths (among other things). These wrappers are bash scripts that build an often large array of arguments, then provide them as a response file via redirect like so:
The above is a simple test case that currently fails. Bash turns this into:
But the device node used there doesn't pass the
isFile
check. This PR broadens that check by usingisReadable
instead.