Hi Bernhard,

Am Dienstag, den 09.06.2015, 13:52 +0200 schrieb Bernhard Übelacker:
> can confirm that both variants are fixing the issue.

Thank you!

> I would prefer the function level fix, but as there is already a change 
> to the build system that does exclusively apply to this file, I think 
> both are equally good.

With upstream patch application in mind, I'd rather modify Makefile.am.
In the per-function variant the attribute intruction would have to be
#ifdef'd out of the way for non-GCC compilers, whereas automake is
already rather GCC-centric anyway.

> (And when OCaml 4.02 enters Stretch we can try removing it again.)

Yes, but I'd like to fix this in Jessie, and it is a lot more
convincing for the Stable Release Managers if the patch has gone the
route through unstable->testing before.

Cheers,

Fabian

Attachment: signature.asc
Description: This is a digitally signed message part

Reply via email to