Re: [simspark-devel] Windows build of Simspark

[Marian B. <mar...@gm...>]

Hi.

On Mon, Feb 16, 2009 at 1:56 PM, Hedayat Vatankhah <hed...@ai...>wrote:

>  Hi Marian,
>
> Thank you for your efforts :)
>
> *"Marian Buchta" <mar...@gm...> <mar...@gm...>*wrote on ۰۹/۰۲/۱۵ 11:42:48:
>
>  Hi Hedayat and all developers.
>
> I successfully compiled Spark package in Windows Platform (Vista),  but I
> have a couple questions about it:
>
> 1. Following projects need OpenGL extension header files[1]:
>
> glext.h – kerosin and imageperceptor
>
> glxext.h, wglext.h – kerosin
>
> I think it is a good idea to add this header files into Spark in
> spark/windows/include/GL (e.g. because of #include <GL/wglext.h>)
>
> Sorry, I don't understand why you want to add these header files to the
> project while they are available in Windows?! What's wrong with the current
> situation that you've just included these files when compiling in Windows?
>

Files, that I mention above (glext.h, glxext.h, wglext.h) are not available
in Windows (XP or Vista) by default. Also they are not available after
installed Visual Studio or Microsoft Vistual C++ 2008 Redistributable
Package. VS 2008 include only files gl.h and glu.h. For that reason this
files must be downloaded from this[1] site. I think, we have two
possibilities:
- add this files into project spark
- user will download this files from[1] site, copy them to the e.g.
C:\library\OpenGL\GL, add path to them and compile project.
What do you think?

>
>
>
> 2. I must add many of "include_directories()" into different projects in
> Spark. My question is: Variable ${Boost_INCLUDE_DIRS} is required in many
> cmakelists.txt but not in all. It is better to add this variable to the each
> project, that are need? Or add this variable in global cmakelists.txt in
> spark directory? Or following variable is the same as previous:
> add_definitions(-D_CRT_SECURE_NO_WARNINGS). Not all project need this.
>
> I think such flags should be added globally.
>

Ok , thanks.

>
>
>
>
> 3. So, my major problem. Projects create executable and library files. All
> libraries are set to dynamic library (SHARED) except libobject and sexp.
> CMake generate Visual studio (I use 2008) projects. But some projects need
> STATIC version of other libraries. E.g. Sparkagent is dependent from oxygen,
> salt and zeitgeist libraries. So, this three libraries must be set to the
> STATIC (.lib) library. If not, Visual studio write out thousands of linking
> error. Maybe I found a solution[3] (please read all discusion), but it need
> many changes in (or redesign) CMmakeLists.txt files. If you have any
> proposal how to fix it, your wellcome :) .
>
> Sorry :( Why some libraries should be static?! Can't we link sparkagent
> with oxygen, salt and zeitgeist dynamic libraries just as what we do in
> Linux? Isn't it possible to add those libraries to
> target_link_libraries(sparkagent)?! IMHO its not reasonable to link
> sparkagent with static oxygen, salt and zeitgeist libraries. It will waste
> space.
>

You have right. I tried figure out how to resolve this problem. Thank you
for idea.

>
>
>
>
> At last some notes:
>
> - In some projects I must add add_dependencies(), because I have a example,
> where one project compiled earlier then other at which first project need
> library of second project, so I have link error of missing library..
>
> I don't remember right now, but maybe its enough to add those libraries to
> target_link_libraries command?!
>
>
>   - I replaced old FindBoost.cmake with new one, because of error with
> finding library path (in ${Boost_LIBRARIES} variable). Old script found both
> libraries (boost thread and boost regex) but in variable ${Boost_LIBRARIES}
> was only second found path library. This new script is from new cmake 2.7
> from this[2] site (file cmake-2.7.20090210-win32-x86.zip).
>
>
>
> Some changes (only for Windows compatibility) I uploaded to the SVN.
> Please, see changelog.
>
>
>  Thanks again,
>
> Hedayat
>
>
>
> Best Regards
>
> Marian
>
>
>
> [1] http://www.opengl.org/registry/
>
> [2] http://www.cmake.org/files/vCVS/
>
> [3] http://www.cmake.org/pipermail/cmake/2009-February/026964.html
>
>
>
> Thank you,
Marian