Hi Robert and Mourad,

Attached the fixed ModulesInstall.cmake module to solve the problem
that spotted Mourad. I prefer change to use  TARGET_SRC and TARGET_H
variables because the other macros (SETUP_PLUGIN and SETUP_EXE) were
using them.

This change needs the previous submission and is compatible with the
android patches I sent yesterday. If you prefer I send all in the same
submission tell me.

Rafa.

2011/3/7 Rafa Gaitan <[email protected]>:
> Hi Mourad,
>
> Thanks for testing!, I'm trying a fresh build on my macosx to test the
> issue.  I thought this was working! :S.
>
> Rafa.
>
>
> 2011/3/7 Mourad Boufarguine <[email protected]>:
>> Hi Rafa & Robert,
>>
>> I gave this files a try with VS and i got some problems with the INSTALL
>> project : header files don't get copied to the CMAKE_INSTALL_PREFIX (I think
>> one should have this problem on unices too).
>> This comes from ModuleInstall.cmake that uses LIB_PUBLIC_HEADERS variable
>> which was renamed by Rafa to TARGET_H for all osg libraries.
>>
>> A quick workaround would be to add SET(LIB_PUBLIC_HEADERS ${TARGET_H})
>> before including ModuleInstall in SETUP_LIBRARY macro, or to use the same
>> variable name.
>>
>> Mourad
>>
>> On Tue, Mar 1, 2011 at 4:10 PM, Rafa Gaitan <[email protected]> wrote:
>>>
>>> Hi Robert,
>>>
>>> Ignore previous submission, I realized that osgQT it was not building
>>> correctly. Attached submission now builds fine on my side.
>>>
>>> Rafa.
>>>
>>> 2011/3/1 Rafa Gaitan <[email protected]>:
>>> > Hi Robert,
>>> >
>>> > I have added the macro SETUP_LIBRARY, similar to SETUP_PLUGIN or
>>> > SETUP_EXE, in order to have a unique entry point to build the
>>> > libraries. With this changes the android integration will be less
>>> > painful (currently is a big IF(ANDROID) for each CMakeLists.txt) and
>>> > more maintainable in the future. I hope next submissions will be for
>>> > supporting android from my colleague Jorge.
>>> >
>>> > It has been tested only in linux, I don't have currently my mac with me.
>>> >
>>> > Greets,
>>> > Rafa.
>>> >
>>> > --
>>> > Rafael Gaitán Linares
>>> > Instituto de Automática e Informática Industrial  http://www.ai2.upv.es
>>> > http://gvsig3d.blogspot.com
>>> > Ciudad Politécnica de la Innovación
>>> > Universidad Politécnica de Valencia
>>> >
>>>
>>>
>>>
>>> --
>>> Rafael Gaitán Linares
>>> Instituto de Automática e Informática Industrial  http://www.ai2.upv.es
>>> http://gvsig3d.blogspot.com
>>> Ciudad Politécnica de la Innovación
>>> Universidad Politécnica de Valencia
>>>
>>> _______________________________________________
>>> osg-submissions mailing list
>>> [email protected]
>>>
>>> http://lists.openscenegraph.org/listinfo.cgi/osg-submissions-openscenegraph.org
>>>
>>
>>
>
>
>
> --
> Rafael Gaitán Linares
> Instituto de Automática e Informática Industrial  http://www.ai2.upv.es
> http://gvsig3d.blogspot.com
> Ciudad Politécnica de la Innovación
> Universidad Politécnica de Valencia
>



-- 
Rafael Gaitán Linares
Instituto de Automática e Informática Industrial  http://www.ai2.upv.es
http://gvsig3d.blogspot.com
Ciudad Politécnica de la Innovación
Universidad Politécnica de Valencia

Attachment: ModuleInstall.cmake
Description: Binary data

_______________________________________________
osg-submissions mailing list
[email protected]
http://lists.openscenegraph.org/listinfo.cgi/osg-submissions-openscenegraph.org

Reply via email to