Re: [Libreoffice] [GBUILD PATCH] merging libraries

2012-01-10 Thread Bjoern Michaelsen
Hi Matúš, On Sun, Jan 08, 2012 at 09:27:26PM +0100, Matúš Kukan wrote: > And I'm not sure about gb_LINKED_LIBS in LinkTarget.mk. > The horrible expression could be correct, > but it's probably not nice to have variable used that way there (but it is > now). It looks to me like it does the right

[Libreoffice] [GBUILD PATCH] merging libraries

2012-01-08 Thread Matúš Kukan
Hi, could someone please review attached patch ? Especially I did not know where to put gb_MERGEDLIBS (or you may come up with better name) (until now it's gb_CORE_LIBS in LinkTarget.mk) so I put it in solenv/gbuild/extensions/pre_MergedLibsList.mk. And I'm not sure about gb_LINKED_LIBS in LinkT