Bug 225889

Package htdig-3.2.0-0.9.b6.fc13 should be fine.

OK
==
- almost silent rpmlint (only some spelling false alarms)
- valid project URL and source URL
- upstream tarball is the same as the one from look-aside cache
- md5 hash matches the sources
- dist tag OK
- BuildRoot tag OK
- license tag OK (though valid only for the contrib directory, all other is LGPLv2+ licensed)
- all patches from CVS are applied
- configure options OK
- make options OK
- %clean OK
- %files OK
- %doc OK


may be better
=============
- 9 lines of the description are the same for htdig and htdig-web subpackage; it would be good to define it as a macro, so that it can be maintained at one place.
- explicit BuildRequires for flex should be either commented, or better made non-explicit
- redundant BuildRequires for autoconf and automake (already required by libtool)
- some patches lack link to upstream thread
- hard-link creation deserves a comment (perhaps should be a symlink instead?)
- %defattr(-,root,root) should be %defattr(-,root,root,-)
- ldconfig is not called at all - I suggest to add following:
%post -p /sbin/ldconfig
%postun -p /sbin/ldconfig

- the option -r of rm may be omitted in the following commands
rm -rf $RPM_BUILD_ROOT%{_libdir}/htdig/*.a
rm -rf $RPM_BUILD_ROOT%{_libdir}/htdig/*.la
rm -rf $RPM_BUILD_ROOT%{_libdir}/htdig_db/*.a
rm -rf $RPM_BUILD_ROOT%{_libdir}/htdig_db/*.la

Tomas, could you please have a look at the above minor packaging issues?

They still seem to be valid in Fedora Rawhide...

(In reply to Kamil Dudka from comment #2)
> may be better
> =============
> - 9 lines of the description are the same for htdig and htdig-web
> subpackage; it would be good to define it as a macro, so that it can be
> maintained at one place.

macro added

> - explicit BuildRequires for flex should be either commented, or better made
> non-explicit

I removed the explicit version. I don't know about the reasons for the version, however the version in Fedora is newer so it should not be needed.

> - redundant BuildRequires for autoconf and automake (already required by
> libtool)

removed

> - some patches lack link to upstream thread

Upstream is dead for 8 years. Nothing will happen and most probably this is the last time someone is touching this package before it will be dropped from the distro in the future.

Honestly I don't know the reasons behind those patches and it seems like a waste of time to try to investigate it. The last post on upstream devel list is from March 2005.

> - hard-link creation deserves a comment (perhaps should be a symlink
> instead?)

I changed it to symbolic.

> - %defattr(-,root,root) should be %defattr(-,root,root,-)

fixed

> - ldconfig is not called at all - I suggest to add following:
> %post -p /sbin/ldconfig
> %postun -p /sbin/ldconfig

Added

> - the option -r of rm may be omitted in the following commands
> rm -rf $RPM_BUILD_ROOT%{_libdir}/htdig/*.a
> rm -rf $RPM_BUILD_ROOT%{_libdir}/htdig/*.la
> rm -rf $RPM_BUILD_ROOT%{_libdir}/htdig_db/*.a
> rm -rf $RPM_BUILD_ROOT%{_libdir}/htdig_db/*.la

modified


Fixed in htdig-3.2.0-0.20.b6.fc22

Looks good!