This service will be undergoing maintenance at 00:00 UTC, 2016-08-01. It is expected to last about 1 hours

Bug 457152

Summary: Review Request: appliance-tools tools for generating appliance images on Fedora
Product: [Fedora] Fedora Reporter: David Huff <dhuff>
Component: Package ReviewAssignee: David Lutterkort <lutter>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: fedora-package-review, hbrock, notting
Target Milestone: ---Flags: lutter: fedora‑review+
kevin: fedora‑cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2008-09-23 13:52:01 EDT Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:

Description David Huff 2008-07-29 17:34:59 EDT
Spec URL:
Tools for generating appliance images on Fedora based systems including
derived distributions such as RHEL, CentOS and others. See for more details.
Comment 1 David Lutterkort 2008-07-29 18:22:17 EDT
  OK - Package name
  N/A - License info is accurate

  Upstream does not include a separate license, though comments in the code
  indicate GPLv2 (would be good if upstream included a license; in that
  case, install it as %doc)

  OK - License tag is correct and licenses are approved
  N/A - License files are installed as %doc
  OK - Specfile name
  OK - Specfile is legible
  OK - No prebuilt binaries included
  OK - BuildRoot value (one of the recommended values)
  OK - PreReq not used
  N/A - Source md5sum matches upstream

  Upstream has not made an official release; it would be good if that
  happened. Barring that, there needs to be a comment before Source0
  explaining how the tarball was generated

  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
  OK - File permissions set properly
  OK - Macro usage is consistent
  OK - rpmlint is silent

  OK - definition of python_sitelib
  OK - Generate and include *.py[co] in sitelib
  OK - Do not include *.py[oc] in /usr/{bin,sbin}
Comment 2 David Lutterkort 2008-07-29 18:23:11 EDT
Please address the Source0 issue above (and since you're upstream, also include
an explicit license)

With these cosmetic changes, APPROVED

Please follow and import
the package. Close this bug as RAWHIDE once it's been successfully imported
and built.
Comment 3 David Huff 2008-08-06 17:33:18 EDT
New Package CVS Request
Package Name: appliance-tools
Short Description: tools for building appliance images
Owners: huff
Branches: F-9 EL-5
Cvsextras Commits: yes
Comment 4 Kevin Fenzi 2008-08-07 16:54:09 EDT
cvs done.