[PUSHED] Re: [PATCH][REVIEW] gbuild conversion: idlc module

Matúš Kukan matus.kukan at gmail.com
Fri Apr 13 04:31:38 PDT 2012


Hi David

On 12 April 2012 23:45, David Ostrovsky <david.ostrovsky at gmx.de> wrote:
>
> thank you for your comments on my patch. I changed it accordingly.

nice patch, thanks

I pushed it with few modifications:
- removed new_style from CustomTarget
- also removed makefile.mk from inc/
- and please, could you be consistent with tabs vs. spaces, I've
cleaned tabs from Executable_idlc.mk

But it was good.

> Having said that, how could we simplify the gnu_getopt treatment in tools/Executable_rscdep.mk?

It was already done, see
http://cgit.freedesktop.org/libreoffice/core/commit/?id=40775ce183ac29e9598cbca59e995c7e77327a1a

Thanks,
Matúš


More information about the LibreOffice mailing list