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

Cmake Bugfix: install rpath for pycaffe (replacement of #1897) #1939

Merged
merged 4 commits into from
Feb 22, 2015

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