Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

--- Comment #3 from D Haley <>  2009-02-10 04:06:40 EDT ---

rpmlint output:
$ rpmlint ../SRPMS/gri-2.12.18-2.fc10.src.rpm
../RPMS/i386/gri-2.12.18-2.fc10.i386.rpm  gri.spec 
2 packages and 1 specfiles checked; 0 errors, 0 warnings.

Koji Build:

* Tue Feb 10 2009 D Haley  <> 2.12.18-2
- Cleared many old changelog entries by D, Kelly & T Powers (year<=2005)
- Drop readline buildreq.
- Fix lic. GPLv2 to GPLv2+
- Create emacs subpackages
- Fix release line to include dist macro

>License is GPLv2+, not GPLv2.

>Remove Requires: readline, this is automatically picked up.

>Remove pushd and popd from setup, you don't need them.

>Change make command to
make %{?_smp_mflags}
as this works fine. The paths are already set by %configure.

>Replace "rm -Rf" with "rm -rf" to be consistent.

>Absolute paths in %files need to be replaced with %{_bindir}, %{_datadir} and
so on.
Whoops. Fixed.

>Changelog is way too long, IMHO you may remove everything that is before
I removed all but the last entry before myself, just to make it clear that this
is based upon someone else's work.

>You'll have to branch the emacs script in its own subpackage, as gri probably
works without it.
Created subpackages emacs-%{name} and emacs-%{name}-el. I am not an emacs user,
(brought up on vi(m)) so this is based upon reading refs [1-3]. So there may be
a mistake in my understanding here.


Configure bugmail:
------- You are receiving this mail because: -------
You are on the CC list for the bug.

Fedora-package-review mailing list

Reply via email to