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 added the bug label Aug 20, 2015
@jsquyres jsquyres added this to the v1.10.0 milestone Aug 20, 2015
@jsquyres jsquyres changed the title ofi mtl: also link in mtl_ofi_LIBS in the static case v1.10: ofi mtl: also link in mtl_ofi_LIBS in the static case Aug 20, 2015
@jsquyres
Copy link
Member Author

@rhc54 or @yburette please review

@rhc54
Copy link

rhc54 commented Aug 20, 2015

Looks right to me - thx!

👍

@yburette
Copy link
Member

I'm still getting errors when configuring with
--enable-static --disable-shared --with-libfabric=yes

Are you guys able to build it this way?
What am I missing?

@jsquyres
Copy link
Member Author

What errors are you getting?

@rhc54 rhc54 removed the RM-Approved label Aug 20, 2015
@jsquyres
Copy link
Member Author

FWIW, I was actually configuring with --enable-static --disable-shared --with-libfabric=/path/to/libfabric/install

@rhc54 rhc54 removed the reviewed label Aug 20, 2015
@yburette
Copy link
Member

Tried again and I get these errors:

mtl_ofi_component.c:(.text+0x306): undefined reference to `fi_dupinfo'
mtl_ofi_component.c:(.text+0x360): undefined reference to `fi_getinfo'
mtl_ofi_component.c:(.text+0x371): undefined reference to `fi_strerror'
mtl_ofi_component.c:(.text+0x3a5): undefined reference to `fi_freeinfo'
mtl_ofi_component.c:(.text+0x3b2): undefined reference to `fi_freeinfo'
mtl_ofi_component.c:(.text+0x56a): undefined reference to `fi_fabric'
mtl_ofi_component.c:(.text+0x59b): undefined reference to `fi_strerror'
mtl_ofi_component.c:(.text+0x66d): undefined reference to `fi_strerror'
mtl_ofi_component.c:(.text+0x7ef): undefined reference to `fi_strerror'
mtl_ofi_component.c:(.text+0x822): undefined reference to `fi_strerror'
mtl_ofi_component.c:(.text+0x883): undefined reference to `fi_strerror'
../../../ompi/.libs/libmpi.a(mtl_ofi_component.o):mtl_ofi_component.c:(.text+0x8d4): more undefined references to `fi_strerror' follow
../../../ompi/.libs/libmpi.a(mtl_ofi_component.o): In function `ompi_mtl_ofi_component_init':
mtl_ofi_component.c:(.text+0x9a0): undefined reference to `fi_freeinfo'
mtl_ofi_component.c:(.text+0x9aa): undefined reference to `fi_freeinfo'
mtl_ofi_component.c:(.text+0x9e7): undefined reference to `fi_strerror'
../../../ompi/.libs/libmpi.a(mtl_ofi.o): In function `ompi_mtl_ofi_del_procs':
mtl_ofi.c:(.text+0x1a2a): undefined reference to `fi_strerror'
collect2: error: ld returned 1 exit status
make[2]: *** [ompi_info] Error 1

@yburette
Copy link
Member

Hold on, I think I know what's going on... User error. Let me try again.

@jsquyres jsquyres force-pushed the pr/mtl-ofi-static-link-fix branch from 455860d to 7cb0d31 Compare August 20, 2015 20:19
@yburette
Copy link
Member

OK, sorry for my clumsiness. All is good 👍

@jsquyres
Copy link
Member Author

Whew. I was getting worried there... 😄

rhc54 pushed a commit that referenced this pull request Aug 20, 2015
v1.10: ofi mtl: also link in mtl_ofi_LIBS in the static case
@rhc54 rhc54 merged commit c5a8adb into open-mpi:v1.10 Aug 20, 2015
@jsquyres jsquyres deleted the pr/mtl-ofi-static-link-fix 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