-
Notifications
You must be signed in to change notification settings - Fork 47
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
Add Makefile for packaging #95
Conversation
python setup.py sdist | ||
|
||
package-wheel: package-deps | ||
python setup.py bdist_wheel --universal |
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'm not sure the --universal
flag is correct here, since we only partially support Python 3. (See https://packaging.python.org/distributing/#universal-wheels) However, if we remove it, I suspect we'll forget to add it back once Python 3 support is more robust. Thoughts?
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'm fine keeping it right now for the sake of testing, as long as we've clearly documented the Python 3 support as WIP.
Makefile
Outdated
.PHONY: clean package package-deps package-source package-upload package-wheel | ||
|
||
package-deps: | ||
pip install --upgrade twine wheel |
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.
Let's set versions on these for ease of reproduceability.
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.
Alternately, maybe make a development requirements.txt
?
python setup.py sdist | ||
|
||
package-wheel: package-deps | ||
python setup.py bdist_wheel --universal |
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'm fine keeping it right now for the sake of testing, as long as we've clearly documented the Python 3 support as WIP.
Add a makefile to simplify making and uploading a package to PyPI. I haven't used this for a FIDO release yet, but have used a similar makefile for other projects.
To use, run
make package
from the directory with the Makefile.make clean
will remove all packaging related artifacts.