Bug 1005796 - Review Request: langdetect - Language Detection Library for Java
Summary: Review Request: langdetect - Language Detection Library for Java
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Zbigniew Jędrzejewski-Szmek
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 1005800
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-09-09 12:14 UTC by gil cattaneo
Modified: 2013-12-14 02:54 UTC (History)
3 users (show)

Fixed In Version: langdetect-1.1-0.2.20120112.fc20
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-12-14 02:54:47 UTC
zbyszek: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description gil cattaneo 2013-09-09 12:14:40 UTC
Spec URL: http://gil.fedorapeople.org/langdetect.spec
SRPM URL: http://gil.fedorapeople.org/-1.1-0.1.20120112.fc19.src.rpm
Description:
This is a language detection library implemented in plain Java.
(aliases: language identification, language guessing).
Fedora Account System Username: gil

Comment 2 Zbigniew Jędrzejewski-Szmek 2013-12-03 22:03:36 UTC
Issues:
1. Please do not add a license file which is not upstream. It is enough to ask upstream to include it.

2. After the license is added, I'd make langdetect-javadoc co-own /usr/share/doc/langdetect/LICENSE-2.0.txt to avoid the extra directory with one file.

3. suggested %description
langdetect is a language detection library implemented in plain Java.
It uses Bayesian filters generated from language samples retrieved from Wikipedia to guess the language of text, achieving accuracy above 99%.

+ Source URL works, checksum matches.
+ Spec file is very nice.
+ License is OK.
+ No rpmlint output.
+ Package follows Java packaging guidelines

Everything seems OK.

Comment 3 Zbigniew Jędrzejewski-Szmek 2013-12-03 22:04:18 UTC
Package is APPROVED.

Comment 4 gil cattaneo 2013-12-04 01:06:55 UTC
(In reply to Zbigniew Jędrzejewski-Szmek from comment #2)
> Issues:
> 1. Please do not add a license file which is not upstream. It is enough to
> ask upstream to include it.
> 
reported @ http://code.google.com/p/language-detection/issues/detail?id=57
usually we must add license txt file if this one is missing

> 2. After the license is added, I'd make langdetect-javadoc co-own
> /usr/share/doc/langdetect/LICENSE-2.0.txt to avoid the extra directory with
> one file.
> 
?
> 3. suggested %description
> langdetect is a language detection library implemented in plain Java.
> It uses Bayesian filters generated from language samples retrieved from
> Wikipedia to guess the language of text, achieving accuracy above 99%.
> 
Done
> + Source URL works, checksum matches.
> + Spec file is very nice.
> + License is OK.
> + No rpmlint output.
> + Package follows Java packaging guidelines
> 
> Everything seems OK.

Thanks!

New Package SCM Request
=======================
Package Name: langdetect
Short Description: Language Detection Library for Java
Owners: gil
Branches: f20
InitialCC: java-sig

Comment 5 Zbigniew Jędrzejewski-Szmek 2013-12-04 01:17:21 UTC
(In reply to gil cattaneo from comment #4)
> (In reply to Zbigniew Jędrzejewski-Szmek from comment #2)
> > Issues:
> > 1. Please do not add a license file which is not upstream. It is enough to
> > ask upstream to include it.
> > 
> reported @ http://code.google.com/p/language-detection/issues/detail?id=57
> usually we must add license txt file if this one is missing
The guidelines say:

It is important to reiterate that in situations where the indicated license does not imply a requirement that the license be distributed along with the source/binaries, Fedora packagers are NOT required to manually include the full license text when it is absent from the source code. but are still encouraged to point out this issue to upstream and encourage them to remedy it. 

... I took this to mean that the license file should *not* be included, but I guess it is allowed, just not required. So you're fine.

> > 2. After the license is added, I'd make langdetect-javadoc co-own
> > /usr/share/doc/langdetect/LICENSE-2.0.txt to avoid the extra directory with
> > one file.
The first part of the sentence was made in light of 1., so let me amend it to:

2. I'd make langdetect-javadoc co-own
/usr/share/doc/langdetect/LICENSE-2.0.txt to avoid the extra directory with
one file.

Comment 6 Gwyn Ciesla 2013-12-04 13:38:16 UTC
Git done (by process-git-requests).

Comment 7 Fedora Update System 2013-12-05 02:24:25 UTC
langdetect-1.1-0.2.20120112.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/langdetect-1.1-0.2.20120112.fc20

Comment 8 Fedora Update System 2013-12-05 21:27:33 UTC
langdetect-1.1-0.2.20120112.fc20 has been pushed to the Fedora 20 testing repository.

Comment 9 Fedora Update System 2013-12-14 02:54:47 UTC
langdetect-1.1-0.2.20120112.fc20 has been pushed to the Fedora 20 stable repository.


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