-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
DOC: migration guide is out of date, README pointer is confusing #1581
Comments
There are three groups that need to do migrations:
In chronological order, without skipping anything, the switches would be:
So there really only is one "big" migration. As mostly names changed, I think all of those migrations are pretty trivial. |
Thank you 🙏 Do you have a list of those? |
This is the primary bug report: https://bugs.debian.org/1029732 There are 10 downstream bugs open, and i've just supplied a patch which should fix two of them in diffoscope. you should be able to see a summary of them on the src:pypdf page of the debian BTS |
oh, and i generated the list of reverse dependencies in debian with |
* Depends: kraft - I don't find any pypdf dependency
It's in tools/erml2pdf.py and tools/watermarkpdf.py, i think.
|
I was looking at https://github.com/unikraft/kraft/tree/staging/kraft, but that was the wrong one. |
I don't think there is anything we can do about this topic anymore. If anybody has concrete suggestions, please raise them. For the moment, I will close this issue. |
In
README.md
, it says:This is confusing in a couple ways:
I'm trying to encourage other packages in debian to migrate from
PyPDF2
topypdf
, and i'd like to point maintainers of those packages to a clean migration guide. Does something like that exist?Without a clear upstream migration guide, i'm inclined to suggest something like "install the last version of PyPDF2 (2.12.1), run your full test suite, and note and resolve any PendingDeprecationWarning that you see."
If you have a better suggestion, i'd be happy to point them to it.
The text was updated successfully, but these errors were encountered: