[Bug 225302] Merge Review: automake
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=225302 Karsten Hopp kars...@redhat.com changed: What|Removed |Added Status Whiteboard|NotReady| --- Comment #9 from Karsten Hopp kars...@redhat.com 2009-12-02 07:25:09 EDT --- I couldn't find any MIT licensed files in the package anymore, dropped MIT from the list. I've also fixed the man page timestamps: http://cvs.fedora.redhat.com/viewvc/devel/automake/automake.spec?revision=1.47view=markup -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 225302] Merge Review: automake
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=225302 Till Maas opensou...@till.name changed: What|Removed |Added CC||opensou...@till.name Status Whiteboard||NotReady --- Comment #8 from Till Maas opensou...@till.name 2009-09-16 19:30:22 EDT --- - The URL is outdated, the new one is: http://sourceware.org/automake/ - The installation of the manpages is missing -p to preserve the timestamp - Why are the manpages not submitted to upstream? Please remove the NotReady from the status Whiteboard once it is clear which part of automake is MIT licensed (see comment:7). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 225302] Merge Review: automake
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=225302
Paul Howarth p...@city-fan.org changed:
What|Removed |Added
CC||p...@city-fan.org
--- Comment #7 from Paul Howarth p...@city-fan.org 2009-09-09 09:01:28 EDT
---
Here's a IMHO neater method of fixing the perl requires/provides that doesn't
involve turning off the automatic dependency generator or requiring out-of-spec
scripts:
Index: automake.spec
===
RCS file: /cvs/pkgs/rpms/automake/devel/automake.spec,v
retrieving revision 1.44
diff -r1.44 automake.spec
10,11d9
Source1:filter-provides-automake.sh
Source2:filter-requires-automake.sh
22,25d19
%define _use_internal_dependency_generator 0
%define __find_provides %{SOURCE1}
%define __find_requires %{SOURCE2}
37a32,37
# Kludge to remove bogus Automake perl dependencies and provides
%global reqfilt /bin/sh -c %{__perl_requires} | grep -Fv 'perl(Automake::'
%define __perl_requires %{reqfilt}
%global provfilt /bin/sh -c %{__perl_provides} | grep -Fv 'perl(Automake::'
%define __perl_provides %{provfilt}
It also has the beneficial side effect of fixing this rpmlint warning:
automake.src: W: strange-permission filter-requires-automake.sh 0775
automake.src: W: strange-permission filter-provides-automake.sh 0775
As for this:
automake.noarch: W: devel-file-in-non-devel-package
/usr/share/automake-1.11/ansi2knr.c
Since automake is itself a developer tool, this can safely be ignored.
Which parts of automake are MIT licensed?
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 225302] Merge Review: automake
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=225302 Jussi Lehtola jussi.leht...@iki.fi changed: What|Removed |Added Status|ASSIGNED|NEW -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 225302] Merge Review: automake
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=225302 --- Comment #6 from Karsten Hopp kars...@redhat.com 2009-01-21 08:38:45 EDT --- NEWS file fixed in automake-1.10.2-2. Can't fix the ansi2knr stuff as this is used by some m4 macros and will be compiled when needed. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 225302] Merge Review: automake
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=225302 --- Comment #4 from Robert Scheck redhat-bugzi...@linuxnetz.de 2009-01-13 17:15:55 EDT --- automake.noarch: W: file-not-utf8 /usr/share/doc/automake-1.10.1/NEWS Easily to solve. automake.noarch: W: devel-file-in-non-devel-package /usr/share/automake-1.10/ansi2knr.c Is that correct? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 225302] Merge Review: automake
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=225302 Ralf Corsepius rc040...@freenet.de changed: What|Removed |Added Flag|needinfo?(rc040...@freenet. | |de) | --- Comment #5 from Ralf Corsepius rc040...@freenet.de 2009-01-13 17:22:25 EDT --- (In reply to comment #4) automake.noarch: W: devel-file-in-non-devel-package /usr/share/automake-1.10/ansi2knr.c Is that correct? Yes. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 225302] Merge Review: automake
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: automake https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225302 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|NEEDINFO Flag||needinfo?([EMAIL PROTECTED] ||e) --- Additional Comments From [EMAIL PROTECTED] 2007-03-13 11:34 EST --- same here, please update the status -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 225302] Merge Review: automake
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: automake https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225302 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |ASSIGNED --- Additional Comments From [EMAIL PROTECTED] 2007-02-20 07:43 EST --- fixed in automake-1.10-4 -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 225302] Merge Review: automake
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: automake
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225302
[EMAIL PROTECTED] changed:
What|Removed |Added
CC||[EMAIL PROTECTED]
--- Additional Comments From [EMAIL PROTECTED] 2007-02-15 22:30 EST ---
Most of the comments having been applied to the autoconf packages also apply
here:
- Use make DESTDIR=... install instead of %makeinstall
- BuildRoot
- BuildArchitectures
- Use Requires(Pre,Preun) instead of Prereq
...
Additionally:
- CONSIDER: Add all %{_*dirs} you are using in %files to ./configure
(cf. https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225296#c3)
- CONSIDER: I'd recommend to filter the 'perl(Automake::*)' provides/requires.
IMO, they are bogus, because these modules are not in perl's module search path,
but are automake internal perl-modules.
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
