Bug 427508 - Review Request: hunspell-hi - Hindi hunspell dictionaries
Summary: Review Request: hunspell-hi - Hindi hunspell dictionaries
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Jason Tibbitts
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-01-04 10:59 UTC by Parag Nemade
Modified: 2008-01-07 16:32 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-01-07 07:14:22 UTC
Type: ---
Embargoed:
j: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Parag Nemade 2008-01-04 10:59:24 UTC
Spec URL: http://paragn.fedorapeople.org/hunspell/hunspell-hi.spec
SRPM URL: http://paragn.fedorapeople.org/hunspell/hunspell-hi-20050726-1.fc8.src.rpm
Description: Hindi hunspell dictionaries

Comment 1 Jason Tibbitts 2008-01-05 22:31:44 UTC
As with the other package, you should include the Copyright file as %doc.

That's the only issue I see, so I'll go ahead and approve this and you can fix
it when you check in.

* source files match upstream:
   2e78f20096dfa254fe92ea7cf6ec3ed19cdf664e68eea964c56651c8e79e8345  
   hi-demo.tar.gz
* package meets naming and versioning guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* summary is OK.
* description is OK.
* dist tag is present.
* build root is OK.
* license field matches the actual license.
* license is open source-compatible.
* license text included in package.
* latest version is being packaged.
* BuildRequires are proper.
* %clean is present.
* package builds in mock (rawhide, x86_64).
* package installs properly
* rpmlint is silent.
* final provides and requires are sane:
   hunspell-hi = 20050726-1.fc9
  =
   hunspell

* %check is not present; I don't know any Hindi so I cannot test this.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* no scriptlets present.
* acceptable content.
* documentation is small, so no -docs subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.

APPROVED

Comment 2 Parag Nemade 2008-01-06 03:14:36 UTC
New Package CVS Request
=======================
Package Name: hunspell-hi
Short Description: Hindi hunspell dictionaries
Owners: pnemade
Branches: F-8
InitialCC:
Cvsextras Commits: no



Comment 3 Kevin Fenzi 2008-01-06 17:35:48 UTC
cvs done. 

Any particular reason to disallow cvsextras? 

Comment 4 Parag Nemade 2008-01-07 03:35:50 UTC
make it yes. I have no problem in making cvsextras commit to yes.

Comment 5 Parag Nemade 2008-01-07 07:14:22 UTC
Built for devel and F-8 successfully.
Thanks to tibbs and kevin.

Comment 6 Kevin Fenzi 2008-01-07 16:32:13 UTC
ok. Done. Thanks.


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