Hi,

On Mon, 2008-12-22 at 17:44 +0100, Carsten Neumann wrote:
> Hi Gerrit,
> 
> > Datum: Mon, 22 Dec 2008 08:40:12 +0800
> > Von: Gerrit Voss <vo...@camtech.ntu.edu.sg>
> 
> > honestly I don't like that to much.
> 
> well, honestly: me neither ;-)

;-)

> > Taking it out of the source dir and
> > putting it into the module dir is rather confusing. I would prefer
> > something like
> > 
> > Source/Base/CMakeLists.txt:
> > 
> > INCLUDE(CMakeLists.libBase.cmake)
> > 
> > and from OpenSG/CMakeLists.txt you can do
> > 
> > ADD_SUBDIRECTORY(Source/Base) 
> > 
> > or you glob for them. Unfortunately AFAIK you can't change the cmake
> > file name ADD_SUBDIRECTORY is looking for.
> 
> I've done this now, attached patch is against r1644, 
> it is at about the same level of conversion as the last patch, i.e. it needs 
> a bit more work before it is ready.

looks a lot better to me too ;-). One minor wish though, could
we hide the different passes inside a function or macro.

Otherwise I'm fine with it.

thanks & kind regards,
  gerrit



------------------------------------------------------------------------------
_______________________________________________
Opensg-users mailing list
Opensg-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/opensg-users

Reply via email to