Bug 622270 - Review Request: elementary-icon-theme - icons from the Elementary Project
Summary: Review Request: elementary-icon-theme - icons from the Elementary Project
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Mamoru TASAKA
QA Contact: Fedora Extras Quality Assurance
: 693375 (view as bug list)
Depends On:
TreeView+ depends on / blocked
Reported: 2010-08-08 14:27 UTC by Tajidin Abdullah
Modified: 2011-04-05 23:04 UTC (History)
4 users (show)

Clone Of:
Last Closed: 2010-08-25 18:33:19 UTC
mtasaka: fedora-review+
kevin: fedora-cvs+

Attachments (Terms of Use)

Description Tajidin Abdullah 2010-08-08 14:27:07 UTC
Spec URL: http://tajidinabd.fedorapeople.org/elementary-icon-theme/elementary-icon-theme.spec
SRPM URL: http://tajidinabd.fedorapeople.org/elementary-icon-theme/elementary-icon-theme-2.4-1.fc13.src.rpm
Description: Icon theme set From Elementary Project

This is my first package I will be needing a sponsor

Comment 1 Tajidin Abdullah 2010-08-08 14:30:05 UTC
Also like to include a link to successful koji build with the srpm 


Comment 2 Mamoru TASAKA 2010-08-08 16:49:23 UTC

Comment 3 Tajidin Abdullah 2010-08-08 21:39:51 UTC
Made corrections to intial submission due to scriplets omission and tarname macro. However having problem with documentation

Spec Url: http://tajidinabd.fedorapeople.org/elementary-icon-theme/elementary-icon-theme.spec

Srpm Url: http://tajidinabd.fedorapeople.org/elementary-icon-theme/elementary-icon-theme-2.4-2.fc13.src.rpm

successful koji scratch

Comment 4 Tajidin Abdullah 2010-08-09 19:22:40 UTC
made more corrections to the spec file cleaned a few issues in regards to the prep and clarified issue with the version number

also still looking for guidance about the %doc i know in fedora documentation is needed. but i couldnt get it to build with the documentation. any help on this issue thank you 

spec url: http://tajidinabd.fedorapeople.org/elementary-icon-theme/elementary-icon-theme.spec

srpm url: http://tajidinabd.fedorapeople.org/elementary-icon-theme/elementary-icon-theme-2.4-3.fc13.src.rpm

Comment 5 Germán Racca 2010-08-10 03:50:12 UTC
Add the following line:


(not tested!)


Comment 6 Tajidin Abdullah 2010-08-10 08:15:04 UTC
waiting for response from upstream in regards to changing name of source package to reflect version number. 

yes that tip did work German thanks for that.

Comment 7 Mamoru TASAKA 2010-08-16 19:47:31 UTC
Some notes:

* License tag
  - From elementary-monochrome/AUTHORS:
     8  This package is licensed under GNU General Public License version 2.
    From elementary/AUTHORS:
     8  This package is licensed under GNU General Public License version 2.
    Therefore the license tag should be "GPLv2".

* Timestamps
  - Please use "cp -a" instead of "cp -R" to keep timestamps on installed
    files as much as possible:

* Documentation
  - I think
    should also be installed as %doc and be installed under %_defaultdocdir.
    The example would be
At %prep:
mkdir -p doc/elementary{,-monochrome}
cp -p elementary-monochrome/[A-Z]* doc/elementary-monochrome/
cp -p elementary/[A-Z]* doc/elementary/

And at %files:
%doc doc/*

Comment 8 Tajidin Abdullah 2010-08-17 09:15:36 UTC
made changes to spec accordingly with the license, timestamps, and documentation 

spec URL: http://tajidinabd.fedorapeople.org/elementary-icon-theme/elementary-icon-theme.spec

srpm URL: http://tajidinabd.fedorapeople.org/elementary-icon-theme/elementary-icon-theme-2.4-5.fc13.src.rpm

thanks for the review

Comment 9 Mamoru TASAKA 2010-08-17 17:14:48 UTC
For -5:

* Directory ownership issue
  - Now the following directories (themselves) are not owned by any packages
    These directories should be owned by this rpm. Please refer

Comment 11 Mamoru TASAKA 2010-08-17 18:45:24 UTC

   This package (elementary-icon-theme) is APPROVED by mtasaka

Comment 12 Tajidin Abdullah 2010-08-18 09:40:19 UTC
New Package SCM Request
Package Name: elementary-icon-theme
Short Description: icons from the Elementary Project
Owners: tajidinabd
Branches: f12 f13 f14

Comment 13 Kevin Fenzi 2010-08-18 21:16:37 UTC
Git done (by process-git-requests).

Comment 14 Mamoru TASAKA 2010-08-25 18:33:19 UTC

Comment 15 Thomas Moschny 2011-04-05 23:04:14 UTC
*** Bug 693375 has been marked as a duplicate of this bug. ***

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