<html> <head> <meta content="text/html; charset=windows-1252" http-equiv="Content-Type"> </head> <body bgcolor="#FFFFFF" text="#000000"> Hi David, hi Matus, thank you for your comments on my patch. I changed it accordingly. *@Matúš:* thank you for your help on IRC. Special usage gnu_getopt lib in windows: Matúš proposed the follow simple solution to include in Executable_idlcpp.mk: ifeq ($(OS),WNT) $(eval $(call gb_Executable_use_libraries,idlcpp,\ gnu_getopt \ )) endif because 1. HAVE_GETOPT is allways NO on windows 2. gnu_getopt is registered already on windows: gb_Library_PLAINLIBS_NONE += [...] gnu_getopt [...] here solenv/gbuild/platform/WNT_INTEL_GCC.mk and here solenv/gbuild/platform/WNT_INTEL_MSC.mk 3. original make file snippet was only active on windows .IF "$(HAVE_GETOPT)" != "YES" .IF "$(GUI)$(COM)" == "WNTMSC" APP1STDLIBS=gnu_getopt.lib .ELIF "$(GUI)$(COM)" == "WNTGCC" APP1STDLIBS=-lgnu_getopt .ENDIF .ENDIF Having said that, how could we simplify the gnu_getopt treatment in tools/Executable_rscdep.mk? David -------- Original Message -------- <table class="moz-email-headers-table" border="0" cellpadding="0" cellspacing="0"> <tbody> <tr> <th align="RIGHT" nowrap="nowrap" valign="BASELINE">Subject: </th> <td>Re: [PATCH][REVIEW] gbuild conversion: idlc module</td> </tr> <tr> <th align="RIGHT" nowrap="nowrap" valign="BASELINE">Date: </th> <td>Thu, 12 Apr 2012 06:33:19 +0200</td> </tr> <tr> <th align="RIGHT" nowrap="nowrap" valign="BASELINE">From: </th> <td>David Tardon <a class="moz-txt-link-rfc2396E" href="mailto:dtardon@redhat.com"><dtardon@redhat.com></a></td> </tr> <tr> <th align="RIGHT" nowrap="nowrap" valign="BASELINE">To: </th> <td>David Ostrovsky <a class="moz-txt-link-rfc2396E" href="mailto:david.ostrovsky@gmx.de"><david.ostrovsky@gmx.de></a></td> </tr> <tr> <th align="RIGHT" nowrap="nowrap" valign="BASELINE">CC: </th> <td>Libreoffice-dev <a class="moz-txt-link-rfc2396E" href="mailto:libreoffice@lists.freedesktop.org"><libreoffice@lists.freedesktop.org></a></td> </tr> </tbody> </table> <pre>On Wed, Apr 11, 2012 at 11:18:46PM +0200, David Ostrovsky wrote: > Hi, > > this is gbuild conversion for idlc module. Good. But it is not quite ready yet, see comments below. > diff --git a/Repository.mk b/Repository.mk > index 1a3851e..638f4f8 100644 > --- a/Repository.mk > +++ b/Repository.mk > @@ -31,6 +31,8 @@ $(eval $(call gb_Helper_register_executables,NONE, \ > bmp \ > bmpsum \ > cppunit/cppunittester \ > + idlc \ > + idlcpp \ > g2g \ > gencoll_rule \ > genconv_dict \ Both binaries should be registered in layer SDKBIN (because of APP1RPATH=SDK). > + > +$(eval $(call gb_Executable_use_libraries,idlc,\ > + sal \ > + reg \ Originally idlc is linked with salhelper too. It is possible it is no longer necessary, but it is also possible that it is necessary on a different platform. (Nitpick: please, keep the libs sorted alphabetically :-) > +)) > +$(eval $(call gb_Executable_use_linked_libs,idlcpp,\ > + gnu_getopt \ > +)) This is wrong: there is no unconditionally built library gnu_getopt. It only worked for you because there is no function gb_Executable_use_linked_libs either (it is called gb_Executable_use_libraries) and your platform's libc has getopt .-) You need to use gb_Executable_use_externals and add a new definition to RepositoryExternals.mk for it, like: ifeq ($(HAVE_GETOPT),YES) # nothing needed define gb_LinkTarget__use_gnu_getopt endef else # !HAVE_GETOPT define gb_LinkTarget__use_gnu_getopt $(call gb_LinkTarget_use_libraries,$(1),\ gnu_getopt \ ) endef $(eval $(call gb_Helper_register_libraries,PLAINLIBS_NONE,\ gnu_getopt \ )) endif # HAVE_GETOPT D. </pre> </body> </html>