Bug 511280 - Review Request: comoonics-cdsl-py - Cdsl tools written in python to manage cdsl (base on the open-sharedroot)
Summary: Review Request: comoonics-cdsl-py - Cdsl tools written in python to manage cd...
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Nils Philippsen
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 511277
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-07-14 14:46 UTC by Marc Grimme
Modified: 2010-08-03 13:36 UTC (History)
5 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2010-08-03 13:36:56 UTC
Type: ---
Embargoed:
nphilipp: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

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


Note You need to log in before you can comment on or make changes to this bug.