[Bug 194279] Review Request: kdeartwork: Additional artwork (themes, sound themes, ...) for KDE

bugzilla at redhat.com bugzilla at redhat.com
Sun Jun 18 04:16:16 UTC 2006


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: kdeartwork: Additional artwork (themes, sound themes, ...) for KDE


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=194279


imlinux at gmail.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
         AssignedTo|bugzilla-sink at leemhuis.info |imlinux at gmail.com




------- Additional Comments From imlinux at gmail.com  2006-06-18 00:08 EST -------
- Name looks good
- Source matches upstream
- Builds fine in mock against devel
- rpmlint:
  SRPM: 
      E: kdeartwork unknown-key GPG#ff6382fa
  kdeartwork:
      CLEAN
  kdeartwork-icons:
      W: kdeartwork-icons no-documentation
      E: kdeartwork-icons standard-dir-owned-by-package /usr/share/icons

One thing.  right now this package owns /usr/share/icons/  Aparently it's not alone.

rpm -qf /usr/share/icons

My box shows 6 packages owning that directory.  The guidelines state that your
package must own all directories it creates and must not own any directories
that another package creates.  My suggestion is to go ahead and take out
ownership of that directory and file a bugzilla for the filesystem package.  If
they refuse with a valid reason then I say go ahead and own it.

I'll wait for your response (and others) before I approve the package but aside
from the ownership issue this package is ready to go, that is of course
depending on what you decide to do for #195480

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.




More information about the Fedora-package-review mailing list