Comment 10 for bug 1195802

Revision history for this message
Anton Gladky (gladky-anton) wrote : Re: [Bug 1195802] Re: 56cdcb breaks LINSOLV compilation on lucid

2013/7/16 Bruno Chareyre <email address hidden>:
> Anton, I don't understand your last changes in cmake.
> This line will trigger the compilation of all combined *.cpp from the three folders in a single yade lib, right?
> ADD_LIBRARY(yade SHARED ${SRC_CORE} ${SRC_PKG} ${SRC_LIB})

Hmm, not quite. It just adds a new library and indicates, which
sources are used.

> Besides I don't see this line called unless chunkSize=0, I'm in doubt to tell if it is really good to do this.

I fixed that hopefully in aa512cc7. I forgot about chunksize>0

> It seems to be more or less equivalent to setting a very big chunksize (except that chunksize would not merge many forlders). The drawback of this is known: high memory requirements.
>
> I would suggest to fix the clump cross-linking issue in source instead of changing the build system.

I think, Christian has fixed that already.

Anton