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 690914 - Support Automake 1.13
Support Automake 1.13
Status: RESOLVED FIXED
Product: at-spi
Classification: Platform
Component: at-spi2-atk
1.32.x
Other Linux
: Normal normal
: ---
Assigned To: Li Yuan
At-spi maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2012-12-31 09:25 UTC by Kerrick Staley
Modified: 2013-01-07 16:09 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch to support Automake 1.13 (2.02 KB, patch)
2012-12-31 09:25 UTC, Kerrick Staley
none Details | Review
Alternative (better) patch to support Automake 1.13 (1.51 KB, patch)
2012-12-31 09:42 UTC, Kerrick Staley
accepted-commit_now Details | Review

Description Kerrick Staley 2012-12-31 09:25:18 UTC
Created attachment 232424 [details] [review]
Patch to support Automake 1.13

at-spi2-atk doesn't build with Automake 1.13 (which was just released), because it checks the Automake version number and only accepts values from 1.9 to 1.12. Automake 1.13 does work, however, so the check should be updated to accept Automake 1.13. A simple patch to do this is attached.
Comment 1 Kerrick Staley 2012-12-31 09:39:39 UTC
The patch I previously supplied fixes the problem in the obvious way, based on how the original code worked. However, I don't think the original code was really correct: the ac_progs list is just a list of candidate executables to look at, so it doesn't need to be changed according to the required Automake version - old versions will be rejected anyway. Also, ac_progs should include "$AUTOMAKE"(allowing the Automake binary to be passed via an environment variable) and "automake" (so that the "automake" binary itself is checked). I'll attach an alternative patch to implement this.
Comment 2 Kerrick Staley 2012-12-31 09:40:38 UTC
Err, I meant automake_progs, not ac_progs.
Comment 3 Kerrick Staley 2012-12-31 09:42:03 UTC
Created attachment 232425 [details] [review]
Alternative (better) patch to support Automake 1.13
Comment 4 Kerrick Staley 2012-12-31 13:52:49 UTC
This is the same issue as
Comment 5 Kerrick Staley 2012-12-31 13:53:11 UTC
This is the same issue as

https://bugzilla.gnome.org/show_bug.cgi?id=690920

The code was just copied/pasted, I think.
Comment 6 Mike Gorse 2013-01-02 16:05:10 UTC
Comment on attachment 232425 [details] [review]
Alternative (better) patch to support Automake 1.13

I'm fine with committing this (do you have commit access?), although I might eventually just pull a newer gnome-autogen.sh after 690920 is fixed.
Comment 7 Patrick Welche 2013-01-07 15:53:44 UTC
Any way of avoiding gnome-autogen.sh and just using autoreconf? (Bug 537772 is five years old now - had the trivial patch to gnome-autogen.sh in it been applied, we wouldn't be here again...)
Comment 8 Mike Gorse 2013-01-07 16:09:49 UTC
I've reworked the at-spi2-atk and pyatspi2 autogen.sh scripts to call autoreconf, as is done by at-spi2-core, so I think that this should be fixed now. Please re-open if not. Thanks.