Skip to content

Conversation

@PDeveze
Copy link
Contributor

@PDeveze PDeveze commented Jul 7, 2016

I have here a few patchs for osc/portals4

@PDeveze
Copy link
Contributor Author

PDeveze commented Jul 7, 2016

Sorry, in that pull request are also the patchs for mtl/portals4. But I think this will not be a problem to integrate the patchs.

@jsquyres
Copy link
Member

jsquyres commented Jul 7, 2016

@regrant @tkordenbrock Please review.

@jsquyres
Copy link
Member

jsquyres commented Jul 7, 2016

@PDeveze Are these related to the commits you have on #1850?

If so, and if you didn't already know: you can continue to add / modify commits on a PR at any time before it is merged (e.g., you could have added these new osc/portals4 commits on #1850, if you wanted to).

@PDeveze
Copy link
Contributor Author

PDeveze commented Jul 7, 2016

Thanks for your tip !
The reason why I created a new branch is to separate the patchs concerning mtl/portals4 and osc/portals4. The next days I will also have patchs and PR on coll/portals4 and btl/portals4. And I will take attention not to mix the different patchs on these different branches.

@PDeveze PDeveze force-pushed the Patchs-on-osc-portals4 branch from d2f7475 to e46d120 Compare July 12, 2016 13:11
@PDeveze PDeveze force-pushed the Patchs-on-osc-portals4 branch from e46d120 to fe52184 Compare July 18, 2016 08:03
@PDeveze PDeveze force-pushed the Patchs-on-osc-portals4 branch from fe52184 to 7aaf16e Compare July 18, 2016 08:58
@tkordenbrock tkordenbrock added this to the v2.0.1 milestone Jul 19, 2016
@tkordenbrock tkordenbrock self-assigned this Jul 19, 2016
@tkordenbrock tkordenbrock merged commit 37ad6aa into open-mpi:master Jul 20, 2016
@PDeveze PDeveze deleted the Patchs-on-osc-portals4 branch July 21, 2016 06:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants