-
Notifications
You must be signed in to change notification settings - Fork 49
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-ci: build with latest autotools #1016
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1016 +/- ##
==========================================
+ Coverage 77.6% 77.61% +<.01%
==========================================
Files 152 152
Lines 25787 25787
==========================================
+ Hits 20013 20014 +1
+ Misses 5774 5773 -1
Continue to review full report at Codecov.
|
src/test/travis-dep-builder.sh
Outdated
@@ -16,7 +16,10 @@ https://github.com/zeromq/czmq/archive/v3.0.2.tar.gz \ | |||
http://downloads.sourceforge.net/ltp/lcov-1.10.tar.gz \ | |||
http://www.open-mpi.org/software/hwloc/v1.11/downloads/hwloc-1.11.0.tar.gz \ | |||
http://www.digip.org/jansson/releases/jansson-2.9.tar.gz \ | |||
http://www.mpich.org/static/downloads/3.1.4/mpich-3.1.4.tar.gz" | |||
http://www.mpich.org/static/downloads/3.1.4/mpich-3.1.4.tar.gz |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should there be a "\" contiunation at the end of this line?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, I suppose so, thanks. Pushed a fix.
Is this one ready? |
I think the spack solution is better, but there is still work to be done in
upstream spack, so I think this simple approach should work for 0.7.0
milestone.
…On Mar 27, 2017 6:40 PM, "Jim Garlick" ***@***.***> wrote:
Is this one ready?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#1016 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAtSUqVom5hIGrxwInSnnxKOBVgJiCf_ks5rqGUhgaJpZM4Mqgxd>
.
|
I'll rebase this shortly once I get access to a computer |
In order to allow travis-dep-builder.sh downloaded and built dependencies to override OS installed tools, move the PATHs for these utils to the front of PATH, instead of the end.
Override travis OS provided autotools with latest GNU packages available. This should give tarballs built with `make dist` in travis to build on latest architectures like ppc64le. Fixes flux-framework#925.
Ok, got around to a rebase on current master. |
Great thanks! Merging. |
Download and install latest autotools in travis builds.
This should resolve #925, though that is a bit difficult to test for certain...
Switching to spack for dependencies can come later, as discussed in #920