Bug 225295

Issues:

* Builroot is not right

* In general textutils shouldn't be in Requires. Which program is 
  it for?

* info files should be compressed automatically

* remove summary end dot

* install-info scriptlets are missing

* gawk and perl seems to be BuildRequires. perl may be omitted.

suggestions:

* the indenting of the beginning of the spec looks bad because there 
  are spaces and tabs.

* I think that BuildArch is prefered over BuildArchitectures

* use %defattr(-,root,root,-) instead of %defattr(-,root,root) for 
  consistency

* having a / at the end of a directory in %files shows visually that
  it is indeed a directory:
%{_datadir}/autoconf-%{version}/

* maybe the testsuite could be run?

Remark:

* make install DESTDIR=.. is preferred over %makeinstall, but here
  it seems that there is no support for DESTDIR.

in autoconf213-2.13-13:
- buildroot fixed
- removed textutils, rebuilt in mock, no errors, no lost files. looks like it
  doesn't need this requirement anymore
- dot removed from summary
- it requires gawk, but not perl (real requirements, not build requirements)
- use install-info
- use BuildArch
- replace tabs with spaces
- fix defattr
- use 'make install DESTDIR=...'

info files are compressed automatically by rpmbuild, it's not required to put
that into the spec file.

* remove the .gz in scriptlets

* remove 
gzip -9nf ${RPM_BUILD_ROOT}%{_infodir}/*

* gawk and perl are also needed during the build, since their values
  are substituted using sed, in some scripts. Therefore they should
  be buildrequires. Same for m4 if I'm not wrong.

done in -14

This is approved. I assigned it to you.

I still have 2 remarks. The rpmlint warning is ignorable
W: autoconf213 devel-file-in-non-devel-package /usr/share/autoconf-2.13/acconfig.h

The install-info scriptlet adds an entry for Autoconf in the Misc
section in dir. It points to the latest autoconf manual and not 
to the autoconf213 manual. I'll attach a patch.

Created attachment 148084 [details]
use autoconf213 info file and add a comment saying it's legacy

Hopefully a very last comment, you didn't used %dist, and I think 
it is a good choice. This package shouldn't ever need a rebuild
(except to check that it actually builds).

-15 has:
- a disttag
- autoconf213 info file. I've done it a little bit different, so that the File:
tag inside info is correct.

I haven't tested, but your changes look good and saner.

I have checked that source is upstream source, but it doesn't
seems to be the case. The checksum, size, and timestamp are
different.

I reassign to me since it seems to be what should be done in the
new version of the review process.

I don't know what the previous maintainer did, but I've downloaded the original
tarball and run a diff against our tarball. There are no changes at all.
I've built a new package with upstream sources to clean this up.

Please consider keeping the source timestamp too in the
future. 

Anyway it is re-approved.

Closing as suggested by Warren in his mail to fedora-maintainers 'Review with
Flags (Version 6)'

(In reply to comment #3)
> * gawk and perl are also needed during the build, [...]
>   Therefore they should be buildrequires.

A nit (not worth fixing):

Both are redundant, as both are in the minimum build environment.

Perl is explicitly listed among the exceptions in Packaging/Guidelines.
gawk is dragged there indirectly, but it is hard to imagine minimal instal
without awk.

(In reply to comment #15)

> Both are redundant, as both are in the minimum build environment.
> 
> Perl is explicitly listed among the exceptions in Packaging/Guidelines.
> gawk is dragged there indirectly, but it is hard to imagine minimal instal
> without awk.

Personally, I think that they shouldn't be in the minimal 
install and hopefully won't be in the future. Moreover they are
really direct build dependencies and it was only a suggestion.

(In reply to comment #16)
> (In reply to comment #15)

> Personally, I think that they shouldn't be in the minimal 
> install and hopefully won't be in the future.
awk is part of the POSIX standard run-time environment.

Unless POSIX changes, you will always find awk installed on any POSIX compliant
system.

> Moreover they are
> really direct build dependencies and it was only a suggestion.
Fedora not having them always installed is a bug in Fedora.

The minimal install and minimal build environment needs not 
be POSIX compliant. There could be a POSIX meta package (I
guess the LSB package already bring in everything needed
for POSIX). I am not saying that it should not be POSIX compliant
but it should only if it makes sense, we shouldn't tie ourselves
to standards -- while still giving the possibility to conform.

More precisely, I personally think that using a comps that has not 
been tuned should give a minimal install that is POSIX compliant, 
but in my opinion we should give room for minimal sets that are 
more minimal than POSIX, and in those minimal install dependencies
should still be correct.

(In reply to comment #18)
> The minimal install and minimal build environment needs not 
> be POSIX compliant. 
Once again: POSIX is an official standard (IEEE Std 1003.1), not an arbitrary
vendor package set.