Bug 476462 – app-accessibility/espeak-1.47.11 version bump

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Chris Brannon (RETIRED) 2013-07-10 23:17:56 UTC]

An ebuild for 1.47.11 is attached.
Notable changes are:

* It is now possible to link against both portaudio and pulseaudio
  simultaneously.  In this case, espeak will use pulse if it can,
  falling back on portaudio if it cannot.

* /usr/share/espeak-data/voices/en is now a regular file,
  rather than a directory.  Portage complains about this, so we remove
  the directory in pkg_preinst.

[Comment 1 Chris Brannon (RETIRED) 2013-07-10 23:18:29 UTC]

Created attachment 353044 [details]
espeak-1.47.11.ebuild

ebuild for app-accessibility/espeak-1.47.11.

[Comment 2 Jeroen Roovers (RETIRED) 2013-07-10 23:53:35 UTC]

Comment on attachment 353044 [details]
espeak-1.47.11.ebuild

--- espeak-1.46.01.ebuild       2013-02-04 13:33:22.830930416 +0100
+++ -   2013-07-11 01:52:12.678123033 +0200
@@ -14,8 +14,6 @@
 SLOT="0"
 KEYWORDS="~alpha ~amd64 ~arm ~hppa ~ia64 ~ppc ~ppc64 ~sparc ~x86 ~amd64-linux ~x86-linux"
 IUSE="portaudio pulseaudio"
-REQUIRED_USE="portaudio? ( !pulseaudio )
-       pulseaudio? ( !portaudio )"
 RDEPEND="media-sound/sox
        pulseaudio? ( media-sound/pulseaudio )
        portaudio? ( >=media-libs/portaudio-19_pre20071207 )"
@@ -25,7 +23,9 @@
 S=${WORKDIR}/${MY_P}
 
 get_audio() {
-       if use portaudio; then
+       if use portaudio && use pulseaudio; then
+               echo runtime
+       elif use portaudio; then
                echo portaudio
        elif use pulseaudio; then
                echo pulseaudio
@@ -63,6 +63,14 @@
        dohtml -r docs/*
 }
 
+pkg_preinst() {
+       local voicedir="${ROOT}/usr/share/${PN}-data/voices/en"
+       local errormsg="${voicedir} not empty.  Please remove it manually."
+       if [ -d "${voicedir}" ]; then
+               rmdir "${voicedir}" || die "${errormsg}"
+       fi
+}
+
 pkg_postinst() {
        if ! use portaudio && ! use pulseaudio; then
                ewarn "Since portaudio and pulseaudio are not in your use flags,"

[Comment 3 William Hubbs 2013-07-11 02:57:48 UTC]

(In reply to Jeroen Roovers from comment #2)
> Comment on attachment 353044 [details]
*snip*

@jer:
Please do not add a diff in a comment; it should be attached. There is no way to apply a diff like that from a comment.

Thanks,

William

[Comment 4 William Hubbs 2013-07-11 05:16:23 UTC]

This is now in portage. Thanks much.