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 622149 - --disable-regex breaks glib2 build
--disable-regex breaks glib2 build
Status: RESOLVED FIXED
Product: glib
Classification: Platform
Component: gregex
2.24.x
Other Linux
: Normal normal
: ---
Assigned To: gtkdev
gtkdev
Depends on:
Blocks:
 
 
Reported: 2010-06-20 05:10 UTC by Andrew Ziem
Modified: 2012-02-27 03:22 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
build logs with and without --disable-regex (40.43 KB, application/x-bzip2)
2010-06-20 05:10 UTC, Andrew Ziem
  Details
regex: Remove --disable-regex option (8.57 KB, patch)
2012-02-11 22:52 UTC, Christian Persch
accepted-commit_now Details | Review
regex: Remove --disable-regex option (8.56 KB, patch)
2012-02-27 03:22 UTC, Matthias Clasen
committed Details | Review

Description Andrew Ziem 2010-06-20 05:10:49 UTC
Created attachment 164104 [details]
build logs with and without  --disable-regex

I'm cross-compiling glib2 2.24.1 to Windows on openSUSE Build Service.  With the  --disable-regex configure option, the build breaks

Creating library file: .libs/libglib-2.0.dll.aCannot export g_match_info_expand_references: symbol not defined
Cannot export g_match_info_fetch: symbol not defined
[..snip..]
Cannot export g_regex_check_replacement: symbol not defined
Cannot export g_regex_error_quark: symbol not defined
[..snip..]
Comment 1 Matthias Clasen 2010-06-21 13:28:24 UTC
Yeah, I'm inclined to remove that configure option. 
Why do you try to build glib without regex support, anyway ?
Comment 2 Andrew Ziem 2010-06-21 14:09:42 UTC
On Windows I ship the giant GTK+/PyGTK stack with my tiny cross-platform application, and to shrink each dependency I've been trying various options including trying ./configure options, CFLAG=-Os, etc.  

So maybe the whole system would work with --disable-regex, but maybe it wouldn't.
Comment 3 Christian Persch 2012-02-11 22:52:30 UTC
Created attachment 207368 [details] [review]
regex: Remove --disable-regex option
Comment 4 Colin Walters 2012-02-13 17:04:53 UTC
One question to ask before we delete code - who introduced it, and and why?

git annotate says:

commit 0196d639753247cb0d8aca289155154ef6daa561
Author: Marco Barisione <marco@barisione.org>
Date:   Thu Mar 15 13:01:31 2007 +0000

And the originating bug is:
https://bugzilla.gnome.org/show_bug.cgi?id=50075

Scanning through the discussion there appears to have been no rationale for adding a configure option.  So let's just remove it.
Comment 5 Colin Walters 2012-02-13 17:06:14 UTC
Review of attachment 207368 [details] [review]:

Looks right.
Comment 6 Matthias Clasen 2012-02-27 03:22:52 UTC
The following fix has been pushed:
d706345 regex: Remove --disable-regex option
Comment 7 Matthias Clasen 2012-02-27 03:22:56 UTC
Created attachment 208467 [details] [review]
regex: Remove --disable-regex option