Bug 460690 - Review Request: echo-artist - Automation tools for echo-icon-theme artists
Summary: Review Request: echo-artist - Automation tools for echo-icon-theme artists
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Luya Tshimbalanga
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-08-29 18:17 UTC by Martin Sourada
Modified: 2008-09-10 06:46 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2008-09-10 06:44:38 UTC
Type: ---
Embargoed:
luya: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Martin Sourada 2008-08-29 18:17:34 UTC
Spec URL: http://mso.fedorapeople.org/packages/SPECS/echo-artist.spec
SRPM URL: http://mso.fedorapeople.org/packages/SRPMS/echo-artist-0.1-1.fc9.src.rpm
Description: Set of scripts for echo-icon-theme artists useful for setting up and maitaining their local git repository and creating and adding new icon sets to it.

Note: I am also upstream for the package.

Comment 1 Luya Tshimbalanga 2008-08-31 10:21:44 UTC
Taking over that package for review that will come soon.

Comment 2 Luya Tshimbalanga 2008-09-01 00:15:02 UTC
Here is a summary:

$ rpmlint rpmbuild/SRPMS/echo-artist-0.1-1.fc9.src.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings

$ rpmlint rpmbuild/RPMS/noarch/echo-artist-0.1-1.fc9.noarch.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

The package is correctly named according to  Package Naming Guidelines
https://fedoraproject.org/wiki/Packaging/NamingGuidelines

Name of spec files matches the %{name}

Package complies to Packaging Guidelines
https://fedoraproject.org/wiki/Packaging/Guidelines

Package is licensed under LGPLv2
Package is upstream and legible.

Source URL matches upstream (https://fedorahosted.org/releases/e/c/echo-icon-theme/)


Package does not use BuuldRequires because of scripts and uses Requires only.
Package successfully compiled using koji.



spec file contains %clean session.

Permission is set correctly %defattr(-,root,root,-)

Package is a set of script thus does not need .desktop.

Uses of macros is consistent. %{buildroot} would be favorite considering the internal change of rpm itself in Fedora 10.

filename are in validated UTF-8 format.


This package has passed all reviews thus ready to be accepted.

Comment 3 Martin Sourada 2008-09-01 08:47:56 UTC
(In reply to comment #2)
> Uses of macros is consistent. %{buildroot} would be favorite considering the
> internal change of rpm itself in Fedora 10.
> 
Good point. Changed :)

New SPEC: http://mso.fedorapeople.org/packages/SPECS/echo-artist.spec
New SRPM: http://mso.fedorapeople.org/packages/SRPMS/echo-artist-0.1-2.fc9.src.rpm

Comment 4 Martin Sourada 2008-09-01 08:55:30 UTC
New Package CVS Request
=======================
Package Name: echo-artist
Short Description: Automation tools for echo-icon-theme artists
Owners: mso
Branches: F-8 F-9
InitialCC:

Comment 5 Kevin Fenzi 2008-09-01 21:12:06 UTC
cvs done.

Comment 6 Fedora Update System 2008-09-01 22:38:37 UTC
echo-artist-0.1-2.fc9 has been submitted as an update for Fedora 9.
http://admin.fedoraproject.org/updates/echo-artist-0.1-2.fc9

Comment 7 Fedora Update System 2008-09-01 22:40:19 UTC
echo-artist-0.1-2.fc8 has been submitted as an update for Fedora 8.
http://admin.fedoraproject.org/updates/echo-artist-0.1-2.fc8

Comment 8 Fedora Update System 2008-09-10 06:44:35 UTC
echo-artist-0.1-2.fc9 has been pushed to the Fedora 9 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 9 Fedora Update System 2008-09-10 06:46:20 UTC
echo-artist-0.1-2.fc8 has been pushed to the Fedora 8 stable repository.  If problems still persist, please make note of it in this bug report.


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