[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] gnu: spice: Fix usbredir for 32 bit platforms.
From: |
Mark H Weaver |
Subject: |
Re: [PATCH] gnu: spice: Fix usbredir for 32 bit platforms. |
Date: |
Sat, 13 Aug 2016 15:50:17 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux) |
Hi David,
David Craven <address@hidden> writes:
> * gnu/packages/spice.scm (usbredir)[origin]: Fetch source from git repo.
> [native-inputs]: Add AUTOCONF, AUTOMAKE and LIBTOOL.
> [arguments]: Add autogen phase.
I see you already pushed this, but there are some problems.
See below.
> @@ -45,19 +47,39 @@
> (package
> (name "usbredir")
> (version "0.7.1")
> + ;(source (origin
> + ; (method url-fetch)
> + ; (uri (string-append
> + ; "http://spice-space.org/download/usbredir/"
> + ; "usbredir-" version ".tar.bz2"))
> + ; (sha256
> + ; (base32
> + ; "1wsnmk4wjpdhbn1zaxg6bmyxspcki2zgy0am9lk037rnl4krwzj0"))))
> + ; FIXME: usbredir 0.7.1 release doesn't build on 32 bit systems.
> + ; issue is fixed in HEAD
> + ; remove 'autogen phase and autoconf, automake, libtool inputs
> (source (origin
> - (method url-fetch)
> - (uri (string-append
> - "http://spice-space.org/download/usbredir/"
> - "usbredir-" version ".tar.bz2"))
> + (method git-fetch)
> + (uri (git-reference
> + (url "http://cgit.freedesktop.org/spice/usbredir")
> + (commit "ac80a5971c6318d73d5fba4b5f13d3a9389558c9")))
> (sha256
> (base32
> - "1wsnmk4wjpdhbn1zaxg6bmyxspcki2zgy0am9lk037rnl4krwzj0"))))
> + "052fywgi72j68dr5ybldncg4vk8iqfrh58la7iazyxxpph9aag1g"))))
This is no longer version "0.7.1", so the version number needs to be
updated accordingly. Please see section 7.6.3 (Version Numbers) in the
manual for our conventions for version numbers of VCS snapshots, and the
recommended code to generate those version numbers.
> (build-system gnu-build-system)
> (propagated-inputs
> `(("libusb" ,libusb)))
> (native-inputs
> - `(("pkg-config" ,pkg-config)))
> + `(("pkg-config" ,pkg-config)
> + ("autoconf" ,autoconf)
> + ("automake" ,automake)
> + ("libtool" ,libtool)))
> + (arguments
> + `(#:phases
> + (modify-phases %standard-phases
> + (add-after 'unpack 'autogen
> + (lambda* _
> + (system* "sh" "autogen.sh"))))))
Phase procedures are supposed to return a boolean indicating whether
they succeeded, but 'system*' returns a number: a result code. In
scheme, all numbers are considered true. Also, you might as well use
'lambda' here instead of 'lambda*', so it should look like this:
(lambda _
(zero? (system* "sh" "autogen.sh")))
It might have been better to just use a simple patch to fix the format
strings than to use a VCS snapshot that might introduce more bugs, but
I guess we can see how it goes.
Anyway, can you push fixes for the other problems?
Mark
- [PATCH 1/5] doc: Make 'Lirc Service' a subsubheading of 'Various Services'., David Craven, 2016/08/02
- [PATCH 3/5] gnu: qemu: Enable spice support., David Craven, 2016/08/02
- Re: [PATCH 3/5] gnu: qemu: Enable spice support., Leo Famulari, 2016/08/05
- Re: [PATCH 3/5] gnu: qemu: Enable spice support., Mark H Weaver, 2016/08/05
- Re: [PATCH 3/5] gnu: qemu: Enable spice support., David Craven, 2016/08/06
- Re: [PATCH 3/5] gnu: qemu: Enable spice support., David Craven, 2016/08/06
- Re: [PATCH 3/5] gnu: qemu: Enable spice support., Mark H Weaver, 2016/08/06
- [PATCH] gnu: spice: Fix usbredir for 32 bit platforms., David Craven, 2016/08/08
- Re: [PATCH] gnu: spice: Fix usbredir for 32 bit platforms., David Craven, 2016/08/08
- Re: [PATCH] gnu: spice: Fix usbredir for 32 bit platforms.,
Mark H Weaver <=
- Re: [PATCH] gnu: spice: Fix usbredir for 32 bit platforms., David Craven, 2016/08/14
- Re: [PATCH] gnu: spice: Fix usbredir for 32 bit platforms., Mark H Weaver, 2016/08/14
[PATCH 4/5] gnu: spice-vdagent: Set Exec path in spice-vdagent.desktop., David Craven, 2016/08/02
[PATCH 2/5] gnu: qemu: Reorder inputs alphabetically., David Craven, 2016/08/02
[PATCH 5/5] services: Add spice vdagent service., David Craven, 2016/08/02