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
dbus: build manpages from xml #99018
Conversation
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.
LGTM after a squash.
Also this should target staging branch as it causes a mass rebuild. |
4f53515
to
73aed95
Compare
I have squashed and rebased onto staging. I have no idea why GitHub did that mass review request. Edit: My best guess is that after I force pushed, but before I changed the base branch in GitHub, this PR included a whole bunch of unrelated commits for a few seconds. Sorry for the unnecessary pings. |
The `docbook_xml_dtd_44` and `docbook-xsl-nons` bits are necessary to prevent `xmlto` from trying to use the network during the build.
73aed95
to
aba3abc
Compare
This is what staging-next is for. I would just ensure that the package you're affecting has the intended changes in it. |
Motivation for this change
I noticed that manual pages like
dbus-send(1)
anddbus-monitor(1)
were missing from my system.The
docbook_xml_dtd_44
anddocbook_xsl
bits are necessary to preventxmlto
from trying to use the network during build.I copied from other nix packages that have the same issue.(Edit: Did it a cleaner way.)Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)I don't think my machine is powerful enough to run the
nixpkgs-review
for this change, given the sheer number of packages that depend ondbus
. I confirmed the manpages were generated correctly and that simple binaries likedbus-send
,dbus-monitor
, anddbus-uuidgen
work (I'm not sure how to use or test the others).