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

Travis builds failing for OSX & beta #490

Closed
Michael-F-Bryan opened this issue Nov 18, 2017 · 7 comments
Closed

Travis builds failing for OSX & beta #490

Michael-F-Bryan opened this issue Nov 18, 2017 · 7 comments
Labels
A-Infrastructure Area: CI, Releases C-papercut Category: A small usability bug

Comments

@Michael-F-Bryan
Copy link
Contributor

For some reason we're encountering a permissions error when running the OSX/beta travis build. An example is trying to merge #465 or #422.

This is the error message:

screenshot_2017-11-16_150757

@budziq or @steveklabnik, have you got any idea what may be causing this?

@Michael-F-Bryan Michael-F-Bryan added A-Infrastructure Area: CI, Releases C-papercut Category: A small usability bug labels Nov 18, 2017
@budziq
Copy link
Contributor

budziq commented Nov 18, 2017

@Michael-F-Bryan on first glance it looks like rust-lang/cargo#4699

@Michael-F-Bryan
Copy link
Contributor Author

Michael-F-Bryan commented Nov 18, 2017

I added a couple debug commands to see what's going on in that directory and that seems to be the issue.

screenshot_2017-11-19_000033

You can see that mdbook-$hash.dSYM is a directory whereas cargo is only trying to remove files, according to that issue you linked.

I'm thinking the easiest solution is to temporarily remove the OSX job until rust-lang/cargo#4699 is solved. @budziq, can you think of any other workaround?

@budziq
Copy link
Contributor

budziq commented Nov 18, 2017

We can either disable osx (or just beta) temporarily or just wait doing nothing untill cargo gets fixed. I'd probably just disable the beta for now as the least valuable check.

But I'm game either way.

@Michael-F-Bryan
Copy link
Contributor Author

I think I'll disable beta because you tend to only ever use stable or nightly.

@budziq
Copy link
Contributor

budziq commented Nov 23, 2017

Looking at the 1.22.1 release notes, the problem with cargo on MacOS beta should be fixed.

@sorin-davidoi
Copy link
Contributor

@Michael-F-Bryan I guess this can be closed?

@Michael-F-Bryan
Copy link
Contributor Author

Yep.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Infrastructure Area: CI, Releases C-papercut Category: A small usability bug
Projects
None yet
Development

No branches or pull requests

3 participants