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 5, 2015

Same as open-mpi/ompi#774, but without the treematch fix (because treematch isn't yet on v2.x).

@rolfv since you clearly looked at open-mpi/ompi#774, please review 😄

@jsquyres jsquyres added the bug label Aug 5, 2015
@jsquyres jsquyres added this to the v2.0.0 milestone Aug 5, 2015
@rolfv
Copy link

rolfv commented Aug 5, 2015

Your indentation of Cisco copyright seems odd, but otherwise looks good to me.

@rolfv
Copy link

rolfv commented Aug 5, 2015

bot:label:reviewed

@rolfv
Copy link

rolfv commented Aug 5, 2015

👍

@ompiteam-bot
Copy link

OMPIBot error: Label "reviewed" is already set on issue 467.

hppritcha added a commit that referenced this pull request Aug 5, 2015
@hppritcha hppritcha merged commit ecd93d0 into open-mpi:v2.x Aug 5, 2015
alex-mikheev pushed a commit to alex-mikheev/ompi-release that referenced this pull request Nov 11, 2015
…ic_coverity_fix

fcoll/static: minor fix for coverity
@jsquyres jsquyres deleted the pr/v2.x-FUNCTION-fixes branch February 3, 2016 22:12
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.

4 participants