Bug 197837

Summary:	app-accesibility/mbrola polish and latin language addition
Product:	Gentoo Linux	Reporter:	Tomasz L. Nowak <nowak2000>
Component:	New packages	Assignee:	William Hubbs <williamh>
Status:	RESOLVED FIXED
Severity:	enhancement	CC:	accessibility, sound
Priority:	High
Version:	unspecified
Hardware:	All
OS:	All
Whiteboard:
Package list:		Runtime testing required:	---
Attachments:	patch adding polish and latin voices by linguas. fixed location for polish license.txt

Description Tomasz L. Nowak 2007-11-02 00:03:11 UTC

Please find attached ebuild with Polish and Latin voices added for app-accessibility/mbrola.
The Polish voice is moved to a right directory after unpacking, and the lowercase "test" directory for Latin is also installed.

Comment 1 Tomasz L. Nowak 2007-11-02 00:04:16 UTC

Created attachment 134978 [details, diff]
patch adding polish and latin voices by linguas. fixed location for polish license.txt

Comment 2 Tomasz L. Nowak 2007-11-02 09:49:18 UTC

Comment on attachment 134978 [details, diff]
patch adding polish and latin voices by linguas. fixed location for polish license.txt 

><HTML><HEAD/><BODY><PRE>--- mbrola-3.0.1h-r4.ebuild	2007-11-02 00:01:52.000000000 +0100
>+++ mbrola-3.0.1h-r5.ebuild	2007-11-02 00:56:22.000000000 +0100
>@@ -2,7 +2,7 @@
> # Distributed under the terms of the GNU General Public License v2
> # $Header: /var/cvsroot/gentoo-x86/app-accessibility/mbrola/mbrola-3.0.1h-r4.ebuild,v 1.13 2007/07/06 10:27:58 uberlord Exp $
> 
>-IUSE="linguas_de linguas_es linguas_fr linguas_it linguas_ro"
>+IUSE="linguas_de linguas_es linguas_fr linguas_it linguas_la linguas_pl linguas_ro"
> 
> S=${WORKDIR}
> 
>@@ -29,6 +29,8 @@
> 		${TCTS}/mbrola/dba/fr6/fr6-010330.zip )
> 	linguas_it? ( ${TCTS}/mbrola/dba/it3/it3-010304.zip
> 		${TCTS}/mbrola/dba/it4/it4-010926.zip )
>+	linguas_la? ( ${TCTS}/mbrola/dba/la1/la1.zip )
>+	linguas_pl? ( ${TCTS}/mbrola/dba/pl1/pl1.zip )
> 	linguas_ro? ( ${TCTS}/mbrola/dba/ro1/ro1-980317.zip )"
> 
> DEPEND="app-arch/unzip"
>@@ -42,6 +44,8 @@
> src_unpack () {
> 	unpack ${A}
> 
>+	[[ -f "pl1" ]] &amp;&amp; mkdir pl1DIR &amp;&amp; mv pl1 pl1.txt license.txt pl1DIR &amp;&amp; mv test pl1DIR/TEST &amp;&amp; mv pl1DIR pl1
>+
> 	case ${ARCH} in
> 		x86|amd64)
> 			cp mbrola-linux-i386 mbrola || die
>@@ -79,6 +83,11 @@
> 			doins ${voice}/TEST/*
> 		fi
> 
>+		if [[ -d "${voice}/test" ]]; then
>+			insinto /opt/${PN}/${voice}/test
>+			doins ${voice}/test/*
>+		fi
>+
> 		[[ -f "${voice}/${voice}.txt" ]] &amp;&amp; dodoc ${voice}/${voice}.txt
> 	done
> 
></PRE></BODY></HTML>

Comment 3 Tomasz L. Nowak 2007-11-02 10:09:18 UTC

I just added license.txt to move changing one line to:

+	[[ -f "pl1" ]] && mkdir pl1DIR && mv pl1 pl1.txt license.txt pl1DIR && mv test pl1DIR/TEST && mv pl1DIR pl1

Comment 4 Tomasz L. Nowak 2007-12-01 21:57:50 UTC

The corrected (repackaged) pl1.zip was received upstream. When it gets published I will update (simplify) the provided patch.

Comment 5 William Hubbs gentoo-dev

2008-01-13 19:13:10 UTC

What is the latest on this?  Has the new .zip been published upstream?

Thanks,

William

Comment 6 Tomasz L. Nowak 2008-01-13 19:41:37 UTC

No, still it hasn't. I pinged them today in case they forgot to publish it. An author of the latin package didn't even reply to my message.

Comment 7 William Hubbs gentoo-dev

2008-03-25 02:56:10 UTC

Hello,

do you want me to go ahead and commit this to the tree, or should I
wait until we hear back from upstream about the new .zip file?

Thanks,

William

Comment 8 Tomasz L. Nowak 2008-04-14 10:44:34 UTC

Yes, please proceed with committing. I can't get upstream replace the 2 files.

Comment 9 William Hubbs gentoo-dev

2008-05-21 01:59:28 UTC

This has been committed as part of -r5.  Thanks for letting me know about it.