Don't declare _GNU_SOURCE on Windows#552
Don't declare _GNU_SOURCE on Windows#552kazu-yamamoto merged 1 commit intohaskell:masterfrom RyanGlScott:T551
_GNU_SOURCE on Windows#552Conversation
Doing so will cause `network` to have an undeclared dependency against the `mingwex` library on Windows, which can cause issues with GHC's runtime linker. (See https://gitlab.haskell.org/ghc/ghc/-/issues/23309 for the full story.) Thankfully, there is no particular need to define `_GNU_SOURCE` on Windows in the first place, so I have guarded its definition with CPP. Fixes #551.
|
Thanks as always. |
|
Thanks! I would definitely appreciate a new Hackage release, as this will fix a pretty serious Windows-only issue. |
|
Done. |
|
Do we also want Hackage revisions for old versions of if impl(ghc >= 9.4.5) && os(windows)
buildable: false(see sol/hpack#548 (comment)) |
@sol : Unfortunately, AFAIK, you cannot add a conditional in a revision, Hackage will refuse your revision attempt. However, could we nevertheless constrain any |
|
@andreasabel would doing that (constraining the base version indiscriminately) potentially break existing freeze files? Or does cabal ignore revisions when there is a freeze file (as it probably should)? |
Sorry, I don't know, I do not use |
|
Freeze files pin the time of Hackage index state when they were created (example), so subsequent revisions will not affect the freeze file. |
That's great! Given this, I think indiscriminately adding |
So yes, all for it 👍 |
@kazu-yamamoto Do you wish to do this? |
|
Sorry for the delay. I wan on vacation. I need to admit that I cannot follow this conversation completely. |
|
The suggestion is to add an upper bound for all older
This could be done with the
|
|
@kazu-yamamoto the alternative would be that every user of (that's at least how I understand the situation) |
|
I agree. |
|
@andreasabel @sol |
I gave it a brief look, but I cannot really check this efficiently since it does not display the diff... If you do |
|
@andreasabel Each commit shows you the diffs. Isn't it good enough? |
Apologies, indeed. I looked through the commits now, and they all set the So, LGTM! |
|
I need to increment |
|
Kazu, thanks for taking care of this 🙏😊
I think I never had to do this explicitly. Shouldn't the tooling already take care of it somehow? |
|
OK. I found |
|
For record, I wrote |
Doing so will cause
networkto have an undeclared dependency against themingwexlibrary on Windows, which can cause issues with GHC's runtime linker. (See https://gitlab.haskell.org/ghc/ghc/-/issues/23309 for the full story.) Thankfully, there is no particular need to define_GNU_SOURCEon Windows in the first place, so I have guarded its definition with CPP.Fixes #551.