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

v4.0.x: btl/usnic: fix usnic_btl_run_tests CPPFLAGS #6450

Merged
merged 2 commits into from
Mar 6, 2019

Conversation

ggouaillardet
Copy link
Contributor

do define the OMPI_LIBMPI_NAME macro via the CPPFLAGS.
The issue occurs when Open MPI is configured with
--enable-opal-btl-usnic-unit-tests

Thanks George Marselis for reporting this issue

Refs. #6441

Signed-off-by: Gilles Gouaillardet gilles@rist.or.jp

(cherry picked from commit b409762)

do define the OMPI_LIBMPI_NAME macro via the CPPFLAGS.
The issue occurs when Open MPI is configured with
--enable-opal-btl-usnic-unit-tests

Thanks George Marselis for reporting this issue

Refs. open-mpi#6441

Signed-off-by: Gilles Gouaillardet <gilles@rist.or.jp>

(cherry picked from commit open-mpi/ompi@b409762)
@ggouaillardet ggouaillardet added this to the v4.0.1 milestone Mar 5, 2019
@ggouaillardet ggouaillardet requested a review from jsquyres March 5, 2019 02:03
@jsquyres jsquyres changed the title btl/usnic: fix usnic_btl_run_tests CPPFLAGS v4.0.x: btl/usnic: fix usnic_btl_run_tests CPPFLAGS Mar 5, 2019
Use $(AM_CPPFLAGS) in $(usnic_btl_run_tests_CPPFLAGS) so that we don't
have to replicate hard-coded values.

Signed-off-by: Jeff Squyres <jsquyres@cisco.com>
(cherry picked from commit 1456377)
@jsquyres
Copy link
Member

jsquyres commented Mar 5, 2019

@ggouaillardet I pushed a 2nd commit here (do not merge here before merging master PR #6459)

@hppritcha hppritcha added the NEWS label Mar 6, 2019
@hppritcha hppritcha merged commit 8b2d804 into open-mpi:v4.0.x Mar 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants