[Bug 467154] Review Request: libvirt-qpid - An interface with libvirt using QMF (qpid modeling framework) which utilizes the Advanced Message Queuing protocol

bugzilla at redhat.com bugzilla at redhat.com
Thu Feb 26 00:08:43 UTC 2009


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=467154


David Lutterkort <lutter at redhat.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED




--- Comment #10 from David Lutterkort <lutter at redhat.com>  2009-02-25 19:08:42 EDT ---
based on the Koji scratch build in
http://koji.fedoraproject.org/koji/taskinfo?taskID=1177393, a few minor things
remain to be fixed:

  OK - Package name
  OK - License info is accurate
  OK - License tag is correct and licenses are approved
  OK - License files are installed as %doc
  OK - Specfile name
  OK - Specfile is legible
  OK - No prebuilt binaries included
  FIX - BuildRoot value (one of the recommended values)
       See https://fedoraproject.org/wiki/Packaging/Guidelines#BuildRoot_tag
  OK - PreReq not used
  FIX - Source md5sum matches upstream
       No upstream release; make Source a URL to the download for the tarball
  OK - No hardcoded pathnames
  OK - Package owns all the files it installs
  OK - 'Requires' create needed unowned directories
  OK - Package builds successfully on i386 and x86_64 (mock)
  OK - BuildRequires sufficient
  FIX - File permissions set properly
     rpmlint complains that
      /usr/share/doc/libvirt-qpid-0.2.12 and /usr/share/libvirt-qpid
     are mode 02755
  OK - Macro usage is consistent
  FIX - rpmlint is silent
     See above warnings about directory perms
  OK - Proper debuginfo packages

-- 
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.




More information about the Fedora-package-review mailing list