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.

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

Merged
merged 3 commits into from
Jun 16, 2017
Merged

Conversation

Hwesta
Copy link
Contributor

@Hwesta Hwesta commented Feb 28, 2017

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.

python setup.py sdist

package-wheel: package-deps
python setup.py bdist_wheel --universal
Copy link
Contributor Author

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?

Copy link
Contributor

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
Copy link
Contributor

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.

Copy link
Contributor

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
Copy link
Contributor

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.

@jhsimpson jhsimpson added this to the 1.3.6 milestone Jun 16, 2017
@jhsimpson jhsimpson merged commit 1cfe10e into openpreserve:master Jun 16, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants