-
Notifications
You must be signed in to change notification settings - Fork 43
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
setup.cfg: fix py_modules #44
base: master
Are you sure you want to change the base?
Conversation
I don't think this does anything we want. pip-autoremove is designed to be a user-run script, not an imported module, as per the documentation:
What are you trying to achieve with this? We may need to add the
[edit.] Actually we already definite the |
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.
Tested on Mac OS 11 and Debian, this works fine for me. I don't have a windows box to test with so have to take @imba-tjd 's word that it works there too.
The original setup.py uses py_modules, see https://github.com/invl/pip-autoremove/pull/42/files#diff-60f61ab7a8d1910d86d9fda2261620314edcae5894d5aaa236b821c7256badd7L11 I'm confident this will solve the issue on windows. Or you can have a try by
|
The launcher in Linux is a python script. In Windows it is an exe file. With python script the first entry in But with an exe launcher, the first entry in Anyway since |
I'm on Windows I moved the pip_autoremove.py file from the \Scripts path into ### \Lib\site-packages |
worked for me too, on Windows, Python 3.10. also works for > pip-autoremove some_package |
fix #43
cc @tresni