JeremyARussell / box2d

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

please use ${LIB_INSTALL_DIR} to send library to the correct directory #252

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
hello, i have a 64 bits system and figured that the install routine have a 
hardcoded path on it:

http://code.google.com/p/box2d/source/browse/trunk/Box2D/Box2D/CMakeLists.txt?sp
ec=svn237&r=199#194

The correctly should be lib64, but only when the system is 64 bits, for x86 
this is by far the correct.

The closest solution that i've found is this variable: 
http://librelist.com/browser//yajl/2010/3/25/patch-install-into-usr-lib64-on-x86
-64-arch-where-needed/

sadly i can't confirm that it exists on cmake 2.6 

Original issue reported on code.google.com by Sombr...@gmail.com on 4 Nov 2011 at 3:01

GoogleCodeExporter commented 9 years ago
This issue was closed by revision r239.

Original comment by erinca...@gmail.com on 14 Nov 2011 at 6:03

GoogleCodeExporter commented 9 years ago
Command: Commit  
Modified: D:\Development\Box2D\Box2D\Box2D\CMakeLists.txt  
Sending content: D:\Development\Box2D\Box2D\Box2D\CMakeLists.txt  
Completed: At revision: 239  

Original comment by erinca...@gmail.com on 14 Nov 2011 at 6:03

GoogleCodeExporter commented 9 years ago
it works now. the following line does the correct job now:

 cmake -DCMAKE_INSTALL_PREFIX=/usr -DLIB_INSTALL_DIR=lib64 .

thanks a lot.

Original comment by Sombr...@gmail.com on 21 Nov 2011 at 2:14

GoogleCodeExporter commented 9 years ago
But the cherry in the cake would be some kind of autodetection system, ;)

those guys from KDE seems to have something like this:

http://kwave.svn.sourceforge.net/viewvc/kwave/tags/Release-0_8_6/CMakeLists.txt?
revision=2682&view=markup

line 90 seems to be a useful strategy. 

Original comment by Sombr...@gmail.com on 21 Nov 2011 at 2:26

GoogleCodeExporter commented 9 years ago
I've made small modifications for reference, 

Original comment by Sombr...@gmail.com on 21 Nov 2011 at 2:37

Attachments: