Bug 511280

Summary: Review Request: comoonics-cdsl-py - Cdsl tools written in python to manage cdsl (base on the open-sharedroot)
Product: [Fedora] Fedora Reporter: Marc Grimme <grimme>
Component: Package ReviewAssignee: Nils Philippsen <nphilipp>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: low    
Version: rawhideCC: fdanapfe, fedora-package-review, hlawatschek, notting, nphilipp
Target Milestone: ---Flags: nphilipp: fedora-review+
kevin: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2010-08-03 13:36:56 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 511277    
Bug Blocks:    

Description Marc Grimme 2009-07-14 14:46:17 UTC
Spec URL: http://www.open-sharedroot.org/development/comoonics-cdsl-py/comoonics-cdsl-py.spec
SRPM URL: http://www.open-sharedroot.org/development/comoonics-cdsl-py/comoonics-cdsl-py-0-2-15-src.rpm/view
Description: comoonics-cdsl-py
Python libraries for managing cdsl environments base on the open-sharedroot concept.

You can create, delete, validate cdsls with different commands.

This library is required for the open-sharedroot feature: 
https://fedoraproject.org/wiki/Features/Opensharedroot

It depends on comoonics-base-py. See bugzilla .

This is my first package. Therefore I need a sponsor. Thanks !!

Comment 1 Nils Philippsen 2009-07-15 12:21:06 UTC
Let's handle the sponsoring part in bug #511277 as it blocks this one.

Comment 3 Frank Danapfel 2009-07-24 14:23:05 UTC
I will have a look at this.

Comment 4 Frank Danapfel 2009-07-24 14:44:43 UTC
Things checked so far:

- GOOD: rpmlint run on doesn't flag errors/warnings
- GOOD: package name according to guidelines
- GOOD: spec file named properly
- GOOD: package meets packaging guidelines
- GOOD: licensed according to licensing guidelines
- GOOD: license in spec file matches license in package
- GOOD: license shipped as documentation
- GOOD: the spec file is written in American English
- GOOD: the spec file is legible
- GOOD: sources used to build the package match upstream source
- GOOD: builds in mock for x86_64/Rawhide
- GOOD: all build dependencies listed

Comment 5 Frank Danapfel 2009-07-27 07:58:37 UTC
The remaining requirements have also been checked:

- PASS: doesn't ship locale files
- PASS: no libraries shipped
- GOOD: package is not relocatable
- GOOD: all shipped directories owned by package, direct dependency or
filesystem
- GOOD: no duplicates in %files
- GOOD: permissions on files are set properly
- GOOD: package has a %clean section
- GOOD: package uses macros consistently
- GOOD: the package contains code, not content
- PASS: no large documentation files
- GOOD: %doc doesn't affect runtime
- PASS: no header files
- PASS: no static libraries
- PASS: no pkgconfig files
- PASS: no libraries included
- PASS: no devel package
- GOOD: no *.la libtool archives
- PASS: no desktop file
- GOOD: doesn't own files or directories owned by other packages
- GOOD: build root is cleaned at the beginning of %install
- GOOD: all file names are valid UTF-8

Comment 6 Nils Philippsen 2009-07-27 13:12:44 UTC
I concur with Frank's review. This package is APPROVED.

Comment 7 Marc Grimme 2009-07-27 14:36:25 UTC
New Package CVS Request
=======================
Package Name: comoonics-cdsl-py
Short Description: Comoonics cdsl utilities and library written in Python 
Owners: elcody02 markhla
Branches: devel
InitialCC:

Comment 8 Kevin Fenzi 2009-07-28 04:47:49 UTC
cvs done.

Comment 9 Nils Philippsen 2010-08-03 13:36:56 UTC
Closing this review ticket (Marc, that would have been your job ;-).