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: 3.3.0 -> 3.5.0 #27707
calibre: 3.3.0 -> 3.5.0 #27707
Conversation
@jfrankenau, thanks for your PR! By analyzing the history of the files in this pull request, we identified @FRidh, @dezgeg and @layus to be potential reviewers. |
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.
Looks good to me. The expression needs to be moved and then the impact of the lxml
bump would have to be tested.
pkgs/top-level/python-packages.nix
Outdated
@@ -2352,6 +2352,28 @@ in { | |||
}; | |||
}; | |||
|
|||
html5-parser = buildPythonPackage rec { |
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.
See the header of this file. Please move to a separate expression.
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.
Done.
@@ -36,7 +36,7 @@ stdenv.mkDerivation rec { | |||
|
|||
# Remove unneeded files and libs | |||
rm -rf resources/calibre-portable.* \ | |||
src/{chardet,cherrypy,html5lib,odf,routes} |
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.
are these not vendored any more that you remove these items?
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.
Yes, these are not bundled with Calibre anymore.
51e6659
to
94ecad6
Compare
Is the |
Yes, it is. I've been playing around with |
@FRidh: ping? (you self-assigned this) |
Testing the Python packages that need to be rebuild at https://headcounter.org/hydra/eval/369772. Other rebuilds I cannot really test. |
I've pushed these commits to staging. |
Motivation for this change
Update.
Calibre now uses the new html5-parser package instead of html5lib. I have also removed some unused dependencies and installed the man pages.
Things done
Please check what applies. Note that these are not hard requirements but mereley serve as information for reviewers.
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)cc @viric @domenkozar @pSub @AndersonTorres