search

hechtus / mopidy-radio-de

radio.de Extension for Mopidy
Apache License 2.0
11 stars 9 forks source link

Add proxy support #6

Closed AlexandrePTJ closed 10 years ago

hechtus commented 10 years ago

Thanks! Do you have an idea how to solve the Travis CI build error?

AlexandrePTJ commented 10 years ago

I think that you don't need to install mopidy==dev throut pip as it is already installed from apt.

2014-02-17 9:52 GMT+01:00 Ronald Hecht notifications@github.com:

Thanks! Do you have an idea how to solve the Travis CI build error?

Reply to this email directly or view it on GitHubhttps://github.com/hechtus/mopidy-radio-de/pull/6#issuecomment-35238352 .

Cordialement, Alexandre Petitjean.

jodal commented 10 years ago

The Travis CI error is caused by updated pip on their servers. In .travis.yml, change the pip line to:

pip install --allow-unverified=mopidy coveralls flake8 mopidy==dev python-dateutil
jodal commented 10 years ago

Yes, you do. The install from apt-get is only to get the non-pure-Python dependencies installed. The Mopidy install from the Debian package goes into /usr/share/mopidy, and won't be found by Python code installed by pip or run from Git repos without tweaking PYTHONPATH. We also want extensions to test against Mopidy's development branch, thus we install Mopidy both with apt-get and pip.

hechtus commented 10 years ago

@jodal Thanks for clarification. Just fixed it!