Skip to content

CMake: do not populate rpath with absolute paths#1376

Open
EstebanDugueperoux2 wants to merge 1 commit intoPixarAnimationStudios:releasefrom
EstebanDugueperoux2:feature/LeaveDefaultCMakeRPathConfig
Open

CMake: do not populate rpath with absolute paths#1376
EstebanDugueperoux2 wants to merge 1 commit intoPixarAnimationStudios:releasefrom
EstebanDugueperoux2:feature/LeaveDefaultCMakeRPathConfig

Conversation

@EstebanDugueperoux2
Copy link

Hi,

I propose this PR to avoid patching opensubdiv source when adding release to conan package manager repository.

This commit comes from https://github.com/conan-io/conan-center-index/blob/master/recipes/opensubdiv/all/patches/3.6.0-0002-cmake-no-rpath.patch

Regards.

@EstebanDugueperoux2
Copy link
Author

Hi @davidgyu,

could I have a review of this PR?

Regards.

@davidgyu
Copy link
Member

Filed as internal issue #OSD-476

@davidgyu
Copy link
Member

@EstebanDugueperoux2 Could you file a Contributor License Agreement? I don't think we have one on file. Thanks!

@EstebanDugueperoux2
Copy link
Author

Hello, done.

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