Re: [master/thud][PATCH 1/2] kaldi: add the kaldi speech recognition toolkit

[Denys Dmytriyenko <denys@ti.com>]

On Tue, Jul 16, 2019 at 09:16:31AM -0400, Jacob Stiffler wrote:
> 
> On 7/15/2019 4:05 PM, Denys Dmytriyenko wrote:
> >On Mon, Jul 15, 2019 at 03:53:13PM -0400, Jacob Stiffler wrote:
> >>On 7/15/2019 3:44 PM, Denys Dmytriyenko wrote:
> >>>On Mon, Jul 15, 2019 at 02:59:36PM -0400, Jacob Stiffler wrote:
> >>>>From: Yimin Zhang <a0216607@ti.com>
> >>>>
> >>>>Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
> >>>>---
> >>>>  .../recipes-support/kaldi/kaldi_git.bb             | 47 ++++++++++++++++++++++
> >>>>  1 file changed, 47 insertions(+)
> >>>>  create mode 100644 meta-arago-extras/recipes-support/kaldi/kaldi_git.bb
> >>>>
> >>>>diff --git a/meta-arago-extras/recipes-support/kaldi/kaldi_git.bb b/meta-arago-extras/recipes-support/kaldi/kaldi_git.bb
> >>>>new file mode 100644
> >>>>index 0000000..6937696
> >>>>--- /dev/null
> >>>>+++ b/meta-arago-extras/recipes-support/kaldi/kaldi_git.bb
> >>>>@@ -0,0 +1,47 @@
> >>>>+SUMMARY = "TI KALDI speech recognition toolkit"
> >>>>+HOMEPAGE = "http://kaldi-asr.org"
> >>>>+LICENSE = "Apache-2.0"
> >>>>+LIC_FILES_CHKSUM = "file://COPYING;md5=35439d742a3df991e4f4f0e68e2d634c"
> >>>>+
> >>>>+PR = "r0"
> >>>>+COMPATIBLE_MACHINE = "dra7xx"
> >>>>+
> >>>>+require recipes-ti/includes/arago-paths.inc
> >>>>+
> >>>>+SRC_URI = "git://git.ti.com/processor-sdk/kaldi.git;protocol=git;branch=${BRANCH}"
> >>>>+BRANCH = "kaldi_1-0-0"
> >>>>+SRCREV = "a472be2e2f8598a3e360bc7ca87ff4f73afa2d9e"
> >>>>+
> >>>>+DEPENDS = " \
> >>>>+	linalg \
> >>>>+	openfst \
> >>>>+"
> >>>>+
> >>>>+S = "${WORKDIR}/git"
> >>>>+
> >>>>+export LINALG_DIR  = "${LINALG_INSTALL_DIR}"
> >>>>+export OPENFST_DIR = "${STAGING_DIR_TARGET}${prefix}"
> >>>>+export TARGET_ROOTDIR="${STAGING_DIR_TARGET}"
> >>>>+
> >>>>+PACKAGE_ARCH = "${MACHINE_ARCH}"
> >>>>+EXTRA_OECONF += "--static --use-cuda=no --mathlib=CLAPACK --clapack-root=${LINALG_INSTALL_DIR}/packages/ti/linalg --fst-root=${STAGING_DIR_TARGET}${prefix} --fst-version=1.6.2 --ti-cross-compile --ti-device=AM57"
> >>>>+
> >>>>+EXTRA_OEMAKE = "-C src"
> >>>>+
> >>>>+do_configure() {
> >>>>+	cd src
> >>>>+	chmod +x configure
> >>>>+	./configure ${EXTRA_OECONF}
> >>>Any specific reason to not inherit autotools and/or calling oe_runconf?
> >>
> >>I was seeing some problem where is was trying to invoke "../configure"
> >>instead of "./configure".  I never completely understood what was causing
> >>this, so I can look into it more if you'd like.
> >Probably because configure is not in ${S}.
> 
> 
> I made progress to get configure invoked using oe_runconf, but it appears
> that this configure is a bit non-standard. The options provided by
> CONFIGUREOPTS (e.g. --build, --host, --target) are not recognized, and cause
> an error. The options "--ti-cross-compile --ti-device=AM57" appear to have
> been implemented to enable cross-compilation.
> 
> So is it better to unset CONFIGUREOPTS and use oe_runconf, or directly
> invoke configure?

I see. Since it's a custom-made configure script with pretty much non-standard 
options, it's probably better to not use autoconf.bbclass, CONFIGUREOPTS and 
maybe even EXTRA_OECONF to avoid confusion. Thoughts?


> >>>>+}
> >>>>+
> >>>>+do_compile() {
> >>>>+	oe_runmake online2bin featbin
> >>>>+}
> >>>>+
> >>>>+do_install() {
> >>>>+	chmod +x export_kaldi.sh
> >>>>+	./export_kaldi.sh --nnet2-online . ${D}${datadir}/ti/examples/kaldi
> >>>>+}
> >>>>+
> >>>>+# to create a package for KALDI
> >>>>+FILES_${PN} += "${datadir}/ti/examples/kaldi"
> >>>>-- 
> >>>>2.7.4
> >>>>
> >>>>_______________________________________________
> >>>>meta-arago mailing list
> >>>>meta-arago@arago-project.org
> >>>>http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago