Bug 485693

Summary: Review Request: mythes-da - Danish thesaurus
Product: [Fedora] Fedora Reporter: Caolan McNamara <caolanm>
Component: Package ReviewAssignee: manuel wolfshant <manuel.wolfshant>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: fedora-package-review, notting
Target Milestone: ---Flags: manuel.wolfshant: fedora-review+
kevin: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2009-02-18 23:01:55 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Caolan McNamara 2009-02-16 12:03:15 UTC
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 20:46:41 UTC
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 23:17:43 UTC
mythes-da of course, not hunspell-da.

Comment 3 Caolan McNamara 2009-02-16 23:18:42 UTC
New Package CVS Request
=======================
Package Name: mythes-da
Short Description: Danish thesaurus
Owners: caolanm
Branches: devel
InitialCC:

Comment 4 Kevin Fenzi 2009-02-18 20:01:39 UTC
cvs done.