Bug 481732 - Review Request: stardict-dic-cs_CZ - czech dictionary for stardict
Summary: Review Request: stardict-dic-cs_CZ - czech dictionary for stardict
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Lubomir Rintel
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-01-27 12:53 UTC by Petr Sklenar
Modified: 2009-04-14 18:58 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-04-14 18:58:42 UTC
Type: ---
Embargoed:
lkundrak: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Petr Sklenar 2009-01-27 12:53:29 UTC
Spec URL: http://people.redhat.com/psklenar/stardict-english-czech.spec
SRPM URL: http://people.redhat.com/psklenar/stardict-english-czech-1.0.0-1.fc10.src.rpmCzech dictionaries for StarDict.

Description:

This is autogenerated dictionary for StarDict.
Data were downloaded from following website:
GNU/FDL: http://slovnik.zcu.cz/

Dictionary was generated using:
GNU/FDL Anglicko-Český slovník to stardict converter version 1.2
You can get conversion script from:
http://cihar.com/software/slovnik

Comment 1 Petr Sklenar 2009-01-27 13:06:16 UTC
Hello, this is my second package, I am seeking sponsor like at bug 481272

There is GNU/FDL licence in separate file /usr/share/doc/stardict-english-czech-20081201/README

Should I rename it to 'LICENSE' or is it OK? Original author has this name of file for licence.

I made mistake for URL in first comment, this is right URL: 
Spec URL: http://people.redhat.com/psklenar/stardict-english-czech.spec
SRPM URL:
http://people.redhat.com/psklenar/stardict-english-czech-1.0.0-1.fc10.src.rpm

Czech dictionaries for StarDict.

Comment 2 manuel wolfshant 2009-01-27 13:13:36 UTC
No, it is not necessary to rename the files from the source (unless there is a very good reason, for instance in order to avoid a conflict with another package)

Comment 3 Lubomir Rintel 2009-01-28 13:29:02 UTC
Taking this.

(In reply to comment #1)
> Hello, this is my second package, I am seeking sponsor like at bug 481272

I could sponsor you, but I can't find any reviews you've participated in. Would you mind pointing me to some, so I can be sure about your packaging abilities? In case you haven't done so, I recommend picking up a couple of review requests and doing preliminary reviews. Thanks!

Comment 4 Petr Sklenar 2009-01-28 13:44:51 UTC
I started with fedora packaging these time so I haven't done any pre-review, but I would like.

Please can you recommend me some review to do preliminary review.

Comment 5 manuel wolfshant 2009-01-28 13:51:05 UTC
Petr, you could start for instance with https://bugzilla.redhat.com/show_bug.cgi?id=480860 which is an easy one.

Comment 6 Petr Sklenar 2009-01-28 14:01:48 UTC
re Comment #5:
OK, thanks much

Comment 7 Lubomir Rintel 2009-01-28 14:11:01 UTC
Here's the review queue:

http://fedoraproject.org/PackageReviewStatus/REVIEW.html

You are probably looking for bugs in NEW state, assigned to (Nobody).
By skimming through the list I've managed to find one package for which the review did not start at all yet: python-argparse (bug #462505)

You may want to help your colleagues with Merge reviews as well ;)

Comment 8 Petr Sklenar 2009-01-29 14:56:18 UTC
Hi,
I made informal review at https://bugzilla.redhat.com/show_bug.cgi?id=480860#c4

Comment 9 Lubomir Rintel 2009-01-29 17:58:28 UTC
(In reply to comment #8)
> Hi,
> I made informal review at https://bugzilla.redhat.com/show_bug.cgi?id=480860#c4

I like it. I'd still like to se at least a couple more, this one was pretty easy :)

A review of this package:

1.) Package name.

To be consistent with other existing practice applied to other stardict dictionary packages I suggest you rename the package to "stardict-dic-cs" or "stardict-dic-cs_CZ".

You may still want to add a Provides: for the upstream name.

2.) Package version.

%define dicdate 20081201
Version: 1.0.0
Source0: ftp://dl.cihar.com/slovnik/stable/stardict-english-czech-%{dicdate}.tar.gz

Is 1.0.0 made up? Why not use 20081201 as version?

3.) URL tag.

URL: ftp://dl.cihar.com/slovnik/stable/stardict-english-czech-%{dicdate}.tar.gz

Purpose of the URL tag is to address the package's home page, with general information about it -- it definitely should not be the same as Source tag.

4.) BuildArchitectures tag.

This is fairly minor, but: it is commonly spelled "BuildArch:"

5.) Setup

%setup -c -T -n %{name}-%{version}
%setup -q -n %{name}-%{version} -D -T -a 0

This could be written as:

%setup -q -c -n %{name}-%{version}

6.) No need to rename the README file

install -p -m 0644 %{name}-%{dicdate}/README LICENSE

Especially when it does not contain the license text.

7.) Your %description is not informative

Description is not meant to contain information about history of the package or its build process, but rather contents of the package. The latter URL belongs to the Url: tag and rest should be replaced with something like:

"Czech-English and English-Czech translation dictionaries for StarDict, a GUI-based dictionary software."

8.) Group.

I guess Applications/Text would be a far better pick than Applications/System.

Comment 10 Lubomir Rintel 2009-02-09 14:45:04 UTC
Petr: ping. Alive? :)

Comment 11 Petr Sklenar 2009-02-09 16:32:26 UTC
hi, yes I live :)

I made changes up your comments, and latest version is here:
 
Spec URL: http://people.redhat.com/psklenar/stardict-dic-cs_CZ.spec

SRPM URL: http://people.redhat.com/psklenar/stardict-dic-cs_CZ-
20081201-2.fc10.src.rpm

Comment 12 Lubomir Rintel 2009-02-19 09:01:04 UTC
Sorry for the delay. I think it is much better now, thanks. Radek already sponsored you, lifting FE_NEEDSPONSOR.

APPROVED

Comment 13 Petr Sklenar 2009-03-02 12:39:40 UTC
New Package CVS Request
=======================
Package Name: stardict-english-czech
Short Description: Czech dictionaries for StarDict.
Owners: psklenar
Branches: F-10 F-11
InitialCC: psklenar

Comment 14 Kevin Fenzi 2009-03-03 00:28:38 UTC
Is the package name right here?
The cvs request has 'stardict-english-czech', comment #11 has 'stardict-dic-cs_CZ'.

Comment 15 Petr Sklenar 2009-03-03 12:55:18 UTC
yes , you are right, I am sorry, my mistake.

again:

New Package CVS Request
=======================
Package Name: stardict-dic-cs_CZ
Short Description: Czech dictionaries for StarDict.
Owners: psklenar
Branches: F-10 F-11
InitialCC: psklenar

Comment 16 Kevin Fenzi 2009-03-03 20:36:06 UTC
cvs done.

Comment 17 Lubomir Rintel 2009-04-14 18:58:42 UTC
I think this was imported and built.


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