Skip to content
This repository was archived by the owner on Sep 30, 2022. It is now read-only.

Conversation

@jsquyres
Copy link
Member

@jsquyres jsquyres commented Aug 6, 2015

This PR applies fixes for open-mpi/ompi#751 to the v2.x branch.

@ggouaillardet @eschnett

@jsquyres jsquyres added the bug label Aug 6, 2015
@jsquyres jsquyres added this to the v2.0.0 milestone Aug 6, 2015
jsquyres and others added 2 commits August 7, 2015 03:26
Per open-mpi/ompi#751, ensure to patch up all Autotools output (not
just in some cases).

Also, adjust the patching process to only write our verbose statements
and a new configure script if the content actually changed as result
of the patching.

(cherry picked from commit open-mpi/ompi@982455a)
Patch generated configure files to fix the libtool.m4 bug the bug
that started with Libtool 2.4.3.

(cherry picked from commit open-mpi/ompi@67fdfdd)
@jsquyres jsquyres force-pushed the pr/v2.x-autogen-updates branch from 8e01fa0 to 4740cb0 Compare August 7, 2015 10:27
@jsquyres
Copy link
Member Author

jsquyres commented Aug 7, 2015

PR updated to include open-mpi/ompi@67fdfdd.

@jsquyres
Copy link
Member Author

@ggouaillardet Can you review this please? Thanks!

@jsquyres
Copy link
Member Author

@eschnett Also, can you try this?

@eschnett
Copy link

@jsquyres I now switched to using snapshots of the 2.x version; I tried v2.x-dev-153-gf6a57cc last. Is there a snapshot version I can try?

@ggouaillardet
Copy link
Contributor

:bot:assign: @ggouaillardet

@ggouaillardet
Copy link
Contributor

👍

jsquyres added a commit that referenced this pull request Aug 11, 2015
autogen.pl: ensure to patch *all* Autotools output
@jsquyres jsquyres merged commit 5a80ea1 into open-mpi:v2.x Aug 11, 2015
@jsquyres jsquyres deleted the pr/v2.x-autogen-updates branch August 11, 2015 12:13
@jsquyres
Copy link
Member Author

@eschnett Snapshots are made nightly. I just merged this into the v2.x branch, so it should be in tonight's snapshot.

That being said, I've had good success with hub, a CLI thingy for pulling down individual pull requests (e.g., exactly for testing a pull request before it is merged).

alex-mikheev pushed a commit to alex-mikheev/ompi-release that referenced this pull request Nov 11, 2015
…_warn

btl/openib: fix compiler warning, by HalR
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants