mimic: new package

Signed-off-by: Ezequiel Garcia <ezequiel@vanguardiasur.com.ar>
---
 package/Config.in        |  1 +
 package/mimic/Config.in  | 39 +++++++++++++++++++++++++++++++++++++++
 package/mimic/mimic.hash |  2 ++
 package/mimic/mimic.mk   | 30 ++++++++++++++++++++++++++++++
 4 files changed, 72 insertions(+)
 create mode 100644 package/mimic/Config.in
 create mode 100644 package/mimic/mimic.hash
 create mode 100644 package/mimic/mimic.mk

Hello,

On Sat, 10 Dec 2016 09:21:54 -0300, Ezequiel Garcia wrote:
> Signed-off-by: Ezequiel Garcia <ezequiel@vanguardiasur.com.ar>

I've applied, with a number of changes, see below.


> diff --git a/package/mimic/Config.in b/package/mimic/Config.in
> new file mode 100644
> index 000000000000..b4322a986401
> --- /dev/null
> +++ b/package/mimic/Config.in
> @@ -0,0 +1,39 @@
> +config BR2_PACKAGE_MIMIC
> +	bool "mimic"
> +        help
> +          Mimic is a fast, lightweight Text-to-speech engine developed by
> +          Mycroft A.I. and VocaliD, based on Carnegie Mellon University's
> +          Flite (Festival-Lite) software. Mimic takes in text and reads it
> +          out loud to create a high quality voice.

I rewrapped the help text so that it fits in 72 character lines.

> +choice
> +	prompt "choose audio backend"

Simplified to just "audio backend"

> +	default BR2_PACKAGE_MIMIC_AUDIO_BACKEND_NONE
> +
> +config BR2_PACKAGE_MIMIC_AUDIO_BACKEND_NONE
> +	bool "No sound backend, only produce wav files"

Simplified to "none", with an added help text.

> +
> +config BR2_PACKAGE_MIMIC_AUDIO_BACKEND_ALSA
> +	bool "alsa"
> +	depends on BR2_TOOLCHAIN_HAS_THREADS # alsa-lib
> +	select BR2_PACKAGE_ALSA_LIB

Changed to a "depends on BR2_PACKAGE_ALSA_LIB"

> +
> +config BR2_PACKAGE_MIMIC_AUDIO_BACKEND_PORTAUDIO
> +	bool "alsa via portaudio"
> +	select BR2_PACKAGE_PORTAUDIO

Changed to a "depends on BR2_PACKAGE_PORTAUDIO"

> +
> +config BR2_PACKAGE_MIMIC_AUDIO_BACKEND_PULSEAUDIO
> +	bool "pulseaudio"
> +	depends on BR2_TOOLCHAIN_HAS_THREADS # pulseaudio
> +	depends on BR2_USE_MMU # pulseaudio
> +	depends on !BR2_STATIC_LIBS # pulseaudio
> +	depends on BR2_TOOLCHAIN_HAS_SYNC_4 # pulseaudio -> json-c
> +	select BR2_PACKAGE_PULSEAUDIO

Changed to a "depends on BR2_PACKAGE_PULSEAUDIO", which avoids the need
to replicate all the dependencies.

> +MIMIC_VERSION = 1.1.0
> +MIMIC_SITE = $(call github,MycroftAI,mimic,$(MIMIC_VERSION))
> +MIMIC_INSTALL_STAGING = NO

I removed this line, as it's useless: it's the default.

> +MIMIC_LICENSE = MIT
> +MIMIC_LICENSE_FILES = LICENSE
> +
> +MIMIC_DEPENDENCIES = host-pkgconf
> +
> +ifeq ($(BR2_PACKAGE_MIMIC_AUDIO_BACKEND_ALSA),y)
> +MIMIC_AUDIO_BACKEND = alsa
> +MIMIC_DEPENDENCIES = alsa-lib

Using = here is wrong: you basically ignore the host-pkgconf
dependency defined above. So I've changed this to += here, and in the
other cases below.

> +else ifeq ($(BR2_PACKAGE_MIMIC_AUDIO_BACKEND_PORTAUDIO),y)
> +MIMIC_AUDIO_BACKEND = portaudio
> +MIMIC_DEPENDENCIES = portaudio
> +else ifeq ($(BR2_PACKAGE_MIMIC_AUDIO_BACKEND_PULSEAUDIO),y)
> +MIMIC_AUDIO_BACKEND = pulseaudio
> +MIMIC_DEPENDENCIES = pulseaudio
> +else ifeq ($(BR2_PACKAGE_MIMIC_AUDIO_BACKEND_NONE),y)
> +MIMIC_AUDIO_BACKEND = none
> +endif
> +
> +MIMIC_CONF_OPTS += --with-audio="$(MIMIC_AUDIO_BACKEND)"

Double quotes unneeded, so I dropped them.

Thanks a lot!

Thomas

>>>>> "Ezequiel" == Ezequiel Garcia <ezequiel@vanguardiasur.com.ar> writes:

Hi,

 > +config BR2_PACKAGE_MIMIC
 > +	bool "mimic"
 > +        help
 > +          Mimic is a fast, lightweight Text-to-speech engine developed by
 > +          Mycroft A.I. and VocaliD, based on Carnegie Mellon University's
 > +          Flite (Festival-Lite) software. Mimic takes in text and reads it
 > +          out loud to create a high quality voice.

Heh, "lightweight" - The tarball is more than 400MB!

> +################################################################################
 > +#
 > +# mimic
 > +#
 > +################################################################################
 > +
 > +MIMIC_VERSION = 1.1.0
 > +MIMIC_SITE = $(call github,MycroftAI,mimic,$(MIMIC_VERSION))
 > +MIMIC_INSTALL_STAGING = NO
 > +MIMIC_LICENSE = MIT
 > +MIMIC_LICENSE_FILES = LICENSE
 > +
 > +MIMIC_DEPENDENCIES = host-pkgconf
 > +
 > +ifeq ($(BR2_PACKAGE_MIMIC_AUDIO_BACKEND_ALSA),y)
 > +MIMIC_AUDIO_BACKEND = alsa
 > +MIMIC_DEPENDENCIES = alsa-lib
 > +else ifeq ($(BR2_PACKAGE_MIMIC_AUDIO_BACKEND_PORTAUDIO),y)
 > +MIMIC_AUDIO_BACKEND = portaudio
 > +MIMIC_DEPENDENCIES = portaudio
 > +else ifeq ($(BR2_PACKAGE_MIMIC_AUDIO_BACKEND_PULSEAUDIO),y)
 > +MIMIC_AUDIO_BACKEND = pulseaudio
 > +MIMIC_DEPENDENCIES = pulseaudio

configure.ac uses PKG_CHECK_MODULES to detect these, so we should depend
on host-pkgconf - I've fixed that in git.

>>>>> "Peter" == Peter Korsgaard <peter@korsgaard.com> writes:

Hi,

 >> +MIMIC_DEPENDENCIES = host-pkgconf
 >> +
 >> +ifeq ($(BR2_PACKAGE_MIMIC_AUDIO_BACKEND_ALSA),y)
 >> +MIMIC_AUDIO_BACKEND = alsa
 >> +MIMIC_DEPENDENCIES = alsa-lib
 >> +else ifeq ($(BR2_PACKAGE_MIMIC_AUDIO_BACKEND_PORTAUDIO),y)
 >> +MIMIC_AUDIO_BACKEND = portaudio
 >> +MIMIC_DEPENDENCIES = portaudio
 >> +else ifeq ($(BR2_PACKAGE_MIMIC_AUDIO_BACKEND_PULSEAUDIO),y)
 >> +MIMIC_AUDIO_BACKEND = pulseaudio
 >> +MIMIC_DEPENDENCIES = pulseaudio

 > configure.ac uses PKG_CHECK_MODULES to detect these, so we should depend
 > on host-pkgconf - I've fixed that in git.

Ehh, must have been -ENOCOFFEE. You indeed already have host-pkgconf,
please ignore.