Bug 226050

libtool-1.5.22-10 prepared for review

Any reason why you're not using %configure?

%configure used to copy config.sub and config.guess files from /usr/share/libtool/config into the top source directory. That's not a good idea when you try to build a newer libtooll to update those files...  I haven't checked for a while if %configure still does that, though.

That behaviour has been removed at Fedora 10 already. Now I've to copy
these files myself into buildroot to get old software rebuilding. So
%configure should work with Fedora >= 10 for you again. Maybe we just
can test it in a scratch build?

Hmm, according to the build logs %configure is just a macro that exports compiler flags and calls ./configure with a bunch of arguments that set destinations, for example on rawhide i586 the arguments are

./configure --build=i386-redhat-linux-gnu --host=i386-redhat-linux-gnu --target=i586-redhat-linux-gnu --program-prefix= --prefix=/usr --exec-prefix=/usr --bindir=/usr/bin --sbindir=/usr/sbin --sysconfdir=/etc --datadir=/usr/share --includedir=/usr/include --libdir=/usr/lib --libexecdir=/usr/libexec --localstatedir=/var --sharedstatedir=/var/lib --mandir=/usr/share/man --infodir=/usr/share/inf

Check out older redhat-rpm-config packages' macros file, e.g. in F8, to see the old definition where it copied files as Karsten says. You can use "rpm --eval %configure", btw.

(In reply to comment #6)
> Check out older redhat-rpm-config packages' macros file, e.g. in F8, to see the
> old definition where it copied files as Karsten says.

OK, but that's no problem anymore since rawhide is F12 :)

> You can use "rpm --eval %configure", btw. 

Duh, true.

[YES] source files match upstream: a4b36980765003b47dd75ac9429f4f11  libtool-2.2.6b.tar.lzma
[YES] package meets naming and versioning guidelines 
[YES] specfile is properly named
[YES] specfile is cleanly written
[???] uses macros consistently: %{name} should be used instead of 
      $PACKAGE and %{_datadir} instead of $datadir
[YES] dist tag is present
[YES] build root is correct
[YES] license field matches the actual license
[YES] license is open source-compatible
[YES] license text included in package
[YES] latest version is being packaged
[???] BuildRequires are proper: no need to include gcc and gcc-c++, because 
      they are added automatically
[YES] compiler flags are appropriate
[YES] %clean is present
[YES] package builds in mock
[YES] debuginfo package looks complete
[NO] rpmlint is silent

$ rpmlint *.rpm
libtool.i686: E: no-binary
libtool.i686: W: file-not-utf8 /usr/share/doc/libtool-2.2.6/ChangeLog.2002
libtool.i686: E: script-without-shebang /usr/share/libtool/config/ltmain.sh
libtool.i686: W: file-not-utf8 /usr/share/doc/libtool-2.2.6/ChangeLog.1998
libtool.i686: W: file-not-utf8 /usr/share/doc/libtool-2.2.6/ChangeLog.1999
libtool.i686: W: file-not-utf8 /usr/share/doc/libtool-2.2.6/ChangeLog.1997
libtool-debuginfo.i686: W: hidden-file-or-dir /usr/src/debug/libtool-2.2.6b/libltdl/.libs
libtool-debuginfo.i686: W: hidden-file-or-dir /usr/src/debug/libtool-2.2.6b/libltdl/.libs
libtool-ltdl.i686: W: shared-lib-calls-exit /usr/lib/libltdl.so.7.2.1 exit
libtool-ltdl-devel.i686: W: no-documentation
5 packages and 0 specfiles checked; 2 errors, 8 warnings.

[YES] final provides and requires look sane
[OK] %check is present, but "make check" is commented out
[YES] no shared libraries are added to the regular linker search paths in app package
[YES] owns the directories it creates
[YES] doesn't own any directories it shouldn't
[YES] no duplicates in %files
[YES] file permissions are appropriate
[YES] scriptlets ok
[YES] code, not content
[YES] documentation is small, so no -docs subpackage is necessary
[YES] %docs are not necessary for the proper functioning of the package
[YES] no headers
[YES] no pkgconfig files
[YES] no libtool .la droppings
[YES] not a GUI app

ping

>rpmlint i386/*-2.6-19.fc14.i386.rpm.rpm
libtool.i386: W: spelling-error %description -l en_US Autotools -> Auto tools, Auto-tools, Autopilots
libtool.i386: W: spelling-error %description -l en_US Autoconf -> Autocrat, Autocross, Autoclave
libtool.i386: W: spelling-error %description -l en_US Automake -> Automaker, Automate, Auto make
libtool.i386: W: spelling-error %description -l en_US ltdl -> ltd, ltd l, Ltd
libtool.i386: E: no-binary
libtool.i386: E: script-without-shebang /usr/share/libtool/config/ltmain.sh
libtool-debuginfo.i386: E: non-standard-dir-perm /usr/lib/debug 0775L
libtool-debuginfo.i386: W: hidden-file-or-dir /usr/src/debug/libtool-2.2.6b/libltdl/.libs
libtool-debuginfo.i386: W: hidden-file-or-dir /usr/src/debug/libtool-2.2.6b/libltdl/.libs
libtool-debuginfo.i386: E: non-standard-dir-perm /usr/src/debug/libtool-2.2.6b/libltdl/libltdl 0775L
libtool-debuginfo.i386: E: non-standard-dir-perm /usr/src/debug/libtool-2.2.6b/libltdl 0775L
libtool-debuginfo.i386: E: non-standard-dir-perm /usr/src/debug/libtool-2.2.6b/libltdl/loaders 0775L
libtool-ltdl.i386: W: spelling-error Summary(en_US) Runtime -> Run time, Run-time, Untimely
libtool-ltdl.i386: W: spelling-error %description -l en_US runtime -> run time, run-time, untimely
libtool-ltdl.i386: W: spelling-error %description -l en_US Autotools -> Auto tools, Auto-tools, Autopilots
libtool-ltdl.i386: W: spelling-error %description -l en_US Autoconf -> Autocrat, Autocross, Autoclave
libtool-ltdl.i386: W: spelling-error %description -l en_US Automake -> Automaker, Automate, Auto make
libtool-ltdl.i386: W: shared-lib-calls-exit /usr/lib/libltdl.so.7.2.1 exit
4 packages and 0 specfiles checked; 6 errors, 12 warnings.


The spelling errors can be ignored as spell doesn't know about auto{conf,make} tools and marks their names as errors.

libtool is an arch package as it has some paths to arch dependend gcc stuff in it, it cannot be a noarch package even though it has no binaries. 

I'm not sure about the debuginfo errors/warnings. libtool ships no .libs subdir and doesn't create /usr/lib/debug, so this might be a rpmlint bug. My umask is set to 0002, which is quite common and shouldn't lead to rpmlint errors.
Should rpmlint be run on the debuginfo packages, btw ? I couldn't find anything about it on our wiki.

shared-lib-calls-exit: There's exactly one exit call in libltdl and that one is in the alloc_die_default function with the comment '/* Unless overridden, exit on memory failure.  */' This function isn't called by anything in libtool-* and is used as some kind of template which should be overwritten when required. 
I'd say we can ignore this rpmlint warning, too.

So the real fixes in -19 are the conversion of the changelogs to utf8, the move of some documentation to the ltdl-devel subpackage and the selfchecks are now enabled

Thank you.
Overall it looks good now.

The .libs directory is created automatically when creating separate debuginfo, and that's ok.

This review can now be closed.