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

Travis: fix the latest breakage - use group: #5055

Merged
merged 1 commit into from Sep 8, 2017

Conversation

Projects
None yet
3 participants
@0xc0170
Member

0xc0170 commented Sep 8, 2017

travis testing

@0xc0170 0xc0170 added the do not merge label Sep 8, 2017

@0xc0170 0xc0170 force-pushed the 0xc0170:fix_ssltravis branch 3 times, most recently Sep 8, 2017

@0xc0170 0xc0170 force-pushed the 0xc0170:fix_ssltravis branch to 42a6dd0 Sep 8, 2017

@0xc0170 0xc0170 changed the title from test to Travis: fix the latest breakage - use group: Sep 8, 2017

@0xc0170 0xc0170 added needs: CI and removed do not merge labels Sep 8, 2017

@0xc0170

This comment has been minimized.

Member

0xc0170 commented Sep 8, 2017

cc @theotherjimmy Seems like we got a fix, however I would like to know what's next from travis ? :-)

@theotherjimmy

This comment has been minimized.

Contributor

theotherjimmy commented Sep 8, 2017

... any day now Oulu-CI.

@geky

This comment has been minimized.

Member

geky commented Sep 8, 2017

Is there more info in this issue? This is causing a lot of travis jobs to fail.

@theotherjimmy

This comment has been minimized.

Contributor

theotherjimmy commented Sep 8, 2017

Looks like it's working!

@theotherjimmy theotherjimmy merged commit e5638b7 into ARMmbed:master Sep 8, 2017

3 checks passed

Cam-CI uvisor Build & Test Success
Details
continuous-integration/jenkins/pr-head This commit looks good
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@theotherjimmy theotherjimmy removed the needs: CI label Sep 8, 2017

@geky

This comment has been minimized.

Member

geky commented Sep 8, 2017

Using sudo also seems to solve the issue, I'm not sure which is correct for future compat with travis/python.

@geky

This comment has been minimized.

Member

geky commented Sep 8, 2017

From what I can tell, we actually want this:

pip install --user -r mbed-os/requirements.txt

I can create another pr

@0xc0170

This comment has been minimized.

Member

0xc0170 commented Sep 9, 2017

@geky The issue is described in the commit message , I provided there a link. Thanks for looking at this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment