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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
tinyproxy: init at 1.8.4 #44342
tinyproxy: init at 1.8.4 #44342
Conversation
@GrahamcOfBorg build tinyproxy |
Failure on x86_64-linux (full log) Attempted: tinyproxy Partial log (click to expand)
|
Failure on aarch64-linux (full log) Attempted: tinyproxy Partial log (click to expand)
|
Looks like building the manpages fails. |
Success on x86_64-darwin (full log) Attempted: tinyproxy 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.
fix failing builds
Thanks for reviewing @xeji, I鈥檒l revisit this and fix the manpages issue |
Please also make sure you build packages locally with sandboxing enabled. This makes it much more likely that if it builds locally, it will also build on our CI. |
Managed to reproduce by turning off my internet connection:
It happens because xsltproc can't find some DTDs (apparently) and reaches to the internet to get them. However just adding a nonet option and a native dependency on docbook_xsl should fix it (testing locally seems fine using NixOS). Thanks again @xeji |
@GrahamcOfBorg build tinyproxy |
Success on x86_64-linux (full log) Attempted: tinyproxy Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: tinyproxy Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: tinyproxy Partial log (click to expand)
|
Thank you, LGTM now.
That's one reason to enable sandboxed builds on your local machine. The sandbox denies network access, so you would have caught this error right away. |
Motivation for this change
Tinyproxy is great 馃槃
This should compile for all platforms.
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)