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

blas/netlib: comment and test Zhpr2 #25

Merged
merged 4 commits into from
Dec 14, 2017
Merged

blas/netlib: comment and test Zhpr2 #25

merged 4 commits into from
Dec 14, 2017

Conversation

vladimir-ch
Copy link
Member

No description provided.

@vladimir-ch
Copy link
Member Author

Do you know what the failure is?

@kortschak
Copy link
Member

I suspect it's a conflict between this and this.

The line that does have a sudo prefix probably shouldn't, but it can probably be fixed by adding sudo to the other line.

@vladimir-ch
Copy link
Member Author

Removed sudo, pushed the commit, saw that the cache needs sudo to be removed, added sudo to the rm command, push-forced the amended commit, deleted sudo from the rm command, push-forced again, and ran go generate. But the checks pass now. PTAL

@kortschak
Copy link
Member

Yeah, I thought you were being a little optimisitic just removing the sudo and leaving the cache artifacts with root ownership.

The sudo command is a bit like heroin, it solves all your problems, but then you keep needing to use it.

@vladimir-ch vladimir-ch merged commit e672cac into master Dec 14, 2017
@kortschak
Copy link
Member

For future reference, you can clear the cache via the travis UI (https://docs.travis-ci.com/user/caching/). This was necessary for master because its cache is independent of the PR caches. I've done that now and restarted the master builds.

@vladimir-ch
Copy link
Member Author

That's good to know. I thought it was strange to control travis by pushing commits.

@vladimir-ch vladimir-ch deleted the zhpr2 branch December 15, 2017 09:55
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.

2 participants