On 07.09.2011 22:32, Norbert Thiebaud wrote:
> On Wed, Sep 7, 2011 at 2:45 PM, Michael Stahl <m...@openoffice.org> wrote:
>> On 07.09.2011 21:15, Peter Foley wrote:
>>>
>>> This is my first attempt at converting a module to gbuild so I'd
>>> appreciate it if someone knowlegeable about gbuild could take a look.
>>>
>>> Thanks,
>>>
>>> Peter
>>
>> hi Peter,
>>
>> looks good, didn't know we have such a simple module :)
>
> I wonder would that work dep wise to just have a packaging step ?
>>
>> only thing missing is that you should add the module to the top-level
>> Module_ooo.mk.
>
> and also tell the slideshow module to depend on it (right now it is
> done via build.lst... but Mesa can be integrated to tail_build, as it
> fulfill the requirement to do so)
> which might be a problem since I don't know that we have an api to
> declare pure include-packaging mode dep... the closest thing we have
> is gb_LinkTarget_add_api, but it is an overkill here as it try to add
> stuff in $INCLUDE...
good catch, that indeed seems to be missing.
guess gb_LinkTarget__add_internal_headers can't be used if the headers
come from a different module, because it would break module-only build?
how about something like this:
define gb_LinkTarget_add_external_headers
$(call gb_LinkTarget_get_external_headers_target,$(1)) :| \
$(call gb_Package_get_target,$(2))
endef
_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice
