-
Notifications
You must be signed in to change notification settings - Fork 358
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
scripts/pythondistdeps: Only add setuptools requirement for egg-info … #973
scripts/pythondistdeps: Only add setuptools requirement for egg-info … #973
Conversation
LGTM. @hroncok r? |
Pretty straight forward. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Damn GitHub reviews, this comment was hanging here for 23 days. Sorry about that.
dist.get_entry_map('gui_scripts')): | ||
if ((dist.get_entry_map('console_scripts') or | ||
dist.get_entry_map('gui_scripts')) and | ||
(lower.endswith('.egg') or |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't know about egg. Can it even have entrypoints?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think so... If I "bdist_egg" the molecule package, and then install the resulting egg, the console_script is installed. However, I don't get useful results from this script on egg files. I'm unclear on whether we ever package egg (or wheel) files, so I'm not actually sure whether that's something that needs to be reviewed and fixed, or dropped.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK. we can keep it be, until proven otherwise.
- Handle version ending with ".*" - Handle compatible-release operator "~=" - Use rich deps for semantically versioned dependencies - Match Python version if minor has multiple digits (e.g. 3.10) - Only add setuptools requirement for egg-info packages rpm-software-management/rpm#951 rpm-software-management/rpm#973 rpm-software-management/rpm#982 Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1758141 Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1777382
scripts/pythondistdeps: Only add setuptools requirement for egg-info packages.
This should address issue #954