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=225675 --- Comment #4 from Jindrich Novy <jnovy@xxxxxxxxxx> 2008-09-11 05:47:26 EDT --- (In reply to comment #3) > Eval of current srpm: > > rpmlint on srpm: > > Unversioned obsoletes, as above, plus: This should be fine since the obsoletes are there because of very ancient BDB versions (mostly all RHL ones) and they will never appear again. > > db4.src: W: mixed-use-of-spaces-and-tabs (spaces: line 140, tab: line 167) > The specfile mixes use of spaces and tabs for indentation, which is a cosmetic > annoyance. Use either spaces or tabs for indentation, not both. This should be fixed now. > > db4.src: W: patch-not-applied Patch22: db-4.5.20-jni-include-dir.patch > A patch is included in your package but was not applied. Refer to the patches > documentation to see what's wrong. D'oh, seems like I forgot to apply the patch actually after rediffing. It is now fixed as well. > rpmlint on rpms: > > No docs for -cxx, -tcl and -static. Still the non-standard exec perm. The problem is that there is no documentation for -static, since it is actually the same as -devel but statically linked. The -cxx, -tcl is shipped without docs since there is shipped only runtime bits and I stuffed all the docs, which are actually of any use by developers into -devel subpackage. Thanks for the review. -- 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@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review