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
calibre: cssutils -> css-parser #53865
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.
Adding the python package should be a separate commit.
Taking your reviews into account, but closing this PR to split it into 2. |
No need for two PRs, just two commits. |
5dbf856
to
491b2bb
Compare
done, thanks for reviewing again @dotlambda |
491b2bb
to
a0e9290
Compare
Calibre has changed its dependency from cssutils to css-parser in kovidgoyal/calibre@dd7d8ea
a0e9290
to
4e1fb4f
Compare
@GrahamcOfBorg build python2.pkgs.css-parser python3.pkgs.css-parser |
@GrahamcOfBorg build calibre |
Motivation for this change
Updates an altered calibre dependency. Calibre now fails with:
because of a recent change kovidgoyal/calibre@dd7d8ea
This also introduces a new package css-parser, which calibre uses.
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)