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
python.pkgs.graphviz: hardcode path to graphviz's bin/ #52523
Conversation
|
||
postPatch = '' | ||
substituteInPlace pydot.py \ | ||
--replace "program_with_args = [program" "program_with_args = ['${graphviz}/bin/' + program" \ |
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.
Should I better use os.path.join
here and in other places?
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.
From this line it's hard to see what's happening or how it should look like. Considering we don't support Windows, I don't think it matters much.
I think the fact that |
ping @FRidh |
I'm currently struggling to convert this into a patch:
Does someone have an idea how to solve this? Apllying |
@dotlambda I was able to get this patch to apply by using the github repo. |
a8c7aca
to
354b0d4
Compare
@GrahamcOfBorg build python2.pkgs.graphviz python3.pkgs.graphviz python2.pkgs.objgraph python3.pkgs.objgraph python2.pkgs.pydot python3.pkgs.pydot |
According to nix-review, this does not introduce any additional build failure. So I'm merging this. |
Motivation for this change
It's not very nice to have
pkgs.graphviz
inpropagatedBuildInputs
.closes #53224
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)