Bug 615192 - Review Request: cjkuni-fonts-ghostscript -Chinese Unicode TrueType font ghostscript files
Summary: Review Request: cjkuni-fonts-ghostscript -Chinese Unicode TrueType font ghost...
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Parag AN(पराग)
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2010-07-16 06:12 UTC by Peng Wu
Modified: 2010-08-30 05:16 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2010-08-30 05:16:25 UTC
Type: ---
Embargoed:
panemade: fedora-review+


Attachments (Terms of Use)

Description Peng Wu 2010-07-16 06:12:43 UTC
Spec URL: http://pwu.fedorapeople.org/cjkuni-fonts-ghostscript/cjkuni-fonts-ghostscript.spec
SRPM URL: http://pwu.fedorapeople.org/cjkuni-fonts-ghostscript/cjkuni-fonts-ghostscript-0.2.20080216.1-43.fc13.src.rpm
Description: 
CJK Unifonts are Unicode TrueType fonts derived from original fonts made available by Arphic Technology under "Arphic Public License" and extended by the CJK Unifonts project.

CJK Unifonts ghostscript files.

Comment 2 Peng Wu 2010-07-23 06:24:00 UTC
According to review comments, fixes license issue.

Here are new rpm spec for review:
Spec URL:
http://pwu.fedorapeople.org/cjkuni-fonts-ghostscript/cjkuni-fonts-ghostscript.spec
SRPM URL:
http://pwu.fedorapeople.org/cjkuni-fonts-ghostscript/cjkuni-fonts-ghostscript-0.3-1.fc13.src.rpm

Comment 3 Parag AN(पराग) 2010-07-27 04:14:46 UTC
Review:
+ package builds in mock (rawhide i686).
koji Build =>http://koji.fedoraproject.org/koji/taskinfo?taskID=2350726
+ rpmlint output for SRPM and for RPM.
cjkuni-fonts-ghostscript.src: W: no-cleaning-of-buildroot %install
cjkuni-fonts-ghostscript.src: W: no-cleaning-of-buildroot %clean
cjkuni-fonts-ghostscript.src: W: no-buildroot-tag
cjkuni-fonts-ghostscript.src: W: no-%clean-section
=> These are not needed to fixed.
+ source files match upstream url (sha1sum)
08521808c1a1ec4cd12f940a2ae9f53fc0b4c685  cjkuni-fonts-gscid-0.3.tgz
08521808c1a1ec4cd12f940a2ae9f53fc0b4c685  cjkuni-fonts-gscid-0.3.tgz.srpm
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ license is open source-compatible.
+ License text is included in package.
+ %doc is present.
+ BuildRequires are proper.
+ %clean is present which is not needed now.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no headers or static libraries.
+ no .pc file present.
+ no -devel subpackage
+ no .la files.
+ no translations are available
+ Does owns the directories it creates.
+ no scriptlets present.
+ no duplicates in %files.
+ file permissions are appropriate.
+ Not a GUI application

Suggestions:
1) looks like license headers in files have text mentioning license is given in COPYING file whereas new upstream tarball provides LICENSE file.

Can it be possible to upstream to change reference of COPYING word to LICENSE word in all 6 files?

APPROVED.

Comment 4 Peng Wu 2010-08-30 05:16:25 UTC
This package has been renamed to ghostscript-chinese.
See: [Bug 625678] Review Request: ghostscript-chinese - Ghostscript Chinese fonts configuration files.


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