Skip to content

Conversation

@Nerei
Copy link

@Nerei Nerei commented Feb 22, 2015

This is proper replacement of #1897 and contains bugfix with RPATH for pycaffe in install-tree.

The problem was:

  1. at build time cmake created symlink
    <caffe_source>/python/caffe/_caffe.so -> <cmake_build_dir>/lib/_caffe.so
  2. at install time cmake installed _caffe.so and updated RPATH properly
  3. next at install time cmake installed content of <caffe_source>/python/caffe directory, and overwrote installed earlier _caffe.so with the symlink ;-)))

@Nerei Nerei changed the title Bugfix: install rpath for pycaffe (replacement of #1897) Cmake Bugfix: install rpath for pycaffe (replacement of #1897) Feb 22, 2015
@Nerei Nerei force-pushed the bugfix/install_rpath_for_pycaffe branch from 659b9fe to 3f1e937 Compare February 22, 2015 16:48
Anatoly Baksheev added 2 commits February 22, 2015 19:56
… was overwritten with symlink created at build time and installed with install(DIRECTORY ...)
@Nerei Nerei force-pushed the bugfix/install_rpath_for_pycaffe branch from 3f1e937 to 645aa03 Compare February 22, 2015 16:56
@BlGene
Copy link
Contributor

BlGene commented Feb 22, 2015

@Nerei Hey, thanks for figuring this out. This works for me.

@shelhamer I'm for merging this.

shelhamer added a commit that referenced this pull request Feb 22, 2015
@shelhamer shelhamer merged commit cb1f4d6 into BVLC:master Feb 22, 2015
@shelhamer
Copy link
Member

Thanks all.

@Nerei Nerei deleted the bugfix/install_rpath_for_pycaffe branch February 22, 2015 19:36
philkr added a commit to philkr/caffe that referenced this pull request Feb 25, 2015
longjon added a commit that referenced this pull request Mar 4, 2015
cmake and python3 bugfixes for #1939 and #1923
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.

3 participants