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.

Sign up for GitHub

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

Jump to bottom

License is dynamic #31

Closed
wants to merge 1 commit into from
Closed

License is dynamic #31

wants to merge 1 commit into from

Conversation

bnavigator
Copy link

@chenrui333
Copy link

cc @Lucretiel

@kloczek
Copy link

kloczek commented Mar 8, 2024

Just found that this PR fixes as well my #32.
Is it possible to merge this PR and release new version? 🤔

algitbot pushed a commit to alpinelinux/aports that referenced this pull request Mar 20, 2024
@ncopa
community/py3-autocommand: fix build
39e9afd 
Upstrem: Lucretiel/autocommand#31
@jaraco
Copy link
Contributor

jaraco commented Apr 4, 2024

The use of the license field in setup.py is meant to be used only to specify a non-standard license. Since this project has a LICENSE file, that should be preferred. My recommendation as maintainer of setuptools, remove the offending license line from setup.py.

@jaraco jaraco mentioned this pull request Apr 4, 2024
Open
@bnavigator
Copy link
Author

#33 is the better fix

@bnavigator bnavigator closed this Apr 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chenrui333 chenrui333 chenrui333 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Cannot install from source distribution on PyPI
4 participants
@bnavigator @chenrui333 @kloczek @jaraco