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

hooks: pyproj: collect metadata for pyproj >= 3.4.0 #505

Merged
merged 1 commit into from Nov 7, 2022

Conversation

rokm
Copy link
Member

@rokm rokm commented Nov 2, 2022

As of version 3.4.0, pyproj's __init__.py queries its own version via importlib.metadata.version(__package__), which requires the package metadata to be collected.

@bwoodsend
Copy link
Member

They did revert it (pyproj4/pyproj#1156). That's why I was putting off doing anything about it. I was hoping they'd release 3.4.1 then we could pretend 3.4.0 never happened.

@rokm
Copy link
Member Author

rokm commented Nov 2, 2022

Ah, fair enough. Let's ignore 3.4, then.

@rokm
Copy link
Member Author

rokm commented Nov 7, 2022

If pyproj 3.4.1 is happening on December 1st, this means we have a whole-month-long window during which we'll be dealing with bug reports about this.

Therefore, I'm reopening this PR, but I've changed the version check from >= 3.0.0 to == 3.4.0 to limit metadata collection only to the problematic release.

We will need to push out new hooks-contrib release, though...

In version 3.4.0, pyproj's `__init__.py` queries its own version
via `importlib.metadata.version(__package__)`, which requires the
package metadata to be collected. This change seems to have been
already reverted, so it will not affect subsequent pyproj releases.
@bwoodsend
Copy link
Member

Go ahead and hit the release button after merging.

@bwoodsend bwoodsend linked an issue Nov 7, 2022 that may be closed by this pull request
6 tasks
@rokm rokm merged commit b35fb38 into pyinstaller:master Nov 7, 2022
@rokm rokm deleted the hook-pyproj-3.4.0 branch November 7, 2022 17:24
@rokm
Copy link
Member Author

rokm commented Nov 7, 2022

I'll try to find out what the deal with double headings in ChangeLog is, and then make a release.

@bwoodsend
Copy link
Member

It'll be the towncrier template again. This is at least the 4th time they've changed their minds on whether the headings should be specified in the template or inserted automatically.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Problem with pyproj 3.4.0 , package not found
2 participants