Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225299
Summary: Merge Review: automake15 Product: Fedora Extras Version: devel Platform: All OS/Version: Linux Status: NEW Severity: normal Priority: normal Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: nobody@fedoraproject.org QAContact: fedora-package-review@redhat.com CC: karsten@redhat.com
Fedora Merge Review: automake15
http://cvs.fedora.redhat.com/viewcvs/devel/automake15/
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Merge Review: automake15
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225299
------- Additional Comments From karsten@redhat.com 2007-02-19 11:46 EST ------- automake15-1.5-18 has the most common review issues fixed, please check...
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Merge Review: automake15
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225299
------- Additional Comments From rc040203@freenet.de 2007-02-19 22:55 EST ------- rpmbuild trashing config.guess and config.sub (c.f. https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225296#c3) also shows effect here:
It causes the config.guess/config.sub being distributed though the automake15 package to be replaced.
Instead of the ancient versions dated 2001-<something> contained in the automake-1.5 tarball, the resulting rpm contains the a bit less ancient versions being distributed by rpm (2003-<something> on FC6).
IMO, automake15 should either ship the original version from the automake-1.5 tarball or an up-to-date version of config.guess/config.sub (from gnu.org).
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Merge Review: automake15
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225299
------- Additional Comments From karsten@redhat.com 2007-02-20 06:10 EST ------- automake15-1.5-19 uses ./configure instead of %configure. Maybe I'll replace config.{guess,sub} with more recent ones when I have some spare time as I need to do a test build of some packages then.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Merge Review: automake15
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225299
karsten@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |NEEDINFO Flag| |needinfo?(rc040203@freenet.d | |e)
------- Additional Comments From karsten@redhat.com 2007-03-13 11:31 EST ------- What's the review status ? Is it approved ?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Merge Review: automake15
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225299
rc040203@freenet.de changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |ASSIGNED Flag|needinfo?(rc040203@freenet.d| |e) |
------- Additional Comments From rc040203@freenet.de 2007-03-13 12:03 EST ------- Why are you asking me?
I never formally reviewed this package, nor do I know how merge reviews are supposed to work.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Merge Review: automake15
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225299
karsten@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |NEEDINFO Flag| |needinfo?(nobody@fedoraproje | |ct.org)
------- Additional Comments From karsten@redhat.com 2007-03-13 12:58 EST ------- probably because your comment looked like some kind of review to me ;-)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Merge Review: automake15
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225299
------- Additional Comments From pertusus@free.fr 2007-03-13 14:39 EST ------- I'll probably review formally when I have time.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Merge Review: automake15
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225299
------- Additional Comments From pertusus@free.fr 2007-03-14 18:24 EST ------- I attach a patch for configure to avoid rerunning autoconf.
I patched as the first patch with %patch10 -b .versioning which allowed to restore timestamps by doing touch -r configure.versioning configure touch -r configure.in.orig configure.in
I think it should be part of automake15-versioning.patch but I let that to you, unless you agree with those changes (including adding the -b .versioning) in that case I could do a patch against the cvs version changing automake15-versioning.patch.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Merge Review: automake15
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225299
------- Additional Comments From pertusus@free.fr 2007-03-14 19:18 EST ------- Created an attachment (id=150101) --> (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=150101&action=vie...) patch configure and Makefile.in to avoid rerunning autotools
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Merge Review: automake15
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225299
------- Additional Comments From pertusus@free.fr 2007-03-14 19:24 EST ------- The patch now applies with %patch10 -p1 -b .versioning
In this patch there are changes that should be in automake-1.5-subdirs-89619.patch (lines like - @set fnord $(MAKEFLAGS); amf=$$2; \ + @set fnord $$MAKEFLAGS; amf=$$2; \ )
And also the patch automake-1.5-info.patch leads to a Makefile.in which isn't what automake would generate based on the patched Makefile.am. Not a big deal, but i'll propose an updated patch once this has been digested.
As a side note automake was run during the make. Is it on purpose? it seems to me that the automake run was the automake from the package itself.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Merge Review: automake15
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225299
pertusus@free.fr changed:
What |Removed |Added ---------------------------------------------------------------------------- BugsThisDependsOn| |224569
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Merge Review: automake15
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225299
Bug 225299 depends on bug 224569, which changed state.
Bug 224569 Summary: erroneous provides https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=224569
What |Old Value |New Value ---------------------------------------------------------------------------- Resolution| |RAWHIDE Status|NEW |CLOSED
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Merge Review: automake15
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225299
bugzilla@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Severity|normal |medium
karsten@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |ASSIGNED Flag|needinfo?(nobody@fedoraproje| |ct.org) |
------- Additional Comments From karsten@redhat.com 2007-04-16 08:33 EST ------- I've added your changes in -22
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Merge Review: automake15
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225299
bugzilla@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Priority|normal |medium
------- Additional Comments From pertusus@free.fr 2007-04-22 15:31 EST ------- Created an attachment (id=153256) --> (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=153256&action=vie...) complete patches of Makefile.in to avoir rerunning autoconf
This patch allows not to run autoconf (and the autoconf call is commented out). As a side note it would be better to cal autoconf-2.13. But yet better not call anything...
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Merge Review: automake15
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225299
pertusus@free.fr changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |NEEDINFO CC| |pertusus@free.fr Flag| |needinfo?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=225299
Jussi Lehtola jussi.lehtola@iki.fi changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |NEW
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=225299
Alexander Kurtakov akurtako@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |CLOSED CC| |akurtako@redhat.com Resolution| |DEFERRED Flag|needinfo? | Last Closed| |2011-07-28 12:21:50
--- Comment #13 from Alexander Kurtakov akurtako@redhat.com 2011-07-28 12:21:50 EDT --- Package has been deprecated. Closing.
package-review@lists.fedoraproject.org