-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Add more Cygwin support #9686
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
Add more Cygwin support #9686
Conversation
|
You don't need to care about |
I don't think so, it still carries the Windows default " |
|
Hmm, not sure why style is failing, I've already formatted the pieces I touched. Will format again anyway. |
|
MSYS package builds fine without |
|
Oh sorry. You are right. Cygwin is not Windows_NT... |
It does build fine, but that's because it skips |
|
GNU testsuite comparison: |
This comment was marked as resolved.
This comment was marked as resolved.
|
GNU testsuite comparison: |
|
Hmm network failures... |
|
GNU testsuite comparison: |
f1fc880 to
ba22121
Compare
|
any update on this? thanks |
|
Still waiting on nix, which is waiting on libc. Unless you don't mind using |
|
Merging this is still useful to reduce num of patches at packaging even if we need newer nix and libc. |
|
Also note that updating |
If not being buildable out of the box is fine then technically this PR could be merged as is. |
|
GNU testsuite comparison: |
|
ok, sounds good! |
Makes more things compile on Cygwin. Basically a continuation of #9535. This requires a pending PR to nix, and after that some packages need to be updated to allow it to compile out of the box on Cygwin.