Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: PackageKit - Package management with dbus and policykit https://bugzilla.redhat.com/show_bug.cgi?id=333451 ------- Additional Comments From panemade@xxxxxxxxx 2007-10-16 23:24 EST ------- Thanks. 1) I found problem in mockbuild => RPM build errors: File not found by glob: /var/tmp/PackageKit-0.1.0-2.fc8-root-mockbuild/usr/share/man/man1/*.1.gz This is happening because of missing BR: docbook-utils in SPEC. 2) After adding missing BR when I mock build it went fine. But rpmlint complained PackageKit.i386: E: non-executable-script /usr/lib/python2.5/site-packages/packagekit/backend.py 0644 This text file contains a shebang or is located in a path dedicated for executables, but lacks the executable bits and cannot thus be executed. If the file is meant to be an executable script, add the executable bits, otherwise remove the shebang or move the file elsewhere. PackageKit.i386: E: non-executable-script /usr/lib/python2.5/site-packages/packagekit/frontend.py 0644 This text file contains a shebang or is located in a path dedicated for executables, but lacks the executable bits and cannot thus be executed. If the file is meant to be an executable script, add the executable bits, otherwise remove the shebang or move the file elsewhere. ===> Once you fix this in upstream. Kindly provide new SRPM for review with fixes mentioned in this comment. PackageKit.i386: W: file-not-utf8 /usr/share/doc/packagekit/reference.html The character encoding of this file is not UTF-8. Consider converting it in the specfile for example using iconv(1). ==> You need to use iconv command on that file. Not needed in SPEC as you are upstream you can release new tarball with this fix. PackageKit.i386: W: non-standard-dir-in-var db Your package is creating a non-standard subdirectory in /var. The standard directories are: account, lib, cache, crash, games, lock, log, opt, run, spool, state, tmp, yp, www, ftp ==> This looks fake warning as /var/db is already owned by filesystem package. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review