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 700557 - autoreconf -i fails in cases of ltmain.sh mismatch
autoreconf -i fails in cases of ltmain.sh mismatch
Status: RESOLVED FIXED
Product: jhbuild
Classification: Infrastructure
Component: general
3.6.x
Other AIX
: Normal normal
: ---
Assigned To: Jhbuild maintainers
Jhbuild QA
Depends on:
Blocks:
 
 
Reported: 2013-05-17 21:13 UTC by John Ralls
Modified: 2013-11-18 11:44 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Use autoreconf -fi to force replacement of ltmain.sh (1.08 KB, patch)
2013-06-07 13:34 UTC, John Ralls
needs-work Details | Review
Use autoreconf -fi to force replacement of ltmain.sh (14.84 KB, patch)
2013-06-11 13:28 UTC, John Ralls
committed Details | Review

Description John Ralls 2013-05-17 21:13:03 UTC
autotools.py's autoreconf -i (line 244 in master as of this writing) call fails if there's already an ltmain.sh in srcdir and the installed libtool is of a different version -- not an uncommon occurence when dealing with tarballs. Changing the call to autoreconf -fi overcomes the problem and should be otherwise harmless.
Comment 1 John Ralls 2013-06-07 13:34:08 UTC
Created attachment 246250 [details] [review]
Use autoreconf -fi to force replacement of ltmain.sh

Maybe a patch will get someone's attention, and in the meantime I can set up gtk-osx-build-setup.sh to apply it.
Comment 2 Javier Jardón (IRC: jjardon) 2013-06-11 11:22:47 UTC
Review of attachment 246250 [details] [review]:

Maybe you need to update the docs in doc/C/index.docbook
Comment 3 John Ralls 2013-06-11 13:28:57 UTC
Created attachment 246517 [details] [review]
Use autoreconf -fi to force replacement of ltmain.sh

OK, done. I went ahead and hand-edited the relevant po files as well, since that bit is in English in the msgstr for all of them (though a couple don't have that string translated).
Comment 4 Emanuele Aina 2013-11-17 10:24:00 UTC
I've been running jhbuild with this patch applied since a while and I've not found any issue. Javier, is ok to push this patch to master?
Comment 5 Javier Jardón (IRC: jjardon) 2013-11-18 11:38:23 UTC
Comment on attachment 246517 [details] [review]
Use autoreconf -fi to force replacement of ltmain.sh

Sure, go for it, sorry for the delay on the reply
Comment 6 Emanuele Aina 2013-11-18 11:41:16 UTC
The following fix has been pushed:
b822e83 Bug 700557 - autoreconf -i fails in cases of ltmain.sh mismatch
Comment 7 Emanuele Aina 2013-11-18 11:44:11 UTC
(In reply to comment #5)
> Sure, go for it, sorry for the delay on the reply

No worries. :D
Pushed, thank you for the review and to John for the patch!