Current SVN Cmake HtFluidDemo broke

[IsaacG]

The current CMake file for the HeightFluidDemo from SVN is broke. On line 20 the prefix lib must be removed to allow Cmake to configure correctly. Attached is the modified CmakeLists.txt that works for me.

[PaulMartz]

I can confirm that this is an issue. These libraries used to use the "Lib" prefix, but not in current svn, and removing the "Lib" prefix from the library names in CMakeLists.txt (as IsaacG suggests) resolves the issue.

In general, what is the formal mechanism for submitting fixes for issues like these? (Erwin, John?)
-Paul

[John McCutchan]

Hi,

The preferred method for submitting bug fixes to Bullet is to file an issue in the google issue tracker and attach any patches there.

Thanks,
John

[John McCutchan]

FYI,

I just committed a fix for this problem. I had fixed it last week but I forgot to commit it.

Thanks,
John

[mi076]

Hello, there is a little problem in that patch in CMakeList.txt, line 7
// SET(CMAKE_BUILD_TYPE "Debug")
("//" instead "#"), it breaks cmake. Thanks ...

[John McCutchan]

doh. Fixed in SVN.

[PaulMartz]

Thanks, John, for commiting this, and for the info about posting patches to the Google group. Tested current svn and all builds fine using CMake and VS8/WinXP.