Re: R17.0-rc1 patch for gzio.c compile breakage.

Daniel Goertzen <[email protected]> Tue, 4 Feb 2014 09:16:54 -0600
Newsgroups gmane.comp.lang.erlang.patches
Message-ID <CAJCf5RztpES0BcCKH+u2qzJPRVM5YSQAA0iCd7hsWasd5gE=Ng@mail.gmail.com>
Thank you for the merge.  And you piqued my curiosity as to the why.

My gentoo zconf.h defines the macro as _Z_OF instead of OF.  Digging
deeper, they made the change on the premise that OF is a stupid, polluting
name for an internal macro.  There was disagreement as to the value of the
change (https://bugs.gentoo.org/show_bug.cgi?id=383179)


Dan.



On Tue, Feb 4, 2014 at 8:27 AM, Björn Gustavsson <[email protected]> wrote:

> On Mon, Feb 3, 2014 at 6:35 PM, Daniel Goertzen
> <[email protected]> wrote:
> > A dependency on zlib.h was recently removed (commit 8a147a7365).  This
> also
> > removes the definition of the macro "OF" which is required by gzio.c.
>  Now
> > "OF" is evidently defined by other headers on some systems, but on at
> least
> > Gentoo Linux this is not the case.
>
> The OF() macro is defined in zconf.h, which is included by zlib.h, so
> I don't understand why the build should break on Gentoo Linux.
>
> But since the use of OF() is completely unnecessary and is an
> unnecessary dependency I have written a commit message for your
> patch and included it in our daily builds.
>
> git fetch [email protected]:bjorng/otp.git bjorn/erts/zlib-1.2.8/OTP-11669
>
> /Bjorn
>
> --
> Björn Gustavsson, Erlang/OTP, Ericsson AB
>

_______________________________________________
erlang-patches mailing list
[email protected]
http://erlang.org/mailman/listinfo/erlang-patches