[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 13/13] gnu: Add beets.
From: |
Leo Famulari |
Subject: |
Re: [PATCH 13/13] gnu: Add beets. |
Date: |
Sun, 27 Mar 2016 18:25:26 -0400 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Sun, Mar 27, 2016 at 05:36:19PM -0400, Leo Famulari wrote:
> On Sat, Mar 26, 2016 at 09:58:32AM -0500, Alex Griffin wrote:
> > I tried building with --rounds=4 on my GuixSD machine, but it errored
> > out after the first build with the same error. Then I tried building
> > with --check on my Debian machine and it succeeded.
> >
> > Last night on IRC we were discussing this more and I think Leo was going
> > to try incorporating something from the nixos beets package, or else
> > wait until the beets developers could help him.
>
> On IRC, (the user who I think is) Alex said that they were able to build
> beets with the attached patch.
>
> All the patch does is invoke the tests with `nosetests -v` instead of
> `python setup.py test`, making use of python2-nose. This is what is done
> by Nixpkgs.
>
> Is there anybody else who could not build beets before? Would you like
> to test the patch?
And now I see that the beets wiki says that the tests are to be run with
nose:
https://github.com/beetbox/beets/wiki/Testing
So, I applied the patch as 36c2c8c82.
>
> > --
> > Alex Griffin
> >
> >
> > On Sat, Mar 26, 2016, at 06:41 AM, Ludovic Courtès wrote:
> > > Leo Famulari <address@hidden> skribis:
> > >
> > > > Can you try building with '#:parallel-tests? #f', as shown in this link?
> > >
> > > I was also going to suggest something along these lines.
> > >
> > > Could you try building with with --rounds=4, say, on your machine? (Or
> > > --check if it’s already there.)
> > >
> > > Thanks,
> > > Ludo’.
> From ba2cb8403ab9340ab8a6da534e9db4cc3f2fbb84 Mon Sep 17 00:00:00 2001
> From: Leo Famulari <address@hidden>
> Date: Sat, 26 Mar 2016 17:53:59 -0400
> Subject: [PATCH] gnu: beets: Run tests with python-nose.
>
> * gnu/packages/music.scm (nose)[native-inputs]: Add python-nose.
> [arguments]: Replace 'check' and use python-nose.
> ---
> gnu/packages/music.scm | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/gnu/packages/music.scm b/gnu/packages/music.scm
> index 89f49c6..48d6214 100644
> --- a/gnu/packages/music.scm
> +++ b/gnu/packages/music.scm
> @@ -1274,13 +1274,16 @@ websites such as Libre.fm.")
> #:phases
> (modify-phases %standard-phases
> (add-after 'unpack 'set-HOME
> - (lambda _ (setenv "HOME" (string-append (getcwd) "/tmp")))))))
> + (lambda _ (setenv "HOME" (string-append (getcwd) "/tmp"))))
> + (replace 'check
> + (lambda _ (zero? (system* "nosetests" "-v")))))))
> (native-inputs
> `(("python2-beautifulsoup4" ,python2-beautifulsoup4)
> ("python2-flask" ,python2-flask)
> ("python2-setuptools" ,python2-setuptools)
> ("python2-mock" ,python2-mock)
> ("python2-mpd2" ,python2-mpd2)
> + ("python2-nose" ,python2-nose)
> ("python2-pathlib" ,python2-pathlib)
> ("python2-pyxdg" ,python2-pyxdg)
> ("python2-pyechonest" ,python2-pyechonest)
> --
> 2.7.3
>
- Re: [PATCH 13/13] gnu: Add beets., (continued)
- Re: [PATCH 13/13] gnu: Add beets., Leo Famulari, 2016/03/18
- Re: [PATCH 13/13] gnu: Add beets., Ludovic Courtès, 2016/03/18
- Re: [PATCH 13/13] gnu: Add beets., Mark H Weaver, 2016/03/25
- Re: [PATCH 13/13] gnu: Add beets., Leo Famulari, 2016/03/25
- Re: [PATCH 13/13] gnu: Add beets., Alex Griffin, 2016/03/25
- Re: [PATCH 13/13] gnu: Add beets., Leo Famulari, 2016/03/25
- Re: [PATCH 13/13] gnu: Add beets., Leo Famulari, 2016/03/25
- Re: [PATCH 13/13] gnu: Add beets., Ludovic Courtès, 2016/03/26
- Re: [PATCH 13/13] gnu: Add beets., Alex Griffin, 2016/03/26
- Re: [PATCH 13/13] gnu: Add beets., Leo Famulari, 2016/03/27
- Re: [PATCH 13/13] gnu: Add beets.,
Leo Famulari <=
- Re: [PATCH 13/13] gnu: Add beets., Ludovic Courtès, 2016/03/28
[PATCH 04/13] gnu: Add python-flask., Leo Famulari, 2016/03/03
[PATCH 10/13] gnu: Add python-jellyfish., Leo Famulari, 2016/03/03
[PATCH 01/13] gnu: Add python-mutagen., Leo Famulari, 2016/03/03