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

Fix pip appveyor #529

Merged
merged 2 commits into from
Apr 22, 2018
Merged

Fix pip appveyor #529

merged 2 commits into from
Apr 22, 2018

Conversation

bryanwweber
Copy link
Member

@bryanwweber bryanwweber commented Apr 21, 2018

Changes proposed in this pull request:

  • Fix errors on Appveyor because of outdated/removed options to pip

Should we use Python 3.6 for the builds and tests here? It's probably a good idea to keep testing with Python 3.5 as long as we support that, but I'm not sure we test with Python 3.6 anywhere else (except the conda packages, I guess)

@speth
Copy link
Member

speth commented Apr 21, 2018

Oh, so SCons can finally be installed without the --egg option? Good. The last time I tried, that was not the case. A minor point, but I would suggest that the commit message note that this option is no longer required, rather than kind of implying that we were using a deprecated and now removed option for no apparent reason.

I don't think there's much difference in testing with Python 3.5 vs 3.6. Testing with an older version at least makes sure that we don't accidentally use features that were only introduced in Python 3.6. I know I do all my local development and testing using Python 3.6, so this might be the main source of testing for Python 3.5. I'd rather not expand the build matrix, just because that would slow down all the CI builds.

pip2 and pip3 used the same cache directory, resulting in a cache
invalidation warning because the Python 2 version of a package is not
compatible with Python 3. Using --no-cache-dir is recommended in
pypa/pip#5250
--egg is removed in pip >= 10.0.0 and is no longer necessary to
to install SCons with more recent versions of setuptools
@bryanwweber
Copy link
Member Author

I think the change is that more recent versions of SCons have support for more recent versions of setuptools (or maybe require more recent versions of setuptools). Also, installing from the wheel (because the wheel package is installed) rather than the sdist may have helped. In any case, we would have to downgrade pip on Appveyor (or maybe just not upgrade pip) to keep using the --egg option, since on #527 it seems like the error is because the option was removed.

Anyways, this is squashed and should be good to merge now.

@bryanwweber bryanwweber merged commit b1a010e into Cantera:master Apr 22, 2018
@bryanwweber bryanwweber deleted the fix_pip_appveyor branch April 25, 2018 13:39
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