[Bug 253757] Review Request: tesseract-langpack - Langpacks for tesseract

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: tesseract-langpack - Langpacks for tesseract


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=253757


rpm at greysector.net changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |rpm at greysector.net




------- Additional Comments From rpm at greysector.net  2007-08-21 16:06 EST -------
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.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.