code-google-com / bullet

Automatically exported from code.google.com/p/bullet
0 stars 0 forks source link

bullet 2.75 cmake don´t compile the .pc file on linux #337

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Hello i have one problem on my fedora bullet package. Cmake don´t compile
the .pc file. Can you please fix that? or is this fixed in trunk?

Original issue reported on code.google.com by linuxdon...@gmail.com on 21 Jan 2010 at 11:50

GoogleCodeExporter commented 9 years ago

But there has been some contribution here:
http://code.google.com/p/bullet/issues/detail?id=319&can=1&q=.pc

Does that help?
Thanks,
Erwin

Original comment by erwin.coumans on 22 Jan 2010 at 12:12

GoogleCodeExporter commented 9 years ago
that not the problem. It are the paths that are needed to be modified by 
compiling.

This part must be modified automatically by compiling:

prefix=@prefix@
exec_prefix=@exec_prefix@
libdir=@libdir@
includedir=@includedir@

Original comment by linuxdon...@gmail.com on 22 Jan 2010 at 12:21

GoogleCodeExporter commented 9 years ago

Can you please provide a patch against latest trunk, I'm happy to apply it.

Original comment by erwin.coumans on 22 Jan 2010 at 12:29

GoogleCodeExporter commented 9 years ago
The bullet.pc.in was created for Bullet's autotools system. There is no cmake 
support 
for that (yet).

Original comment by erwin.coumans on 22 Jan 2010 at 12:37

GoogleCodeExporter commented 9 years ago
Sorry i can´t give you an patch i can´t code :(

Is the cmake support for that planned?

Original comment by linuxdon...@gmail.com on 22 Jan 2010 at 1:09

GoogleCodeExporter commented 9 years ago
We have no plans in for this, but we are happy to accept a patch in this area.

Let's close the issue for now (re-open if there is a patch)
Thanks,
Erwin

Original comment by erwin.coumans on 22 Jan 2010 at 5:45

GoogleCodeExporter commented 9 years ago
Here is an Patch from  Rex Dieter

Original comment by linuxdon...@gmail.com on 9 Mar 2010 at 11:45

Attachments:

GoogleCodeExporter commented 9 years ago

Original comment by linuxdon...@gmail.com on 9 Mar 2010 at 11:48

Attachments:

GoogleCodeExporter commented 9 years ago

I had a look but the libraries that CMake generates have capitals, so does this 
actually work?

Name: bullet
Description: Bullet Continuous Collision Detection and Physics Library
Requires:
Version: @BULLET_VERSION@
Libs: -L@LIB_DESTINATION@ -lbulletdynamics -lbulletcollision -lbulletmath
Cflags: -I@INCLUDE_INSTALL_DIR@/bullet

Original comment by erwin.coumans on 11 Mar 2010 at 6:05

GoogleCodeExporter commented 9 years ago
erwin.coumans is that wrong? -I@INCLUDE_INSTALL_DIR@/bullet
Because this dir don´t get createt on install there are only folder in
/usr/include/Bullet* they are 3 folder or so.

Original comment by linuxdon...@gmail.com on 12 Mar 2010 at 12:20

GoogleCodeExporter commented 9 years ago
No idea. Bullet wasn't designed to be 'installed' system wide.

Things are so much simpler if you use it as a static library, include the 'src' 
folder 
and link against the libraries. CMake is supposed to make that process very 
simple.

If someone want to hassle with 'install' or .pc files or autotools, I'm happy 
to accept 
contributions.

Original comment by erwin.coumans on 12 Mar 2010 at 3:42

GoogleCodeExporter commented 9 years ago
Static library are not allowed. I will do some tests with programms that need 
bullet
and compile them on my system and then i will see if the app find it or not.

Original comment by linuxdon...@gmail.com on 12 Mar 2010 at 4:06

GoogleCodeExporter commented 9 years ago
Just out of curiosity, why are static libraries not allowed?

CMake has an option to create dynamic libraries. Why is it necessary to 
'install' them system wide?
Thanks,
Erwin

Original comment by erwin.coumans on 12 Mar 2010 at 10:54

GoogleCodeExporter commented 9 years ago
Fwiw, I just duped bullet.pc.cmake from bullet.pc.in contents as a basis for 
this
patch.  So, it would appear that bullet.pc.in is wrong in that regard too.

Original comment by rdieter@gmail.com on 12 Mar 2010 at 3:35

GoogleCodeExporter commented 9 years ago
alright, bullet.pc.in seems to have been fixed in issue #319 , so 
bullet.pc.cmake
needs to be adjusted accordingly as well.

Original comment by rdieter@gmail.com on 12 Mar 2010 at 3:37

GoogleCodeExporter commented 9 years ago
@rdieter that change will be fixed when i update the bullet for fedora to 
version 2.76.

Original comment by linuxdon...@gmail.com on 12 Mar 2010 at 5:22

GoogleCodeExporter commented 9 years ago
Can you provide a new patch, based on latest svn trunk?

Thanks a lot,
Erwin

Original comment by erwin.coumans on 12 Mar 2010 at 6:54

GoogleCodeExporter commented 9 years ago
Here is the patch for the lastest trunk.
But i found another problem.
Cmake want to install bullet-include files in /usr/include and not in
/usr/include/bullet .
But with GNU tools it installs correctly into /usr/include/bullet
Have you an idea?
I have fixed that in the patch to but it seams that cmake don´t install into 
it :(
But i found wat is wrong i will make an second patch for this. But for the first
please add this patch to trunk :)

But the too patch´s you can add into trunk that patches don´t have any errors.

Original comment by linuxdon...@gmail.com on 14 Mar 2010 at 12:02

Attachments:

GoogleCodeExporter commented 9 years ago

I justed applied the patch in latest trunk.
http://code.google.com/p/bullet/source/detail?r=2079

Can you test it, and see if pkg-config works now with cmake?

Original comment by erwin.coumans on 31 Mar 2010 at 9:01

GoogleCodeExporter commented 9 years ago
I must add some more patches that it works correctly. But thank you for adding 
this
first patch :) 

Original comment by linuxdon...@gmail.com on 1 Apr 2010 at 12:53