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
pythonPackages.pdfx: init at 1.3.1 #42815
Conversation
8fdb80f
to
ae38bb3
Compare
}; | ||
|
||
prePatch = '' | ||
sed -i -e "s|pdfminer2|pdfminer.six|" setup.py |
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.
I think you should package pdfminer2 instead of using an old version.
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.
Wasn't pdfminer.six
explicitly replaced in metachris/pdfx@cee76e1?
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.
Wasn't pdfminer.six explicitly replaced in metachris/pdfx@cee76e1?
I didn't knew that, but seems that there is no significant changes in pdfminer2 pdfminer/pdfminer.six@master...metachris:master.
pdfx works with pdfminer.six, but i can package pdfminer2 if you prefer it
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.
Let's first wait for the maintainer's answer to metachris/pdfx#28.
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.
As the package seems unmaintained, I'd say we just go with your change. However, please add a link to your PR as a comment.
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.
Sorry for the delay, I've added a comment with the link
pname = "pdfx"; | ||
version = "1.3.1"; | ||
|
||
src = fetchFromGitHub { |
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.
I have opened metachris/pdfx#29 so that fetchPypi
can be used.
ae38bb3
to
5b718c0
Compare
|
||
meta = with stdenv.lib; { | ||
inherit (src.meta) homepage; | ||
description = "Extract references (pdf, url, doi, arxiv) and metadata from a PDF."; |
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.
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.
fixed :)
5b718c0
to
6b8b6b6
Compare
@GrahamcOfBorg build python2.pkgs.pdfx python3.pkgs.pdfx |
Success on x86_64-linux (full log) Attempted: python2.pkgs.pdfx, python3.pkgs.pdfx Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: python2.pkgs.pdfx, python3.pkgs.pdfx Partial log (click to expand)
|
Motivation for this change
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)