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: fix build process #21

Merged
merged 4 commits into from
Dec 7, 2017
Merged

travis: fix build process #21

merged 4 commits into from
Dec 7, 2017

Conversation

kortschak
Copy link
Member

@kortschak kortschak commented Dec 7, 2017

We have been seeing no-output time outs for OpenBLAS recently. This is an attempt to mitigate that.

During looking at the process I noticed that we don't seem to output the OpenBLAS sha1 anymore, the second commit restores that behaviour.

Finally, it appears that travis-ci have changed how they are shelling things and setting CGO_LDFLAGS only once at the site of install does not impact on the env of the go test invocation, so it is added there in the third commit.

Please take a look.

@kortschak
Copy link
Member Author

kortschak commented Dec 7, 2017

For some reason that I don't understand the allow_failures is not working for this repo; the master branch should appear in a separate section of the travis build page (this is what happens for gonum/gonum). I cannot see any difference between the two repos in this section of .travis.yml.

Resolved. There were two matrix: stanzas and that is not OK.

@kortschak kortschak changed the title travis: wait longer for OpenBLAS build travis: fix build process Dec 7, 2017
@kortschak
Copy link
Member Author

I don't really understand why we are installing CBLAS from netlib when we build OpenBLAS, but I'm going to leave that for another PR.

Copy link
Member

@vladimir-ch vladimir-ch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should also update the README (it still uses github.com at one place, references cgo package, ...).

@kortschak
Copy link
Member Author

I'll make those changes in the next PR.

@kortschak kortschak merged commit 9f546bb into master Dec 7, 2017
@kortschak kortschak deleted the wait-longer branch December 7, 2017 20:09
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