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
djvulibre: {adopt, clean-up, upd description} #69271
Conversation
I woopsed that PR, so here is clean one. Already squashed commits. |
@GrahamcOfBorg eval |
@GrahamcOfBorg eval |
Guys, if you can believe it, I with Hoeg already talked and tested it in #69046. Hoeg asked to make cosmetic changes, those changes triggered a rebuild of package, and since that - also a sub-graph, including all versions of There is why moved to P.S. |
The plan was to avoid another big rebuild when the split goes in. |
Well, sorry, I perceived the discussion as i wrote. The code is clear aslo. I don't want to see how people waste their time trying to evaluate cosmetics that already was evaluated and reviewed. I want to do atomic PRs. I would not turn completed cosmetic coffee-break devoted PR into recursive yak shaving PR. I also wrote a big hearty letter about why I do not do that, but I put it the table. Life thought me that. But it is also obvious why big PRs should not happen. As soon as work done and reviewed - I would send/accept what is improved, and do another atomic change. I hope you all have a calm time. It feels that the community is tired. NixPkgs achieved the max 12.7 warp speed, while 10 warp - is a speed on Linux Kernel changes. Thank you. I wrote a bug-report screen play in the issue tracker, to explain the issue with |
There is a mass-rebuild in staging anyway. So it does not matter in the end. |
Motivation for this change
djview
package. But there is all other tools for management.djvulibre
and often statically include it, and most utils seems to depend ondjvulibre
binaries that are very unix-like flexible.The value of package is all utils for DJVU management.
New description of the package much more on point, package has a set of binaries and mans for them.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)