Skip to content

Comments

Fix caffe rpath#5865

Merged
Noiredd merged 1 commit intoBVLC:masterfrom
cheshirekow:fix/caffe_rpath
Jan 31, 2018
Merged

Fix caffe rpath#5865
Noiredd merged 1 commit intoBVLC:masterfrom
cheshirekow:fix/caffe_rpath

Conversation

@cheshirekow
Copy link

The logic for setting the library RPATH checks whether or not
${CMAKE_INSTALL_PREFIX}/lib is a system directory, and if not
adds it to the library RPATH. However, caffe does not install
to ${CMAKE_INSTALL_PREFIX}/lib, it installs to
${CMAKE_INSTALL_PREFIX}/${CMAKE_INSTALL_LIBDIR} (from
GNUInstallDirs). CMAKE_INSTALL_LIBDIR may be something like
"lib/x86_64-linux-gnu"

The logic for setting the library RPATH checks whether or not
${CMAKE_INSTALL_PREFIX}/lib is a system directory, and if not
adds it to the library RPATH. However, caffe does not install
to ${CMAKE_INSTALL_PREFIX}/lib, it installs to
${CMAKE_INSTALL_PREFIX}/${CMAKE_INSTALL_LIBDIR} (from
GNUInstallDirs). CMAKE_INSTALL_LIBDIR may be something like
"lib/x86_64-linux-gnu"
@Noiredd
Copy link
Member

Noiredd commented Nov 2, 2017

I have restarted the tests that failed prior to #5973, it passes now.

@Noiredd Noiredd merged commit d1208db into BVLC:master Jan 31, 2018
beniz pushed a commit to jolibrain/caffe that referenced this pull request Feb 3, 2018
XinYao1994 pushed a commit to XinYao1994/caffe that referenced this pull request Aug 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants