Bug 475037 - Review Request: stardict-dic-hi - Hindi Dictionary for stardict
Summary: Review Request: stardict-dic-hi - Hindi Dictionary for stardict
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Parag AN(पराग)
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-12-06 21:23 UTC by Rakesh Pandit
Modified: 2009-03-09 22:59 UTC (History)
3 users (show)

Fixed In Version: 3.0.1-4.fc9
Clone Of:
Environment:
Last Closed: 2009-02-02 15:05:04 UTC
Type: ---
Embargoed:
panemade: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Rakesh Pandit 2008-12-06 21:23:05 UTC
Description:
Hindi Dictionary for stardict.

SRPM: http://rakesh.fedorapeople.org/srpm/stardict-dic-hi-3.0.1-1.fc10.src.rpm
SPEC: http://rakesh.fedorapeople.org/spec/stardict-dic-hi.spec

Original work is by Sriram Chaudhry. He has converted original shabdanjali dictionary to what it is at http://rakesh.fedorapeople.org/misc/shabdanjali-fedora.tar.gz

I had been in touch with him, and he will be requesting to keep this tarball at http://ltrc.iiit.net/showfile.php?filename=downloads/shabdanjali-stardict/index.html But that would be a long process. To cut short I would host his work to start this review process quickly and will update later on in case he gets hosting on iiit site.

Comment 1 Rakesh Pandit 2008-12-08 03:35:49 UTC
Will check with possibilities to resolve authentic source issue and update shortly.

Comment 2 Rakesh Pandit 2009-01-08 19:07:52 UTC
I had a detailed conversation with author and he an emplyee at an Institute which has a very long and tiring process of updating the site hosting the source file. The link to that page is:

http://ltrc.iiit.net/showfile.php?filename=downloads/shabdanjali-stardict/index.html

The source there contains a binary rpm which I have removed using a shell script and re-compressed the resulting stuff and included into package. The normal process of removing it will be too long, so this workaround.

SPEC: http://rakesh.fedorapeople.org/spec/stardict-dic-hi.spec
SRPM: http://rakesh.fedorapeople.org/srpm/stardict-dic-hi-3.0.1-2.fc10.src.rpm

Comment 3 Parag AN(पराग) 2009-01-30 08:40:56 UTC
how can I use generate-tarball.sh? Provide usage and add same in SPEC file.

Comment 5 Parag AN(पराग) 2009-01-30 09:53:46 UTC
Review:
+ package builds in mock.
 Koji build => http://koji.fedoraproject.org/koji/taskinfo?taskID=1093215
+ rpmlint is silent for SRPM and for RPM.
- packaged source file will does not match generated source as md5sum always changes whenever we generate tarball from upstream source.
+ 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.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ %doc files present.
+ BuildRequires are proper.
+ defattr usage is correct.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ no static libraries.
+ no .pc file present.
+ no -devel subpackage exists.
+ no .la files.
+ no translations are available.
+ Does owns the directories it creates.
+ no duplicates in %files.
+ file permissions are appropriate.
+ no scriptlets are used.
+ Not a GUI app.

Suggestions:
1) preserve timestamps in cp command using "cp -p"

APPROVED.

Comment 6 Rakesh Pandit 2009-01-30 11:24:27 UTC
Updated:
http://rakesh.fedorapeople.org/spec/stardict-dic-hi.spec
http://rakesh.fedorapeople.org/srpm/stardict-dic-hi-3.0.1-4.fc10.src.rpm

Thanks for review.

New Package CVS Request
=======================
Package Name: stardict-dic-hi
Short Description: Hindi Dictionary for stardict
Owners: rakesh
Branches: F-9 F-10
InitialCC:
Cvsextras Commits: yes

Comment 7 Kevin Fenzi 2009-02-01 19:04:32 UTC
cvs done.

Comment 8 Fedora Update System 2009-03-09 22:49:45 UTC
stardict-dic-hi-3.0.1-4.fc10 has been pushed to the Fedora 10 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 9 Fedora Update System 2009-03-09 22:59:28 UTC
stardict-dic-hi-3.0.1-4.fc9 has been pushed to the Fedora 9 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.