-
Notifications
You must be signed in to change notification settings - Fork 27
Fix building package with venv #112
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
base: master
Are you sure you want to change the base?
Conversation
When building the package, the build fails as it cannot find requirements.txt in the package. Fixed this by moving the requirements into setup.py
Hi how did you invoke setup.py ? because my release process in Github Actions works as expected. Also, I'd like to keep the requirements separate from setup.py, so either I'll find a solution for this issue, or make a switch to Poetry and pyproject.toml |
Hi |
I suspected you wanted to keep requirements.txt. Problem is, I could not get setuptools to include it in the generated tarball but not have it be part of the package. Thats why the current solution |
What build module are you refering to ?
|
I'm using the instructions as detailed here: https://packaging.python.org/tutorials/packaging-projects/
|
Hi
I tried creating a package on my system and found that the build process fails:
This is caused by requirements.txt not being part of the package. Adding it to the package caused problems when installing as it appears one directory lower than the package directory (checksec). To fix this I moved the requirements into the setup.py.
Hope this is considered to be merged.
Thanks
Rogier