[PATCH][REVIEW] gbuild conversion: idlc module

Matúš Kukan matus.kukan at gmail.com
Wed Apr 11 23:50:17 PDT 2012

On 12 April 2012 06:33, David Tardon <dtardon at redhat.com> wrote:
> 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.

salhelper is often needed only for Windows.

> ifeq ($(HAVE_GETOPT),YES)
> # nothing needed

Maybe also
$(call gb_LinkTarget_add_defs,$(1),\
so it could be used here:

> $(eval $(call gb_Helper_register_libraries,PLAINLIBS_NONE,\
>    gnu_getopt \
> ))

It is already registered in solenv/gbuild/platform/
This whole getopt thing is windows only (I wonder if it is still
relevant ? maybe yes)
So we need to write ifeq ($(OS),WNT) either to RepositoryExternal or
to makefiles.


More information about the LibreOffice mailing list