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=226661 --- Comment #2 from Roman Rakus <rrakus@xxxxxxxxxx> 2009-11-30 09:12:22 EDT --- >- BAD: rpmlint flags an error: >yaboot.src: W: summary-ended-with-dot Linux bootloader for Power Macintosh "New >World" computers. fixed >yaboot.src:48: W: unversioned-explicit-obsoletes ybin fixed >yaboot.src:110: E: hardcoded-library-path in $RPM_BUILD_ROOT/usr/lib/yaboot/ >yaboot.src:123: E: hardcoded-library-path in /usr/lib/yaboot/addnote >yaboot.src:124: E: hardcoded-library-path in /usr/lib/yaboot/ofboot >yaboot.src:125: E: hardcoded-library-path in /usr/lib/yaboot/yaboot >yaboot.src:126: E: hardcoded-library-path in /usr/lib/yaboot/yaboot.debug This is special case. Yaboot binaries are needed to be in /usr/lib/yaboot dir. >yaboot.ppc: W: summary-ended-with-dot Linux bootloader for Power Macintosh "New >World" computers. fixed >yaboot.ppc: W: obsolete-not-provided ybin fixed >yaboot.ppc: E: statically-linked-binary /usr/lib/yaboot/yaboot >yaboot.ppc: W: executable-stack /usr/lib/yaboot/yaboot >yaboot.ppc: W: unstripped-binary-or-object /usr/lib/yaboot/yaboot.debug >yaboot.ppc: E: statically-linked-binary /usr/lib/yaboot/yaboot.debug >yaboot.ppc: W: executable-stack /usr/lib/yaboot/yaboot.debug Special case. Yaboot is bootloader, is not linux executable file, is meant to be executed only by OpenFirmware >yaboot.ppc: W: file-not-utf8 /usr/share/doc/yaboot-1.3.14/yaboot-howto.de.sgml fixed >- GOOD: package is named according to guidelines >- GOOD: spec file named properly >- BAD: wrong build root >see Packaging/Guidelines fixed >- GOOD: package is licensed with a Fedora approved license and meets the >Licensing Guidelines >- GOOD: the License field in the package spec file does not match the actual >license >- GOOD: license files are shipped, but not marked as documentation (see below) >- CHECK: spec file must be written in American English () Is or isn't? >- CHECK: The spec file is legible >summary ends with dot - should not be there fixed >- GOOD: source upstream >- GOOD: builds in mock >- CHECK: no build dependencies necessary beyond base packages >is ybin necessary, isn it obsolete fixed >- GOOD: doesn't ship locale files >- GOOD: no libraries shipped >- BAD: package is not relocatable >see rpmlint output see above >- BAD: shipped directories owned by package, direct dependency or >filesystem: >/usr/lib/yaboot/ is not owned fixed >- GOOD: no duplicates in %files >- CHECK: wrong location of files >-rwxr-xr-x 1 root root 6132 Oct 29 12:02 >/usr/lib/yaboot/addnote >-rw-r--r-- 1 root root 11410 Oct 29 12:02 >/usr/lib/yaboot/ofboot >-rw-r--r-- 1 root root 198192 Oct 29 12:02 >/usr/lib/yaboot/yaboot >-rw-r--r-- 1 root root 454294 Oct 29 12:02 >/usr/lib/yaboot/yaboot.debug >is the location of these files ok? smoreover the last three have no executable >permissions - seems to be bogus location for me See above. >- CHECK: permissions on files are set properly >see the previous >- GOOD: package has a %clean section >- GOOD: package uses macros consistently >- GOOD: no large documentation files >- GOOD: header files >- GOOD: no static libraries >- GOOD: no pkgconfig files >- GOOD: no libraries included >- GOOD: no devel package >- GOOD: no *.la libtool archives >- GOOD: no desktop file >- GOOD: owns files or directories owned by other packages: >- BAD: build root is cleaned at the beginning of %install >is not cleaned fixed >- GOOD: all file names are valid UTF-8 > >/usr/share/doc/yaboot-1.3.14/README.man.patch >and >/usr/share/doc/yaboot-1.3.14/Makefile > should not be in documentation fixed See task http://koji.fedoraproject.org/koji/taskinfo?taskID=1838144 -- 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