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
cmake: 3.11.2 -> 3.12.1 #44910
cmake: 3.11.2 -> 3.12.1 #44910
Conversation
Please rebase this on staging. What's the reason for keeping 4 different cmake versions? Are they all being used? |
From a quick survey:
I'm testing removing both of these usages and using 3.12 only. |
104da2e
to
e841ffe
Compare
Darwin stdenv seems to be ok, still waiting to see if this breaks kicad. |
KiCad fixed their CMake 3.11 incompatibility long before the 5.0.0 release, and both (KiCad fix in git mirror of upstream: KiCad/kicad-source-mirror@2e0b75c) |
Is cmake not using the provided libuv? It shouldn’t be building the vendored one. |
It's only using the vendored one when building with |
@GrahamcOfBorg build cmake |
Success on aarch64-linux (full log) Attempted: cmake Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: cmake Partial log (click to expand)
|
Timed out, unknown build status on x86_64-darwin (full log) Attempted: cmake Partial log (click to expand)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems nice. Darwin was thoroughly tested as well, apparently.
cmake 3.12 update in #44910 modified search-path.patch and broke cmake 2.8. This restores the previous version of the patch as search-path-2.8.patch
Motivation for this change
Keeping packages up to date. Additionally I want to use the new (as of 3.12)
target_link_libraries()
support for Object Libraries in a project.I assume this will trigger a mass-rebuild, and so should go to staging. Do I rebase on top of current staging (or staging-next) or leave it on top of master?
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)