Conversation
|
|
||
| assert aclSupport -> acl != null; | ||
| assert selinuxSupport -> libselinux != null && libsepol != null; | ||
| assert hostPlatform.libc != "msvcrt"; |
There was a problem hiding this comment.
I think it would make more sense to avoid the assertion and just do platforms = platforms.unix below. Assertions are annoying because there is no way to override them with something like allowUnsupportedPlatform.
GrahamcOfBorg
added
10.rebuild-darwin: 0
| cc_solib = getLib cc; | ||
| # The wrapper scripts use 'cat' and 'grep', so we may need coreutils. | ||
| coreutils_bin = if nativeTools then "" else getBin coreutils; | ||
| coreutils_bin = if (nativeTools || targetPlatform.isWindows) then "" else getBin coreutils; |
There was a problem hiding this comment.
I still think this one is in error? It means cc-wrapper won't use coreutils, which seems wrong.
There was a problem hiding this comment.
No. this is right. It should not use coreutils on windows. We don't have them on windows.
There was a problem hiding this comment.
But that means you won't use them on Linux when you are making windows binaries. Use hostPlatform to turn it just when compiling on Windows (for any target).
There was a problem hiding this comment.
hmm... I'll try hostPlatform, let's see. My issue was that it tried to buld coreutils for windows when cross compiling windows stuff on linux.
Ericson2314
left a comment
Ericson2314
left a comment
There was a problem hiding this comment.
I think we don't want the cc-wrapper changes. The rest is good, though.
|
@angerman Did you ever run into this bug: ? Here is the build: |
|
@matthewbauer I haven't. But I remember there being some similar sounding issues with recent git checkouts of GHC. I haven't looked into those though... |
|
Applied in acaa6c9. |
4 participants
Motivation for this change
Broken out of #43559
Things done
sandboxinnix.confon non-NixOS)nix-shell -p nox --run "nox-review wip"./result/bin/)nix path-info -Sbefore and after)