Bug 485693 - Review Request: mythes-da - Danish thesaurus
Review Request: mythes-da - Danish thesaurus
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
All Linux
medium Severity medium
: ---
: ---
Assigned To: manuel wolfshant
Fedora Extras Quality Assurance
Depends On:
  Show dependency treegraph
Reported: 2009-02-16 07:03 EST by Caolan McNamara
Modified: 2009-02-18 18:01 EST (History)
2 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2009-02-18 18:01:55 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
wolfy: fedora‑review+
kevin: fedora‑cvs+

Attachments (Terms of Use)

  None (edit)
Description Caolan McNamara 2009-02-16 07:03:15 EST
Spec URL: http://people.redhat.com/caolanm/rpms/mythes-da.spec
SRPM URL: http://people.redhat.com/caolanm/rpms/mythes-da-0.1.9-0.1.beta.fc10.src.rpm
Description: Danish thesaurus
Comment 1 manuel wolfshant 2009-02-16 15:46:41 EST
All well, package APPROVED

PS: since you are using cp -p to install the actual dictionaries, it would be nice to do:
tr -d '\r' < README_th_da_DK.txt > README_th_da_DK.txt.new
touch -r  README_th_da_DK.txt README_th_da_DK.txt.new
mv -f README_th_da_DK.txt.new README_th_da_DK.txt
tr -d '\r' < README_th_en-US.txt > README_th_en-US.txt.new
touch -r README_th_en-US.txt README_th_en-US.txt.new
mv -f README_th_en-US.txt.new README_th_en-US.txt
Comment 2 Caolan McNamara 2009-02-16 18:17:43 EST
mythes-da of course, not hunspell-da.
Comment 3 Caolan McNamara 2009-02-16 18:18:42 EST
New Package CVS Request
Package Name: mythes-da
Short Description: Danish thesaurus
Owners: caolanm
Branches: devel
Comment 4 Kevin Fenzi 2009-02-18 15:01:39 EST
cvs done.

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