search

enthought / ensemble

High-level widgets for building Python applications
Other
4 stars 7 forks source link

Test--try to fix Travis failure #37

Closed noraderam closed 9 years ago

noraderam commented 9 years ago

Travis failures may be due to pip version change. Try removing deprecated flag.

jwiggins commented 9 years ago

Travis is failing because of an import error when building Enable. Maybe look at the recent changes made to dev_requirements.txt in Enable?

It's failing to find ETSConfig, which is part of Traits, which is installed before Enable. I'm not really sure how that's happening.

noraderam commented 9 years ago

From what I've been able to determine, traits is being cloned before enable, but there's no sign of it being actually installed. Looking in a different Travis run--https://travis-ci.org/enthought/enable/jobs/55853416--Travis is first "obtaining" packages; later it will say "Installing collected packages". I don't know why enable is being installed right away.

noraderam commented 9 years ago

Chaco has a similar .travis.yml and dev_requirements.txt. The most recent build there was successful but used an older version of pip (1.5.6 rather than 6.0.7, it's from December).

jwiggins commented 9 years ago

Hmm... I'm at a loss on this one. I started reading through pip's source code, but I don't really have time for that right now.

I think that pip is running setup.py immediately after cloning, but with a non-build command. If that's really the case, there are a couple of solutions. You could redo the development versioning PR in Enable (enthought/enable#166) so that it gets the version from Enable's __init__.py instead of Kiva's. Or, you could break up the dev_requirements.txt file here so that it gets Enable's dependencies first, then gets Enable.

I don't really love either of those solutions, so I'm open to suggestions.

itziakos commented 9 years ago

I think that the real problem here is that the setup.py of enable does not state that traits is a requirement (https://github.com/enthought/enable/blob/master/setup.py#L248). As a result pip does not know that it has to install traits before enable.

noraderam commented 9 years ago

Chaco has a similar .travis.yml and dev_requirements.txt. The most recent build there was successful but used an older version of pip (1.5.6 rather than 6.0.7, it's from December).

Let me amend that, I didn't see the most recent build. The same failure happens there: https://travis-ci.org/enthought/chaco/jobs/51650294