After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 591840 - configure fails with autoconf 2.64
configure fails with autoconf 2.64
Status: RESOLVED FIXED
Product: glib
Classification: Platform
Component: general
2.21.x
Other Linux
: Normal normal
: ---
Assigned To: gtkdev
gtkdev
Depends on:
Blocks:
 
 
Reported: 2009-08-14 19:04 UTC by Frederic Peters
Modified: 2010-08-02 08:49 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Update GLIB_CHECK_COMPILE_WARNINGS macro to work with Autoconf 2.64 (1.12 KB, patch)
2009-08-14 19:04 UTC, Frederic Peters
committed Details | Review

Description Frederic Peters 2009-08-14 19:04:02 UTC
After upgrading to autoconf 2.64, glib configure fails with:

  checking for GNUC visibility attribute... ./configure: line 31209: syntax error: unexpected end of file

I reduced the problem as best as I could and on autoconf-bugs Ralf Wildenhues identified the issue.
  http://lists.gnu.org/archive/html/bug-autoconf/2009-08/msg00054.html

I'll quote his conclusion:

> Autoconf 2.64 changed AS_IF a bit to be more compact, and to allow
> things like
>   AS_IF(...) && ...
>
> as documented in NEWS:
>
>   ** The m4sh macros AS_IF and AS_CASE can now be used in shell lists.
>      The responsibility for supplying a trailing newline now belongs to
>      the call site, but since most users did not add dnl, this generally
>      results in fewer empty lines in configure.
>
> As a consequence, you need to remove the last dnl in the m4_ifvaln line.
> With 2.63, it will lead to a harmless extra newline in the configure
> script.
Comment 1 Frederic Peters 2009-08-14 19:04:57 UTC
Created attachment 140796 [details] [review]
Update GLIB_CHECK_COMPILE_WARNINGS macro to work with Autoconf 2.64

Quoting its NEWS file: "The m4sh macros AS_IF and AS_CASE can now be
used in shell lists.  The responsibility for supplying a trailing
newline now belongs to the call site, but since most users did not add
dnl, this generally results in fewer empty lines in configure."
Comment 2 Frederic Peters 2009-08-14 19:05:53 UTC
(patch tested with both 2.63 and 2.64)
Comment 3 Matthias Clasen 2009-08-14 19:14:06 UTC
Thanks, please commit.
Comment 4 Frederic Peters 2009-08-14 19:38:44 UTC
commit c24f6e55276fac68b08df34b0d350f65afe994d9
Author: Frédéric Péters <fpeters@0d.be>
Date:   Fri Aug 14 21:00:41 2009 +0200

    Update GLIB_CHECK_COMPILE_WARNINGS macro to work with Autoconf 2.64
    
    Quoting its NEWS file: "The m4sh macros AS_IF and AS_CASE can now be
    used in shell lists.  The responsibility for supplying a trailing
    newline now belongs to the call site, but since most users did not add
    dnl, this generally results in fewer empty lines in configure."
    
    http://bugzilla.gnome.org/show_bug.cgi?id=591840
Comment 5 Andy Wingo 2010-07-21 11:12:26 UTC
Is this one OK to apply to glib-2-20 as well? Work still uses the older glib.
Comment 6 Andy Wingo 2010-08-02 08:49:04 UTC
Cherry-picked fpeters' c24f6e552 to glib-2-20.