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 696920 - Failure to configure gdm using jhbuild for gnome 3.8
Failure to configure gdm using jhbuild for gnome 3.8
Status: RESOLVED FIXED
Product: gdm
Classification: Core
Component: general
3.8.x
Other Linux
: Normal normal
: ---
Assigned To: GDM maintainers
GDM maintainers
Depends on:
Blocks:
 
 
Reported: 2013-03-30 19:16 UTC by Matt Smith
Modified: 2013-08-27 15:00 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
This patch modifies the configure.ac file. (437 bytes, patch)
2013-03-31 12:48 UTC, Matt Smith
none Details | Review
modifies the Makefile.am to change the aclocal path. (314 bytes, patch)
2013-03-31 12:50 UTC, Matt Smith
none Details | Review
Patch to fix build (265 bytes, patch)
2013-08-26 10:12 UTC, Seán de Búrca
committed Details | Review

Description Matt Smith 2013-03-30 19:16:37 UTC
When I try to build gdm the configure script fails. Here is the complete  output of the autogen.sh call.

http://pastie.org/7185681#136-137

To reproduce I use:

jhbuild buildone gdm

The checkout succeeds, but the config fails. It fails with the error:

./configure: line 16338: syntax error near unexpected token `0.40.0'
./configure: line 16338: `IT_PROG_INTLTOOL(0.40.0)'

I believe other packages that I have build use the above macro, with a slightly different syntax, and they compile. I am more than willing to provide any necessary information. I am just not sure where to start with this.
Comment 1 Matt Smith 2013-03-31 12:48:44 UTC
Created attachment 240199 [details] [review]
This patch modifies the configure.ac file.

Removes a line as per suggestion in libtoolize.
Comment 2 Matt Smith 2013-03-31 12:50:47 UTC
Created attachment 240200 [details] [review]
modifies the Makefile.am to change the aclocal path.

As per the 'consider' offered by libtoolize, adds the flags so that this will get built. The flag included is to add the  directory ../m4
Comment 3 Matt Smith 2013-03-31 12:52:36 UTC
Okay I solved this two ways. 

A) I added the following to my jhbuildrc:

    addpath('ACLOCAL_PATH', '/usr/share/aclocal')

B) I used the above patches. The  reason I think that a patch should be made is because the number of other modules using this system is very significant and they all build. So gdm turns out to be something of an exception.
Comment 4 Seán de Búrca 2013-08-26 10:12:03 UTC
Created attachment 253101 [details] [review]
Patch to fix build

This problem still exists in master. The patch I am attaching is sufficient to solve this problem.
Comment 5 David King 2013-08-27 14:44:07 UTC
Review of attachment 253101 [details] [review]:

The change looks fine to me (as a gnome-common maintainer), but please add a descriptive commit message and link to this bug in the commit message before pushing.