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 313753 - Expansion of NEON_SSL breaks configure
Expansion of NEON_SSL breaks configure
Status: RESOLVED FIXED
Product: gnome-vfs
Classification: Deprecated
Component: Build
cvs (head)
Other All
: Normal normal
: ---
Assigned To: gnome-vfs maintainers
gnome-vfs maintainers
Depends on:
Blocks: 305641
 
 
Reported: 2005-08-17 19:34 UTC by Stepan Kasal
Modified: 2005-08-23 15:27 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
a patch (759 bytes, patch)
2005-08-17 19:36 UTC, Stepan Kasal 		committed Details | Review
various fixes in configure.in (7.04 KB, patch)
2005-08-17 19:39 UTC, Stepan Kasal 		committed Details | Review

Description Stepan Kasal 2005-08-17 19:34:17 UTC 
./configure crashes on sytems with latest neon installed.

This is because of this:

The imported version of neon uses NEON_SSL as #define.

But current version has an Autoconf macro with the same name.  When m4 files
with this macro are installed in the aclocal directory, this breaks the
AC_DEFINE call which is used in acinclude.m4.
Comment 1 Stepan Kasal 2005-08-17 19:36:23 UTC 
Created attachment 50868 [details] [review]
a patch

This ugly hack fixes the problem.

(Of course, the problem will disappear as soon as ./imported/neon will be
replaced by current version and acinclude.m4 will hold current neon macros.)
Comment 2 Stepan Kasal 2005-08-17 19:39:08 UTC 
Created attachment 50869 [details] [review]
various fixes in configure.in

When I was sniffing in configure.in, I corrected some m4 quoting, one typo
(--enable-http_neon), and many various cosmetic changes.

OK to commit?
Comment 3 Stepan Kasal 2005-08-17 19:57:49 UTC 
One more comment:
To reproduce the problem as described here, you need aclocal from Automake >= 1.8.

With aclocal < 1.8, acinclude.m4 was placed at the beginning of aclocal.m4, and
thus LIBNEON_SOURCE_CHECKS was redefined by the definition in neon.m4. This
wasn't healthy either: NEON_SSL was never defined in config.h, and the ssl code
in neon was not compiled.
Comment 4 Christian Kellner 2005-08-19 03:18:00 UTC 
Ohh gosh autocrap. Thanks. Looks good to me. Well as good as autocrap foo can
look anyway.
Comment 5 Stepan Kasal 2005-08-23 15:27:52 UTC 
It can, but it takes some work.  If TFM is not enough, feel free to ask us at
gnu.org Autoconf list.  ;-)

Committed.