search

andrewhwood / alembic

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

FIND_LIBRARY in AlembicPRMan.cmake don't work #229

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?
1.edit init_Alembic.cmd want to make PRMan build:

edit line : set BASE_ARGS=--with-prman=%RMANTREE% --disable-maya 
--generator=%Generator%

2. modification : SET( PRMAN_LINK_FLAGS "/nologo /dll 
/LIBPATH:\"%RMANTREE%\lib\" libprman.lib" )
               to---> SET( PRMAN_LINK_FLAGS "/nologo /dll /LIBPATH:\"%RMANTREE%\\lib\" libprman.lib" )

3.run init_Alembic.cmd

What is the expected output? What do you see instead?

-- About to include AlembicPRMan.cmake
-- PRMan not found

What version of the product are you using? On what operating system?
43(1.0)
win7
vc9
Please provide any additional information below.

fix:
modification : AlembicPRMan.cmake

FIND_LIBRARY( ALEMBIC_PRMAN_LIBPRMAN prman
              PATHS
              "${ALEMBIC_PRMAN_ROOT}/lib/"
              NO_DEFAULT_PATH
              NO_CMAKE_ENVIRONMENT_PATH
              NO_CMAKE_PATH
              NO_SYSTEM_ENVIRONMENT_PATH
              NO_CMAKE_SYSTEM_PATH
              DOC "The prman library" )
to-------->

FIND_LIBRARY( ALEMBIC_PRMAN_LIBPRMAN libprman.lib
              PATHS
              "${ALEMBIC_PRMAN_ROOT}/lib/"
              NO_DEFAULT_PATH
              NO_CMAKE_ENVIRONMENT_PATH
              NO_CMAKE_PATH
              NO_SYSTEM_ENVIRONMENT_PATH
              NO_CMAKE_SYSTEM_PATH
              DOC "The prman library" )

Original issue reported on code.google.com by vdcoo...@gmail.com on 6 Sep 2011 at 8:34

GoogleCodeExporter commented 9 years ago 
Are you sure it can't find the library with:
FIND_LIBRARY( ALEMBIC_PRMAN_LIBPRMAN prman

Your suggestion:
FIND_LIBRARY( ALEMBIC_PRMAN_LIBPRMAN libprman.lib

won't work with non Windows platforms.

Original comment by miller.lucas on 15 Oct 2011 at 12:20

GoogleCodeExporter commented 9 years ago 
Issue 228 has been merged into this issue.

Original comment by miller.lucas on 15 Oct 2011 at 12:22

GoogleCodeExporter commented 9 years ago

Original comment by miller.lucas on 7 Nov 2011 at 8:18

GoogleCodeExporter commented 9 years ago 
Hopefully this is better with some of the CMake changes in 1.5.0.

Original comment by miller.lucas on 13 Sep 2013 at 6:14