Skip to content
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

Closed
dkg opened this issue Jan 26, 2023 · 8 comments
Closed

DOC: migration guide is out of date, README pointer is confusing #1581

dkg opened this issue Jan 26, 2023 · 8 comments
Labels
nf-documentation Non-functional change: Documentation

Comments

@dkg
Copy link
Contributor

dkg commented Jan 26, 2023

In README.md, it says:

NOTE: pypdf>=3.1.0 improved a lot compared to pyPdf<2.0.0 and compared to PyPDF2 < 2.0.0. Please read the migration guide.

This is confusing in a couple ways:

  • i think the PyPDF < 2.0.0 is supposed to be PyPDF2 < 3.0.0
  • the link points to a migration from v1 to v2, but the text implies that this is about the migration from v2 to v3

I'm trying to encourage other packages in debian to migrate from PyPDF2 to pypdf, 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.

@MartinThoma
Copy link
Member

MartinThoma commented Jan 26, 2023

There are three groups that need to do migrations:

  • pyPdf users
  • PyPDF2 < 2.0.0 users
  • PyPDF2 >= 2.0.0 users

In chronological order, without skipping anything, the switches would be:

  • Upgrade from pyPdf to PyPDF2 < 2.0.0: Almost no work. Just replace pyPdf by PyPDF2
  • Upgrade from PyPDF2<2.0.0 to PyPDF2>=2.0.0: A lot of renamings; see the migration guide
  • Upgrade from PyPDF2>=2.0.0 to PyPDF2>=3.0.0: No work, if you did the migration before (it just actually removed stuff)
  • Upgrade from PyPDF2>=3.0.0 to pypdf>=3.1.0: Just replace PyPDF2 by pypdf.

So there really only is one "big" migration. As mostly names changed, I think all of those migrations are pretty trivial.

@MartinThoma MartinThoma added the nf-documentation Non-functional change: Documentation label Jan 26, 2023
@MartinThoma MartinThoma changed the title migration guide is out of date, README pointer is confusing DOC: migration guide is out of date, README pointer is confusing Jan 26, 2023
@MartinThoma
Copy link
Member

I'm trying to encourage other packages in Debian to migrate from PyPDF2 to pypdf

Thank you 🙏

Do you have a list of those?

@dkg
Copy link
Contributor Author

dkg commented Jan 26, 2023

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

@dkg
Copy link
Contributor Author

dkg commented Jan 26, 2023

oh, and i generated the list of reverse dependencies in debian with apt rdepends python3-pypdf2 from a debian testing/unstable system, in case that's what you're asking.

@MartinThoma
Copy link
Member

MartinThoma commented Jan 27, 2023

i generated the list of reverse dependencies in debian with apt rdepends python3-pypdf2 from a debian testing/unstable system, in case that's what you're asking

Yes!

Finished ✔️

In progress ⏳

Dead projects 💀

@dkg
Copy link
Contributor Author

dkg commented Jan 31, 2023 via email

@MartinThoma
Copy link
Member

I was looking at https://github.com/unikraft/kraft/tree/staging/kraft, but that was the wrong one.
I've opened an issue there: dragotin/kraft#199

@MartinThoma
Copy link
Member

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
nf-documentation Non-functional change: Documentation
Projects
None yet
Development

No branches or pull requests

2 participants