Skip to content

Update pip and setuptools on TravisCI; fix travis_init#521

Merged
bgruening merged 2 commits intogalaxyproject:masterfrom
peterjc:travis_pip_update
Jul 22, 2016
Merged

Update pip and setuptools on TravisCI; fix travis_init#521
bgruening merged 2 commits intogalaxyproject:masterfrom
peterjc:travis_pip_update

Conversation

@peterjc
Copy link
Contributor

@peterjc peterjc commented Jul 22, 2016

Fixes #520 by explicitly updating pip and setuptools before installing planemo.

Also fixes a regression I introduced recently in 364fec2 (typo in function name)

peterjc added 2 commits July 22, 2016 10:06
Closes galaxyproject#520 where the dependency chain planemo, cwltool, rdflib,
html5lib failed as that needed setuptools version 18.5 or later.
This was a recent mistake of mine, I must not have have
the git pre-commit hook installed on the machine I used
as that caught both points:

planemo/commands/cmd_travis_init.py:49:1: E302 expected 2 blank lines, found 1
planemo/commands/cmd_travis_init.py:81:9: F821 undefined name 'warning'
@peterjc peterjc mentioned this pull request Jul 22, 2016
@nsoranzo
Copy link
Member

Upgrading setuptools should be enough, we had to do the same for tools-iuc TravisCI: galaxyproject/tools-iuc#887

@peterjc
Copy link
Contributor Author

peterjc commented Jul 22, 2016

Possibly updating setuptools only is enough right now, but I've been bitten by an old pip before with other TravisCI setups, so this seems a sensible precautionary measure.

@bgruening
Copy link
Member

CWL errors again. I think it's fine to merge this.
Thanks @peterjc!

@bgruening bgruening merged commit d0d21e5 into galaxyproject:master Jul 22, 2016
@peterjc peterjc deleted the travis_pip_update branch July 22, 2016 11:31
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.

pip install planemo fail

3 participants