Bug 253757 - Review Request: tesseract-langpack - Langpacks for tesseract
Summary: Review Request: tesseract-langpack - Langpacks for tesseract
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Michał Bentkowski
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2007-08-21 18:27 UTC by Karol Trzcionka
Modified: 2011-01-19 21:57 UTC (History)
4 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2007-08-24 16:34:31 UTC
Type: ---
Embargoed:
mr.ecik: fedora-review+
j: fedora-cvs+


Attachments (Terms of Use)

Description Karol Trzcionka 2007-08-21 18:27:52 UTC
Spec URL: http://karlik.nonlogic.org/tesseract/tesseract-langpack.spec
SRPM URL: http://karlik.nonlogic.org/tesseract/tesseract-langpack-2.00-1.fc7.src.rpm
Description:
Langpacks for tesseract. The english version is included in core package.
It will be working with: http://karlik.nonlogic.org/tesseract/tesseract.spec

Comment 1 Dominik 'Rathann' Mierzejewski 2007-08-21 20:06:54 UTC
Some comments after a quick glance:

Tarball versioning suggests that these files are for tesseract version 2.0. I
think you should use Requires: %{upstreamname} = %{version}. They might work
with newer releases, too, but check with upstream if this is true or not.

In each %files section, you should mark the files using the %lang macro, i.e.:
%lang(fr) %{_datadir}/%{upstreamname}/tessdata/fra*

I might do a formal review in a few days, if nobody beats me to it.

Comment 2 Karol Trzcionka 2007-08-21 20:24:33 UTC
see: http://code.google.com/p/tesseract-ocr/downloads/list
There is that the lagpacks are for 2.00 and up
Thanks for suggest with %lang, I will fix it after review (with other changes or
before build on koji).

Comment 3 Michał Bentkowski 2007-08-24 12:06:46 UTC
REVIEW:
 * rpmlint:
W: tesseract-langpack-de no-documentation
E: tesseract-langpack-de zero-length /usr/share/tesseract/tessdata/deu.user-
words
W: tesseract-langpack-es no-documentation
E: tesseract-langpack-es zero-length /usr/share/tesseract/tessdata/spa.user-
words
W: tesseract-langpack-fr no-documentation
E: tesseract-langpack-fr zero-length /usr/share/tesseract/tessdata/fra.user-
words
W: tesseract-langpack-it no-documentation
E: tesseract-langpack-it zero-length /usr/share/tesseract/tessdata/ita.user-
words
W: tesseract-langpack-nl no-documentation
E: tesseract-langpack-nl zero-length /usr/share/tesseract/tessdata/nld.user-
words
E: tesseract-langpack-nl script-without-shebang /usr/share/tesseract/tessdata/
nld.DangAmbigs

nld.DangAmbigs has to be chmoded -x

 * sums (sha1):
d8d89e2681421a144f3a1ec61ee0c530686f5096  tesseract-2.00.deu.tar.gz
24f58cab2b1418816ba09e6f28e33aee34605889  tesseract-2.00.fra.tar.gz
8ac6c41708950660fd66789f5925068577fb89b1  tesseract-2.00.ita.tar.gz
8bc0a5877c1992ab10a9e124643fd9006ff30350  tesseract-2.00.nld.tar.gz
49206b79407cb23bc23b1cd95bee02544f2fcf9b  tesseract-2.00.spa.tar.gz

every sum matches with one given on http://code.google.com/p/tesseract-ocr/
downloads/list

everything else is fine. 

(In reply to comment #1)
> In each %files section, you should mark the files using the %lang macro, i.e.:
> %lang(fr) %{_datadir}/%{upstreamname}/tessdata/fra*

It doesn't need to be done while these aren't locale files actually, but 
language data files.

I'll approve this. Just chmod -x nld.DangAmbigs when you submit this.

Comment 4 Karol Trzcionka 2007-08-24 12:44:14 UTC
New Package CVS Request
=======================
Package Name: tesseract-langpack
Short Description: Langpacks for tesseract
Owners: karlikt
Branches: F-7
InitialCC: 
Cvsextras Commits: yes

Comment 5 Kevin Fenzi 2007-08-24 15:52:16 UTC
cvs done.

Comment 6 Karol Trzcionka 2007-08-24 16:34:31 UTC
Thanks.

Comment 7 Simon 2011-01-19 21:33:48 UTC
Karol, please add a SCM request for EL-6 as well like in tesseract.
https://bugzilla.redhat.com/show_bug.cgi?id=220979#c19

Comment 8 Karol Trzcionka 2011-01-19 21:47:03 UTC
Package Change Request
======================
Package Name: tesseract-langpack
New Branches: el6
Owners: karlik cassmodiah

Comment 9 Jason Tibbitts 2011-01-19 21:57:04 UTC
Git done (by process-git-requests).


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