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
yosys: fix build on darwin #38826
yosys: fix build on darwin #38826
Conversation
Success on aarch64-linux (full log) Attempted: yosys Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: yosys Partial log (click to expand)
|
@@ -32,7 +32,7 @@ stdenv.mkDerivation rec { | |||
|
|||
enableParallelBuilding = true; | |||
nativeBuildInputs = [ pkgconfig ]; | |||
buildInputs = [ tcl readline libffi python3 bison flex ]; | |||
buildInputs = [ gcc tcl readline libffi python3 bison flex ]; |
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.
Why does this need gcc?
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.
I guess you should ask the authors…
There is a “config-clang” target in the makefile, but I did not manage to get the build go through (I got an error related to C++ header files). Should I investigate further or is the work-around of using GCC acceptable?
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.
Sorry for the delay. Just adding gcc doesn't work properly so I don't think it's actually needed. But if you don't want to spend more time on this, overriding the stdenv in all-packages.nix is the proper way to use gcc in all platforms.
{
foo = callPackage ./foo { stdenv = gccStdenv; };
}
Success on aarch64-linux (full log) Attempted: yosys Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: symbiyosys, yosys Partial log (click to expand)
|
Thanks for the tip, @LnL7. Meanwhile I figured out how to fix the build of |
Success on x86_64-linux (full log) Attempted: symbiyosys, yosys Partial log (click to expand)
|
Motivation for this change
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)