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

Not all Python authors expressed? #35

Closed
proycon opened this issue Feb 27, 2023 · 4 comments
Closed

Not all Python authors expressed? #35

proycon opened this issue Feb 27, 2023 · 4 comments
Assignees
Labels
bug Something isn't working ready implemented/ready but still pending release

Comments

@proycon
Copy link
Member

proycon commented Feb 27, 2023

In https://tools.dev.clariah.nl/pagexml-tools we see only one author whereas the pyproject.toml has multiple. Maybe caused by PEP517 only supporting one? Investigate..

@proycon
Copy link
Member Author

proycon commented Feb 28, 2023

It seems importlib.metadata (standard library) only exposes a single author, even though technically multiple authors do appear in the PKG-INFO (comma separated). The second simply gets stripped it seems. This problem was also illustrated in proycon/codemetapy#28.

The core metadata specification makes no mention of supporting multiple authors. The field is not marked as being "multiple use" either.

However, pyproject.toml does support multiple authors so it seems we are, again referring to proycon/codemeta#28, not getting the full metadata as it goes beyond core metadata. We may need to parse pyproject.toml ourselves to get it (which I didn't really want to do).

@proycon
Copy link
Member Author

proycon commented Feb 28, 2023

This library may also be worth a try to get the information from the PKG-INFO: https://pypi.org/project/pkginfo/

@proycon
Copy link
Member Author

proycon commented Feb 28, 2023

On further investigation, I was wrong and do get multiple authors when invoking importlib.metadata (stdlib) or the importlib_metadata 3rd party package. Just like the PKG-INFO says. Only when mediated via the pep517 package is this information missing.

@proycon
Copy link
Member Author

proycon commented Feb 28, 2023

However, PKG-INFO and egg-info may be an older method (distutils, setuptools). On a project with a pyproject.toml with poetry, I don't seem to get one and I rely on pep517 :/

proycon added a commit to proycon/codemetapy that referenced this issue Mar 1, 2023
@proycon proycon added the ready implemented/ready but still pending release label Mar 1, 2023
@proycon proycon closed this as completed Mar 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ready implemented/ready but still pending release
Development

No branches or pull requests

1 participant