[PATCH] configure.ac: turn valgrind automagic into --enable-valgrind option

Matúš Kukan matus.kukan at gmail.com
Mon Nov 17 06:17:03 PST 2014


Hi,

On 2 November 2014 13:48, Andreas Sturmlechner
<andreas.sturmlechner at gmail.com> wrote:
> Hi!
>
> Please consider a patch to configure.ac.
>
> - valgrind headers might be present, but not wished for libreoffice
> - if valgrind support is desired but for some reason not found, configure should not keep quiet and continue

Although I don't see a reason for this, it does not mean there is none.
Thanks.

> Attaching a patch that hopes to achieve this in as few lines as possible.

Hmm, could you please attach it as a real patch - not just diff - or
even better,
push to gerrit and send licence statement too, as all explained in:
https://wiki.documentfoundation.org/Development/gerrit
In gerrit, there is bigger chance someone will notice your nice patch.

Thanks,

Matus


More information about the LibreOffice mailing list