[Bug 333451] Review Request: PackageKit - Package management with dbus and policykit

[bugzilla at redhat.com]

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 at gmail.com  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.