Bug 231486 - Review Request: mecab-ipadic - IPA dictorionary for MeCab
Review Request: mecab-ipadic - IPA dictorionary for MeCab
Status: CLOSED NEXTRELEASE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Hans de Goede
Fedora Package Reviews List
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2007-03-08 12:22 EST by Mamoru TASAKA
Modified: 2007-11-30 17:11 EST (History)
0 users

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2007-03-24 15:10:49 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
hdegoede: fedora‑review+
dennis: fedora‑cvs+


Attachments (Terms of Use)

  None (edit)
Description Mamoru TASAKA 2007-03-08 12:22:55 EST
Spec URL: http://www.ioa.s.u-tokyo.ac.jp/~mtasaka/dist/extras/development/SPECS/mecab-ipadic.spec
SRPM URL: http://www.ioa.s.u-tokyo.ac.jp/~mtasaka/dist/extras/development/SRPMS/mecab-ipadic-2.7.0.20060707-1.fc7.src.rpm
Mockbuild log on FC-devel i386: http://www.ioa.s.u-tokyo.ac.jp/~mtasaka/dist/extras/development/LOGS/MOCK-mecab-ipadic.log

Description:
MeCab IPA is a dictionary for MeCab using CRF estimation
based on IPA corpus.

NOTE:
This package is very similar with mecab-jumandic
(bug 229929 : already imported)

rpmlint on binary rpm:
-----------------------------------------------------
W: mecab-ipadic invalid-license MIT-ish (See LICENSE.*)
E: mecab-ipadic no-binary
E: mecab-ipadic only-non-binary-in-usr-lib
W: mecab-ipadic-EUCJP invalid-license MIT-ish (See LICENSE.*)
E: mecab-ipadic-EUCJP only-non-binary-in-usr-lib
------------------------------------------------------
* For license: see LICENSE.Fedora
* no executable binaries are installed, but the
  installed data is compiled by arch-dependent binaries
  (in mecab: bug 229927, already imported) and the 
  created data are arch-dependent.
Comment 1 Hans de Goede 2007-03-24 03:46:04 EDT
MUST:
=====
* rpmlint output is:
W: mecab-ipadic invalid-license MIT-ish (See LICENSE.*)
E: mecab-ipadic no-binary
E: mecab-ipadic only-non-binary-in-usr-lib
W: mecab-ipadic-EUCJP invalid-license MIT-ish (See LICENSE.*)
E: mecab-ipadic-EUCJP only-non-binary-in-usr-lib
W: mecab-ipadic invalid-license MIT-ish (See LICENSE.*)
These are "normal" for this package and can be ignored
* Package and spec file named appropriately
* Packaged according to packaging guidelines
* License ok (approved by Spot)
* spec file is legible and in Am. English.
* Source matches upstream
* Compiles and builds on devel x86_64
* BR: ok
* No locales
* No shared libraries
* Not relocatable
* Package owns / or requires all dirs
0 No duplicate files & Permissions
* %clean & macro usage OK
* Contains code only
* %doc does not affect runtime, and isn't large enough to warrent a sub package
* no -devel package needed
* no .desktop file required

Approved!
Comment 2 Mamoru TASAKA 2007-03-24 08:25:57 EDT
Thank you!!

Request for CVS admin:
------------------------------------------------
New Package CVS Request
=======================
Package Name:              mecab-ipadic
Short Description:         IPA dictorionary for MeCab
Owners:                    mtasaka@ioa.s.u-tokyo.ac.jp
Branches:                  devel FC-6 FC-5
InitialCC:                 (nobody)
------------------------------------------------
Comment 3 Mamoru TASAKA 2007-03-24 08:28:28 EDT
Oops...

------------------------------------------------
New Package CVS Request
=======================
Package Name:              mecab-ipadic
Short Description:         IPA dictionary for MeCab
Owners:                    mtasaka@ioa.s.u-tokyo.ac.jp
Branches:                  devel FC-6 FC-5
InitialCC:                 (nobody)
------------------------------------------------
Comment 4 Dennis Gilmore 2007-03-24 11:02:01 EDT
Branched
Comment 5 Mamoru TASAKA 2007-03-24 15:10:49 EDT
Rebuilt on all branches.
Closing as CLOSED NEXTRELEASE.

Thank you for reviewing!!

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