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
[RDY] cmake: don't override cmakeDir when set #40687
Conversation
Success on x86_64-linux (full log) Attempted: cmake Partial log (click to expand)
|
Success on aarch64-linux (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.
Looks good to me, but this should go to staging first.
When dontUseCmakeBuildDir is true (aka the default), it overrides cmakeDir regardless of the package configuration. While packaging netbee, I needed to both keep dontUseCmakeBuildDir to true (some hardcoded paths expect the build folder) and set cmakeDir (since CMakeList.txt was in a subfolder) which proved impossible. Here is the fix.
Success on x86_64-linux (full log) Attempted: cmake Partial log (click to expand)
|
I hadn't realize I could change the branch myself. It's done. |
Success on aarch64-linux (full log) Attempted: cmake Partial log (click to expand)
|
@LnL7 is that ok ? |
When dontUseCmakeBuildDir is true (aka the default), it overrides
cmakeDir regardless of the package configuration.
While packaging netbee, I needed to both keep dontUseCmakeBuildDir to
true (some hardcoded paths expect the build folder) and set cmakeDir
(since CMakeList.txt was in a subfolder) which proved impossible.
Here is the fix.
Motivation for this change
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)