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 python_requires>=3.6 and a conda recipe #46

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

analog-cbarber
Copy link

This adds a python_requires>=3.6 attribute to setup.py.

The code already assumes at least python 3.6, so we may just as well make it explicit in the package metadata.

The conda.recipe can be used to directly build a conda package that you can test locally or deploy to private channels.

@analog-cbarber analog-cbarber requested a review from a team as a code owner July 8, 2021 16:07
path: ..

build:
script: python setup.py install --single-version-externally-managed --record=record.txt
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
script: python setup.py install --single-version-externally-managed --record=record.txt
script: pip install -vv .

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These changes actually break the script install. It ends up requiring the version of python used to build it.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The recipe produces a noarch python recipe that can be installed on any platform or supported version of python. But if the suggested changes are made, the resulting conda-recipe script will only be runnable on the version of python it was built on.

conda.recipe/meta.yaml Show resolved Hide resolved
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.

2 participants