all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Efraim Flashner <efraim@flashner.co.il>
To: Nicolas Goaziou <mail@nicolasgoaziou.fr>
Cc: 50663@debbugs.gnu.org, phodina@protonmail.com
Subject: [bug#50663] Add Mycroft - Voice Assistant
Date: Tue, 25 Jan 2022 12:49:33 +0200	[thread overview]
Message-ID: <Ye/VvVG2FWBFUE8a@3900XT> (raw)
In-Reply-To: <87r18wbvnq.fsf@nicolasgoaziou.fr>


[-- Attachment #1.1: Type: text/plain, Size: 1138 bytes --]

On Mon, Jan 24, 2022 at 11:45:13PM +0100, Nicolas Goaziou wrote:
> Hello,
> 
> phodina via Guix-patches via <guix-patches@gnu.org> writes:
> 
> > The aim of this patch set is to bring in the MyCroft - private and
> > open Voice Assistant.
> 
> Thank you.
> 
> I fixed some descriptions, removed labels from inputs, fixed compilation
> with GCC 10 (for Mimic). Then I tried to apply the patch set. Alas
> Mycroft core fails to build. You may want to have a look at it.
> 
> I'm sending back the updated patches to you, if that helps.
> 
> Regards,
> -- 
> Nicolas Goaziou

I made some changes to python-mycroft-core but I couldn't get it to pass
the sanity-check or the test suite. I've attached my changes. Also make
sure you sort the inputs alphabetically, I found at least a duplicate
python-pyxdg. Also some of the patches include a line or two of
unrelated indentation changes in other packages.

-- 
Efraim Flashner   <efraim@flashner.co.il>   רנשלפ םירפא
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

[-- Attachment #1.2: python-mycroft-core-changes --]
[-- Type: text/plain, Size: 4054 bytes --]

diff --git a/gnu/packages/python-xyz.scm b/gnu/packages/python-xyz.scm
index 1834152c3c..b9c4529cd5 100644
--- a/gnu/packages/python-xyz.scm
+++ b/gnu/packages/python-xyz.scm
@@ -21986,63 +21986,61 @@ (define-public python-mycroft-core
          "02r0vxw0hsihnvviwn4fyspwky3kwq42f9z455q1s70k0snzhb28"))))
     (build-system python-build-system)
     (arguments
-     `(#:phases
+     `(#:tests? #f      ; TODO
+       #:phases
        (modify-phases %standard-phases
          (add-after 'unpack 'patch-requirements
            (lambda* (#:key inputs #:allow-other-keys)
-             ;; Use newer version of pkg
+             ;; Use newer version of packages.
+             (setenv "MYCROFT_LOOSE_REQUIREMENTS" "1")
              (substitute* "requirements/requirements.txt"
-               (("mycroft-messagebus-client==") "mycroft-messagebus-client>=")
-               (("tornado==") "tornado>=")
-               (("pyserial==") "pyserial>=")
-               (("psutil==") "psutil>=")
-               (("padaos==") "padaos>=")
-               (("precise-runner==") "precise-runner>=")
-               (("pocketsphinx==") "pocketsphinx>=")
-               (("python-dateutil==") "python-dateutil>=")
-               (("fasteners==") "fasteners>=")
-               (("requests-futures==") "requests-futures>=")
-               (("pillow==") "pillow>=")
-               (("PyYAML==5.4") "PyYAML>=5.3.1")
-               (("pyxdg==") "pyxdg>=")
                (("requests>=2.20.0,<2.26.0") "requests>=2.20.0"))))
+         (replace 'sanity-check
+           (lambda args
+             ;(setenv "PYTHONPATH"
+             ;        (string-append "./build/lib:"
+             ;                       (or (getenv "PYTHONPATH")
+             ;                           "")))
+             (apply (assoc-ref %standard-phases 'sanity-check) args)))
          (replace 'check
-           (lambda* (#:key tests? #:allow-other-keys)
+           (lambda* (#:key tests? #:allow-other-keys #:rest args)
              (when tests?
                (setenv "PYTHONPATH"
                        (string-append "./build/lib:"
                                       (or (getenv "PYTHONPATH")
-                                          "")))))))))
+                                          "")))
+               (apply (assoc-ref %standard-phases 'check) args)))))))
     (inputs (list mycroft-mimic pocketsphinx))
     (propagated-inputs
-     (list python-fasteners
+     (list python-adapt-parser
+           python-dateutil
+           python-fann2
+           python-fasteners
+           python-gtts
            python-inflection
-           python-pyxdg
+           python-lingua-franca
+           python-msk
+           python-msm
            python-mycroft-messagebus-client
-           python-psutil
-           python-tornado-6
-           python-petact
-           python-precise-runner
            python-padaos
-           python-speech-recognition
            python-padatious
-           python-msk
+           python-petact
            python-pillow
-           python-gtts
-           python-requests-futures
-           python-pyserial
            python-pocketsphinx
-           python-adapt-parser
-           python-lingua-franca
-           python-pyyaml
-           python-fasteners
+           python-psutil
+           python-pyaudio
            python-pyee
-           python-psutil
-           python-fann2
            python-pyxdg
-           python-websocket-client
+           python-pyyaml
+           python-pyserial
+           python-precise-runner
            python-requests
-           python-requests-futures))
+           python-requests-futures
+           python-speech-recognition
+           python-tornado-6
+           python-websocket-client))
+    (native-inputs
+     (list python-pytest))
     (home-page "https://github.com/HelloChatterbox/HolmesIV")
     (synopsis "Mycroft Core, the Mycroft Artificial Intelligence platform")
     (description "This module provides a hackable open source voice assistant - Mycroft")

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

  reply	other threads:[~2022-01-25 10:57 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-09-18 17:01 [bug#50663] [PATCH 01/18] gnu: Add python-pyee phodina via Guix-patches via
2021-09-18 17:04 ` [bug#50663] [PATCH 02/18] gnu: Add python-fann2 phodina via Guix-patches via
2021-09-18 18:58 ` [bug#50663] [PATCH 03/19] gnu: Add python-padaos phodina via Guix-patches via
2021-09-18 18:59 ` [bug#50663] [PATCH 04/19] gnu: Add python-precise-runner phodina via Guix-patches via
2021-09-18 18:59 ` [bug#50663] [PATCH 05/19] gnu: Add python-petact phodina via Guix-patches via
2021-09-18 19:00 ` [bug#50663] [PATCH 06/19] gnu: Add python-padatious phodina via Guix-patches via
2021-09-18 19:01 ` [bug#50663] [PATCH 07/19] gnu: Add python-lazy phodina via Guix-patches via
2021-09-18 19:01 ` [bug#50663] [PATCH 08/19] gnu: Add python-pako phodina via Guix-patches via
2021-09-18 19:02 ` [bug#50663] [PATCH 09/19] gnu: Add python-adapt-parser phodina via Guix-patches via
2021-09-18 19:03 ` [bug#50663] [PATCH 10/19] gnu: Add python-msm phodina via Guix-patches via
2021-09-18 19:03 ` [bug#50663] [PATCH 11/19] gnu: Add python-msk phodina via Guix-patches via
2021-09-18 19:03 ` [bug#50663] [PATCH 12/19] gnu: Add python-lingua-franca phodina via Guix-patches via
2021-09-18 19:04 ` [bug#50663] [PATCH 13/19] gnu: Add python-pocketsphinx phodina via Guix-patches via
2021-09-18 19:04 ` [bug#50663] [PATCH 14/19] gnu: Add python-gtts phodina via Guix-patches via
2021-09-18 19:11 ` [bug#50663] [PATCH 16/19] gnu: Add python-mycroft-messagebus-client phodina via Guix-patches via
2021-09-18 19:11 ` [bug#50663] [PATCH 17/19] gnu: Add mycroft-mimic phodina via Guix-patches via
2021-09-18 19:12 ` [bug#50663] [PATCH 18/19] gnu: Add python-speech-recognition phodina via Guix-patches via
2021-09-18 19:12 ` [bug#50663] [PATCH 19/19] gnu: Add python-mycroft-core phodina via Guix-patches via
2021-09-18 19:16 ` [bug#50663] [PATCH 02/19] gnu: Add python-pyee phodina via Guix-patches via
2021-12-08 11:48 ` [bug#50663] Add Mycroft - Voice Assistant phodina via Guix-patches via
2022-01-24 22:45   ` Nicolas Goaziou
2022-01-25 10:49     ` Efraim Flashner [this message]
2022-01-26 12:53       ` phodina via Guix-patches via
2022-02-15 14:37 ` [bug#50663] [PATCH v3 1/2] gnu: Add python-xdg phodina via Guix-patches via

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=Ye/VvVG2FWBFUE8a@3900XT \
    --to=efraim@flashner.co.il \
    --cc=50663@debbugs.gnu.org \
    --cc=mail@nicolasgoaziou.fr \
    --cc=phodina@protonmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.