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

Populate fcl.pc with the correct installation paths. #118

Merged
merged 1 commit into from
Apr 10, 2016

Conversation

mband
Copy link
Contributor

@mband mband commented Apr 5, 2016

On my Ubuntu 14.04 with CMake 2.8.12 I found the following issue with fcl.pc:

# This file was generated by CMake for fcl
prefix=/some/path/projects/fcl_implementation/install_target
exec_prefix=${prefix}
libdir=/some/path/projects/fcl_implementation/install_target/lib
includedir=/some/path/projects/fcl_implementation/install_target/include

But the library got placed in /some/path/projects/fcl_implementation/install_target/lib/x86_64-linux-gnuand NOT in /some/path/projects/fcl_implementation/install_target/lib as specified.

This PR fixes this issue in a similar way as to how it is done in https://github.com/danfis/libccd/blob/master/ccd.pc.in

@mband
Copy link
Contributor Author

mband commented Apr 5, 2016

Is there a proper way to trigger a rerun of the Travis CI build? As it failed due to not being able to download the Ubuntu packages...

@jslee02
Copy link
Member

jslee02 commented Apr 5, 2016

Without additional commit, it can be triggered by someone has authority. I reran the failed Travis CI builds.

@scpeters
Copy link
Contributor

scpeters commented Apr 5, 2016

Two builds had spurious failures, and I just re-triggered them.

@scpeters
Copy link
Contributor

scpeters commented Apr 5, 2016

travis is fixed. +1

@sherm1
Copy link
Member

sherm1 commented Apr 5, 2016

👍 - merge?

@sherm1 sherm1 merged commit 893630a into flexible-collision-library:master Apr 10, 2016
@jslee02 jslee02 added this to the FCL 0.5.0 milestone Apr 10, 2016
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.

4 participants