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
rstudio: fix build to use compatible boost version #42383
Conversation
@GrahamcOfBorg build rstudio |
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: rstudio Partial log (click to expand)
|
Failure on x86_64-linux (full log) Attempted: rstudio Partial log (click to expand)
|
Build error is unrelated to this PR, see https://hydra.nixos.org/build/76464044, and should be fixed first. |
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: rstudio Partial log (click to expand)
|
3a83195
to
cc29280
Compare
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.
Thank you!
rstudio = libsForQt5.callPackage ../applications/editors/rstudio { }; | ||
rstudio = libsForQt5.callPackage ../applications/editors/rstudio { | ||
boost = boost166; | ||
}; |
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.
Versions should be overridden in all-packages.nix
, like this. It makes it easier to spot and remove outdated overrides.
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.
Thanks, much clearer.
Motivation for this change
This fixes compile errors, currently hidden until a compatible qt5 version (5.11.1) is used (which is not the case in master yet: #42351 ).
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)