Package: guix-patches;
Reported by: Leo Famulari <leo <at> famulari.name>
Date: Sat, 21 Mar 2020 22:22:02 UTC
Severity: normal
Done: Christopher Baines <mail <at> cbaines.net>
Bug is archived. No further changes may be made.
To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 40178 in the body.
You can then email your comments to 40178 AT debbugs.gnu.org in the normal way.
Toggle the display of automated, internal messages from the tracker.
View this report as an mbox folder, status mbox, maintainer mbox
guix-patches <at> gnu.org:bug#40178; Package guix-patches.
(Sat, 21 Mar 2020 22:22:02 GMT) Full text and rfc822 format available.Leo Famulari <leo <at> famulari.name>:guix-patches <at> gnu.org.
(Sat, 21 Mar 2020 22:22:02 GMT) Full text and rfc822 format available.Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):
From: Leo Famulari <leo <at> famulari.name> To: guix-patches <at> gnu.org Subject: QEMU for american fuzzy lop Date: Sat, 21 Mar 2020 18:21:04 -0400
These patches remove the qemu-minimal-2.10 package without breaking the american-fuzzy-lop package.
guix-patches <at> gnu.org:bug#40178; Package guix-patches.
(Sat, 21 Mar 2020 22:25:01 GMT) Full text and rfc822 format available.Message #8 received at 40178 <at> debbugs.gnu.org (full text, mbox):
From: Leo Famulari <leo <at> famulari.name> To: 40178 <at> debbugs.gnu.org Subject: [PATCH 1/2] gnu: Prefix licenses in (gnu packages debug). Date: Sat, 21 Mar 2020 18:24:16 -0400
* gnu/packages/debug.scm: Add a license prefix.
---
gnu/packages/debug.scm | 18 +++++++++---------
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/gnu/packages/debug.scm b/gnu/packages/debug.scm
index 718a76310d..1de8ba3677 100644
--- a/gnu/packages/debug.scm
+++ b/gnu/packages/debug.scm
@@ -22,7 +22,7 @@
(define-module (gnu packages debug)
#:use-module (guix packages)
- #:use-module (guix licenses)
+ #:use-module ((guix licenses) #:prefix license:)
#:use-module (guix download)
#:use-module (guix git-download)
#:use-module (guix utils)
@@ -96,7 +96,7 @@ isolate a small failure-inducing substring of a large input that causes your
program to exhibit a bug.")
;; See License.txt, which is a bsd-3 license, despite the project's
;; home-page pointing to a bsd-2 license.
- (license bsd-3)))
+ (license license:bsd-3)))
(define-public c-reduce
(package
@@ -156,7 +156,7 @@ property of interest (such as triggering a compiler bug) and automatically
produces a much smaller C/C++ program that has the same property. It is
intended for use by people who discover and report bugs in compilers and other
tools that process C/C++ code.")
- (license ncsa)))
+ (license license:ncsa)))
(define-public american-fuzzy-lop
(let ((machine (match (or (%current-target-system)
@@ -283,7 +283,7 @@ targeted binary. This substantially improves the functional coverage for the
fuzzed code. The compact synthesized corpora produced by the tool are also
useful for seeding other, more labor- or resource-intensive testing regimes
down the road.")
- (license asl2.0))))
+ (license license:asl2.0))))
(define-public stress-make
(let ((commit "9e92dff8f0157f012aaf31de5b8b8112ad720100")
@@ -346,8 +346,8 @@ Stress Make, then it is likely that the @code{Makefile} contains no race
conditions.")
;; stress-make wrapper is under BSD-3-modifications-must-be-indicated,
;; and patched GNU Make is under its own license.
- (license (list (non-copyleft "LICENSE.md")
- gpl3+)))))
+ (license (list (license:non-copyleft "LICENSE.md")
+ license:gpl3+)))))
(define-public zzuf
(package
@@ -369,7 +369,7 @@ conditions.")
(description "Zzuf is a transparent application input fuzzer. It works by
intercepting file operations and changing random bits in the program's
input. Zzuf's behaviour is deterministic, making it easy to reproduce bugs.")
- (license (non-copyleft "http://www.wtfpl.net/txt/copying/"))))
+ (license (license:non-copyleft "http://www.wtfpl.net/txt/copying/"))))
(define-public scanmem
(package
@@ -427,7 +427,7 @@ several different times. After several scans of the process, scanmem isolates
the position of the variable and allows you to modify its value.")
;; The library is covered by LGPLv3 or later; the application is covered
;; by GPLv3 or later.
- (license (list lgpl3+ gpl3+))))
+ (license (list license:lgpl3+ license:gpl3+))))
(define-public rr
(package
@@ -490,4 +490,4 @@ execution of applications (trees of processes and threads). Debugging extends
GDB with very efficient reverse-execution, which in combination with standard
GDB/x86 features like hardware data watchpoints, makes debugging much more
fun.")
- (license expat)))
+ (license license:expat)))
--
2.25.1
guix-patches <at> gnu.org:bug#40178; Package guix-patches.
(Sat, 21 Mar 2020 22:25:01 GMT) Full text and rfc822 format available.Message #11 received at 40178 <at> debbugs.gnu.org (full text, mbox):
From: Leo Famulari <leo <at> famulari.name> To: 40178 <at> debbugs.gnu.org Subject: [PATCH 2/2] gnu: Remove qemu-minimal-2.10. Date: Sat, 21 Mar 2020 18:24:17 -0400
* gnu/packages/virtualization.scm (qemu-minimal-2.10): Remove variable.
* gnu/packages/debug.scm (qemu-for-american-fuzzy-lop): New variable.
(american-fuzzy-lop)[inputs]: Remove custom-qemu and add
qemu-for-american-fuzzy-lop.
[arguments]: Adjust reference to QEMU in the 'install-qemu' phase.
---
gnu/packages/debug.scm | 246 ++++++++++++++++++++++++--------
gnu/packages/virtualization.scm | 28 ----
2 files changed, 190 insertions(+), 84 deletions(-)
diff --git a/gnu/packages/debug.scm b/gnu/packages/debug.scm
index 1de8ba3677..f389b770df 100644
--- a/gnu/packages/debug.scm
+++ b/gnu/packages/debug.scm
@@ -28,16 +28,23 @@
#:use-module (guix utils)
#:use-module (guix build-system gnu)
#:use-module (guix build-system cmake)
+ #:use-module (gnu packages)
+ #:use-module (gnu packages attr)
#:use-module (gnu packages autotools)
#:use-module (gnu packages base)
#:use-module (gnu packages bash)
+ #:use-module (gnu packages bison)
#:use-module (gnu packages code)
+ #:use-module (gnu packages compression)
#:use-module (gnu packages flex)
#:use-module (gnu packages gdb)
#:use-module (gnu packages glib)
#:use-module (gnu packages gtk)
#:use-module (gnu packages golang)
+ #:use-module (gnu packages image)
+ #:use-module (gnu packages linux)
#:use-module (gnu packages llvm)
+ #:use-module (gnu packages ncurses)
#:use-module (gnu packages ninja)
#:use-module (gnu packages perl)
#:use-module (gnu packages pkg-config)
@@ -46,7 +53,9 @@
#:use-module (gnu packages python-xyz)
#:use-module (gnu packages readline)
#:use-module (gnu packages serialization)
+ #:use-module (gnu packages texinfo)
#:use-module (gnu packages virtualization)
+ #:use-module (gnu packages xdisorg)
#:use-module (ice-9 match)
#:use-module (srfi srfi-1))
@@ -183,61 +192,7 @@ tools that process C/C++ code.")
(file-name (git-file-name name version))))
(build-system gnu-build-system)
(inputs
- `(("custom-qemu"
- ;; The afl-qemu tool builds qemu 2.10.0 with a few patches applied.
- ,(package (inherit qemu-minimal-2.10)
- (name "afl-qemu")
- (inputs
- `(("afl-src" ,source)
- ,@(package-inputs qemu-minimal)))
- ;; afl only supports using a single afl-qemu-trace executable, so
- ;; we only build qemu for the native target.
- (arguments
- `(#:modules ((srfi srfi-1)
- ,@%gnu-build-system-modules)
- ,@(substitute-keyword-arguments (package-arguments qemu-minimal)
- ((#:configure-flags config-flags)
- ``(,(string-append "--target-list=" ,machine "-linux-user")
- ,@(remove (λ (f) (string-prefix? "--target-list=" f))
- ,config-flags)))
- ((#:phases qemu-phases)
- `(modify-phases ,qemu-phases
- (add-after
- 'unpack 'apply-afl-patches
- (lambda* (#:key inputs #:allow-other-keys)
- (let* ((afl-src (assoc-ref inputs "afl-src"))
- (patch-dir "qemu_mode/patches"))
- (copy-recursively (string-append afl-src "/"
- patch-dir)
- patch-dir)
- (install-file
- (string-append patch-dir
- "/afl-qemu-cpu-inl.h")
- ".")
- (copy-file (string-append afl-src "/config.h")
- "./afl-config.h")
- (install-file (string-append afl-src "/types.h")
- ".")
- (substitute* "afl-qemu-cpu-inl.h"
- (("\\.\\./\\.\\./config.h") "afl-config.h"))
- (substitute* (string-append patch-dir
- "/cpu-exec.diff")
- (("\\.\\./patches/") ""))
-
- ;; These were already applied to qemu-minimal-2.10.
- (for-each (lambda (obsolete-patch)
- (delete-file (string-append
- patch-dir "/"
- obsolete-patch)))
- (list "configure.diff"
- "memfd.diff"))
-
- (for-each (lambda (patch-file)
- (invoke "patch" "--force" "-p1"
- "--input" patch-file))
- (find-files patch-dir
- "\\.diff$"))
- #t))))))))))))
+ `(("qemu" ,qemu-for-american-fuzzy-lop)))
(arguments
`(#:make-flags (list (string-append "PREFIX=" (assoc-ref %outputs "out"))
(string-append "DOC_PATH=$(PREFIX)/share/doc/"
@@ -267,7 +222,7 @@ tools that process C/C++ code.")
;; TODO: Build and install the afl-llvm tool.
'install 'install-qemu
(lambda* (#:key inputs outputs #:allow-other-keys)
- (let ((qemu (assoc-ref inputs "custom-qemu"))
+ (let ((qemu (assoc-ref inputs "qemu"))
(out (assoc-ref outputs "out")))
(symlink (string-append qemu "/bin/qemu-" ,machine)
(string-append out "/bin/afl-qemu-trace"))
@@ -285,6 +240,185 @@ useful for seeding other, more labor- or resource-intensive testing regimes
down the road.")
(license license:asl2.0))))
+(define-public qemu-for-american-fuzzy-lop
+ (let ((machine (match (or (%current-target-system)
+ (%current-system))
+ ("x86_64-linux" "x86_64")
+ ("i686-linux" "i386")
+ ("aarch64-linux" "aarch64")
+ ("armhf-linux" "arm")
+ ("mips64el-linux" "mips64el")
+ ;; Prevent errors when querying this package on unsupported
+ ;; platforms, e.g. when running "guix package --search="
+ (_ "UNSUPPORTED"))))
+ (hidden-package
+ (package
+ (name "qemu")
+ (version "2.10.2")
+ (source (origin
+ (method url-fetch)
+ (uri (string-append "https://download.qemu.org/qemu-"
+ version ".tar.xz"))
+ (sha256
+ (base32
+ "17w21spvaxaidi2am5lpsln8yjpyp2zi3s3gc6nsxj5arlgamzgw"))
+ (patches
+ (search-patches "qemu-glibc-2.27.patch"))))
+ (build-system gnu-build-system)
+ (arguments
+ `(;; Running tests in parallel can occasionally lead to failures, like:
+ ;; boot_sector_test: assertion failed (signature == SIGNATURE): (0x00000000 == 0x0000dead)
+ #:parallel-tests? #f
+ #:configure-flags
+ (list (string-append "--target-list=" ,machine "-linux-user"))
+ #:make-flags '("V=1")
+ #:phases
+ (modify-phases %standard-phases
+ (replace 'configure
+ (lambda* (#:key inputs outputs (configure-flags '())
+ #:allow-other-keys)
+ ;; The `configure' script doesn't understand some of the
+ ;; GNU options. Thus, add a new phase that's compatible.
+ (let ((out (assoc-ref outputs "out")))
+ (setenv "SHELL" (which "bash"))
+
+ ;; While we're at it, patch for tests.
+ (substitute* "tests/libqtest.c"
+ (("/bin/sh") (which "sh")))
+
+ ;; The binaries need to be linked against -lrt.
+ (setenv "LDFLAGS" "-lrt")
+ (apply invoke
+ `("./configure"
+ ,(string-append "--cc=" (which "gcc"))
+ ;; Some architectures insist on using HOST_CC
+ ,(string-append "--host-cc=" (which "gcc"))
+ "--disable-debug-info" ; save build space
+ "--enable-virtfs" ; just to be sure
+ ,(string-append "--prefix=" out)
+ ,(string-append "--sysconfdir=/etc")
+ ,@configure-flags)))))
+ (add-after
+ 'unpack 'apply-afl-patches
+ (lambda* (#:key inputs #:allow-other-keys)
+ (let* ((afl-src (assoc-ref inputs "afl-source"))
+ (patch-dir "qemu_mode/patches"))
+ (copy-recursively (string-append afl-src "/"
+ patch-dir)
+ patch-dir)
+ (install-file
+ (string-append patch-dir
+ "/afl-qemu-cpu-inl.h")
+ ".")
+ (copy-file (string-append afl-src "/config.h")
+ "./afl-config.h")
+ (install-file (string-append afl-src "/types.h")
+ ".")
+ (substitute* "afl-qemu-cpu-inl.h"
+ (("\\.\\./\\.\\./config.h") "afl-config.h"))
+ (substitute* (string-append patch-dir
+ "/cpu-exec.diff")
+ (("\\.\\./patches/") ""))
+
+ ;; These were already applied to qemu-minimal-2.10.
+ (for-each (lambda (obsolete-patch)
+ (delete-file (string-append
+ patch-dir "/"
+ obsolete-patch)))
+ (list "configure.diff"
+ "memfd.diff"))
+
+ (for-each (lambda (patch-file)
+ (invoke "patch" "--force" "-p1"
+ "--input" patch-file))
+ (find-files patch-dir
+ "\\.diff$"))
+ #t)))
+ (add-after 'install 'install-info
+ (lambda* (#:key inputs outputs #:allow-other-keys)
+ ;; Install the Info manual, unless Texinfo is missing.
+ (when (assoc-ref inputs "texinfo")
+ (let* ((out (assoc-ref outputs "out"))
+ (dir (string-append out "/share/info")))
+ (invoke "make" "info")
+ (for-each (lambda (info)
+ (install-file info dir))
+ (find-files "." "\\.info"))))
+ #t))
+ ;; Create a wrapper for Samba. This allows QEMU to use Samba without
+ ;; pulling it in as an input. Note that you need to explicitly install
+ ;; Samba in your Guix profile for Samba support.
+ (add-after 'install-info 'create-samba-wrapper
+ (lambda* (#:key inputs outputs #:allow-other-keys)
+ (let* ((out (assoc-ref %outputs "out"))
+ (libexec (string-append out "/libexec")))
+ (call-with-output-file "samba-wrapper"
+ (lambda (port)
+ (format port "#!/bin/sh
+exec smbd $@")))
+ (chmod "samba-wrapper" #o755)
+ (install-file "samba-wrapper" libexec))
+ #t))
+ (add-before 'configure 'prevent-network-configuration
+ (lambda _
+ ;; Prevent the build from trying to use git to fetch from the net.
+ (substitute* "Makefile"
+ (("@./config.status")
+ "")) #t))
+ (add-before 'check 'disable-unusable-tests
+ (lambda* (#:key inputs outputs #:allow-other-keys)
+ (substitute* "tests/Makefile.include"
+ ;; Comment out the test-qga test, which needs /sys and
+ ;; fails within the build environment.
+ (("check-unit-.* tests/test-qga" all)
+ (string-append "# " all)))
+ (substitute* "tests/Makefile.include"
+ ;; Comment out the test-char test, which needs networking and
+ ;; fails within the build environment.
+ (("check-unit-.* tests/test-char" all)
+ (string-append "# " all)))
+ #t)))))
+ (native-inputs
+ `(("python-2" ,python-2) ; QEMU 2 needs Python 2
+ ("glib:bin" ,glib "bin")
+ ("perl" ,perl)
+ ("flex" ,flex)
+ ("bison" ,bison)
+ ("pkg-config" ,pkg-config)
+ ("texinfo" ,texinfo)))
+ (inputs
+ `(("afl-source" ,(package-source american-fuzzy-lop))
+ ("alsa-lib" ,alsa-lib)
+ ("attr" ,attr)
+ ("glib" ,glib)
+ ("libaio" ,libaio)
+ ("libattr" ,attr)
+ ("libcap" ,libcap)
+ ("libjpeg" ,libjpeg-turbo)
+ ("libpng" ,libpng)
+ ("ncurses" ,ncurses)
+ ("pixman" ,pixman)
+ ("util-linux" ,util-linux)
+ ("zlib" ,zlib)))
+ (home-page "https://www.qemu.org")
+ (synopsis "Machine emulator and virtualizer (without GUI)")
+ (description
+ "QEMU is a generic machine emulator and virtualizer.
+
+When used as a machine emulator, QEMU can run OSes and programs made for one
+machine (e.g. an ARM board) on a different machine---e.g., your own PC. By
+using dynamic translation, it achieves very good performance.
+
+When used as a virtualizer, QEMU achieves near native performances by
+executing the guest code directly on the host CPU. QEMU supports
+virtualization when executing under the Xen hypervisor or using
+the KVM kernel module in Linux. When using KVM, QEMU can virtualize x86,
+server and embedded PowerPC, and S390 guests.")
+ ;; Many files are GPLv2+, but some are GPLv2-only---e.g., `memory.c'.
+ (license license:gpl2)
+ ;; Several tests fail on MIPS.
+ (supported-systems (delete "mips64el-linux" %supported-systems))))))
+
(define-public stress-make
(let ((commit "9e92dff8f0157f012aaf31de5b8b8112ad720100")
(revision "1")) ;No official source distribution
diff --git a/gnu/packages/virtualization.scm b/gnu/packages/virtualization.scm
index 03f0ceaeee..b7e2c415fe 100644
--- a/gnu/packages/virtualization.scm
+++ b/gnu/packages/virtualization.scm
@@ -284,34 +284,6 @@ server and embedded PowerPC, and S390 guests.")
'("libusb" "mesa" "sdl2" "spice" "virglrenderer" "gtk+"
"usbredir" "libdrm" "libepoxy" "pulseaudio" "vde2")))))
-;; The GRUB test suite fails with later versions of Qemu, so we
-;; keep it at 2.10 for now. See
-;; <https://lists.gnu.org/archive/html/bug-grub/2018-02/msg00004.html>.
-;; This package is hidden since we do not backport updates to it.
-(define-public qemu-minimal-2.10
- (hidden-package
- (package
- (inherit qemu-minimal)
- (version "2.10.2")
- (source (origin
- (method url-fetch)
- (uri (string-append "https://download.qemu.org/qemu-"
- version ".tar.xz"))
- (sha256
- (base32
- "17w21spvaxaidi2am5lpsln8yjpyp2zi3s3gc6nsxj5arlgamzgw"))
- (patches
- (search-patches "qemu-glibc-2.27.patch"))))
- ;; qemu-minimal-2.10 needs Python 2. Remove below once no longer necessary.
- (native-inputs `(("python-2" ,python-2)
- ,@(fold alist-delete (package-native-inputs qemu-minimal)
- '("python-wrapper" "python-sphinx"))))
- (inputs
- (fold alist-delete (package-inputs qemu-minimal)
- ;; Disable seccomp support, because it's not required for the GRUB
- ;; test suite, and because it fails with libseccomp 2.4.2 and later.
- '("libseccomp"))))))
-
(define-public libosinfo
(package
(name "libosinfo")
--
2.25.1
guix-patches <at> gnu.org:bug#40178; Package guix-patches.
(Sun, 22 Mar 2020 13:13:02 GMT) Full text and rfc822 format available.Message #14 received at 40178 <at> debbugs.gnu.org (full text, mbox):
From: Danny Milosavljevic <dannym <at> scratchpost.org> To: Leo Famulari <leo <at> famulari.name> Cc: 40178 <at> debbugs.gnu.org Subject: Re: [bug#40178] [PATCH 1/2] gnu: Prefix licenses in (gnu packages debug). Date: Sun, 22 Mar 2020 14:12:11 +0100
[Message part 1 (text/plain, inline)]
LGTM!
[Message part 2 (application/pgp-signature, inline)]
guix-patches <at> gnu.org:bug#40178; Package guix-patches.
(Sun, 22 Mar 2020 13:17:02 GMT) Full text and rfc822 format available.Message #17 received at 40178 <at> debbugs.gnu.org (full text, mbox):
From: Danny Milosavljevic <dannym <at> scratchpost.org> To: Leo Famulari <leo <at> famulari.name> Cc: 40178 <at> debbugs.gnu.org Subject: Re: [bug#40178] [PATCH 2/2] gnu: Remove qemu-minimal-2.10. Date: Sun, 22 Mar 2020 14:16:11 +0100
[Message part 1 (text/plain, inline)]
Hi Leo,
On Sat, 21 Mar 2020 18:24:17 -0400
Leo Famulari <leo <at> famulari.name> wrote:
> + (lambda _
> + ;; Prevent the build from trying to use git to fetch from the net.
> + (substitute* "Makefile"
> + (("@./config.status")
> + "")) #t))
Maybe newline before the #t.
Otherwise LGTM!
[Message part 2 (application/pgp-signature, inline)]
guix-patches <at> gnu.org:bug#40178; Package guix-patches.
(Sun, 22 Mar 2020 14:23:01 GMT) Full text and rfc822 format available.Message #20 received at 40178 <at> debbugs.gnu.org (full text, mbox):
From: Marius Bakke <mbakke <at> fastmail.com> To: Leo Famulari <leo <at> famulari.name>, 40178 <at> debbugs.gnu.org Subject: Re: [bug#40178] [PATCH 2/2] gnu: Remove qemu-minimal-2.10. Date: Sun, 22 Mar 2020 15:22:44 +0100
[Message part 1 (text/plain, inline)]
Leo Famulari <leo <at> famulari.name> writes:
> * gnu/packages/virtualization.scm (qemu-minimal-2.10): Remove variable.
> * gnu/packages/debug.scm (qemu-for-american-fuzzy-lop): New variable.
> (american-fuzzy-lop)[inputs]: Remove custom-qemu and add
> qemu-for-american-fuzzy-lop.
> [arguments]: Adjust reference to QEMU in the 'install-qemu' phase.
Thanks for this. A couple of suggestions:
> - ;; afl only supports using a single afl-qemu-trace executable, so
> - ;; we only build qemu for the native target.
Perhaps preserve this comment close to the (let ((machine ...))) line?
> + (add-after 'install 'install-info
> + (lambda* (#:key inputs outputs #:allow-other-keys)
> + ;; Install the Info manual, unless Texinfo is missing.
> + (when (assoc-ref inputs "texinfo")
> + (let* ((out (assoc-ref outputs "out"))
> + (dir (string-append out "/share/info")))
> + (invoke "make" "info")
> + (for-each (lambda (info)
> + (install-file info dir))
> + (find-files "." "\\.info"))))
> + #t))
> + ;; Create a wrapper for Samba. This allows QEMU to use Samba without
> + ;; pulling it in as an input. Note that you need to explicitly install
> + ;; Samba in your Guix profile for Samba support.
> + (add-after 'install-info 'create-samba-wrapper
> + (lambda* (#:key inputs outputs #:allow-other-keys)
> + (let* ((out (assoc-ref %outputs "out"))
> + (libexec (string-append out "/libexec")))
> + (call-with-output-file "samba-wrapper"
> + (lambda (port)
> + (format port "#!/bin/sh
> +exec smbd $@")))
> + (chmod "samba-wrapper" #o755)
> + (install-file "samba-wrapper" libexec))
> + #t))
> + (add-before 'configure 'prevent-network-configuration
> + (lambda _
> + ;; Prevent the build from trying to use git to fetch from the net.
> + (substitute* "Makefile"
> + (("@./config.status")
> + "")) #t))
I think we can remove all the phases above for simplicity.
> + (native-inputs
> + `(("python-2" ,python-2) ; QEMU 2 needs Python 2
> + ("glib:bin" ,glib "bin")
> + ("perl" ,perl)
> + ("flex" ,flex)
> + ("bison" ,bison)
> + ("pkg-config" ,pkg-config)
> + ("texinfo" ,texinfo)))
As well as the texinfo input.
> + (inputs
> + `(("afl-source" ,(package-source american-fuzzy-lop))
> + ("alsa-lib" ,alsa-lib)
> + ("attr" ,attr)
> + ("glib" ,glib)
> + ("libaio" ,libaio)
> + ("libattr" ,attr)
> + ("libcap" ,libcap)
> + ("libjpeg" ,libjpeg-turbo)
> + ("libpng" ,libpng)
> + ("ncurses" ,ncurses)
> + ("pixman" ,pixman)
> + ("util-linux" ,util-linux)
> + ("zlib" ,zlib)))
It's good to keep a list of explicit inputs here instead of having to
delete any new inputs to 'qemu-minimal'. However...
> + (home-page "https://www.qemu.org")
> + (synopsis "Machine emulator and virtualizer (without GUI)")
> + (description
> + "QEMU is a generic machine emulator and virtualizer.
> +
> +When used as a machine emulator, QEMU can run OSes and programs made for one
> +machine (e.g. an ARM board) on a different machine---e.g., your own PC. By
> +using dynamic translation, it achieves very good performance.
> +
> +When used as a virtualizer, QEMU achieves near native performances by
> +executing the guest code directly on the host CPU. QEMU supports
> +virtualization when executing under the Xen hypervisor or using
> +the KVM kernel module in Linux. When using KVM, QEMU can virtualize x86,
> +server and embedded PowerPC, and S390 guests.")
> + ;; Many files are GPLv2+, but some are GPLv2-only---e.g., `memory.c'.
> + (license license:gpl2)
> + ;; Several tests fail on MIPS.
> + (supported-systems (delete "mips64el-linux" %supported-systems))))))
This section can be removed, as it gets inherited and does not affect
the derivation.
In any case LGTM, too!
[signature.asc (application/pgp-signature, inline)]
guix-patches <at> gnu.org:bug#40178; Package guix-patches.
(Sun, 22 Mar 2020 18:30:02 GMT) Full text and rfc822 format available.Message #23 received at 40178 <at> debbugs.gnu.org (full text, mbox):
From: Leo Famulari <leo <at> famulari.name> To: Marius Bakke <mbakke <at> fastmail.com> Cc: 40178 <at> debbugs.gnu.org Subject: Re: [bug#40178] [PATCH 2/2] gnu: Remove qemu-minimal-2.10. Date: Sun, 22 Mar 2020 14:29:40 -0400
On Sun, Mar 22, 2020 at 03:22:44PM +0100, Marius Bakke wrote:
> > - ;; afl only supports using a single afl-qemu-trace executable, so
> > - ;; we only build qemu for the native target.
>
> Perhaps preserve this comment close to the (let ((machine ...))) line?
Done.
> > + (add-after 'install 'install-info
> > + (add-after 'install-info 'create-samba-wrapper
> > + (add-before 'configure 'prevent-network-configuration
>
> I think we can remove all the phases above for simplicity.
Done.
> > + ("texinfo" ,texinfo)))
>
> As well as the texinfo input.
Done.
> > + (home-page "https://www.qemu.org")
> > + (synopsis "Machine emulator and virtualizer (without GUI)")
> > + (description
> > + "QEMU is a generic machine emulator and virtualizer.
> > +
> > +When used as a machine emulator, QEMU can run OSes and programs made for one
> > +machine (e.g. an ARM board) on a different machine---e.g., your own PC. By
> > +using dynamic translation, it achieves very good performance.
> > +
> > +When used as a virtualizer, QEMU achieves near native performances by
> > +executing the guest code directly on the host CPU. QEMU supports
> > +virtualization when executing under the Xen hypervisor or using
> > +the KVM kernel module in Linux. When using KVM, QEMU can virtualize x86,
> > +server and embedded PowerPC, and S390 guests.")
> > + ;; Many files are GPLv2+, but some are GPLv2-only---e.g., `memory.c'.
> > + (license license:gpl2)
> > + ;; Several tests fail on MIPS.
> > + (supported-systems (delete "mips64el-linux" %supported-systems))))))
>
> This section can be removed, as it gets inherited and does not affect
> the derivation.
This qemu-for-american-fuzzy-lop package doesn't inherit from anything.
My goal was to completely separate it from the "real" QEMU package so
that we can ignore about it and let whoever is interesting in AFL take
care of it. It wouldn't be a problem to inherit these fields, but
instead I will edit them to describe what this package is for.
> In any case LGTM, too!
Thanks for the review! Pushed as 268efd97c255d75ac488c163ff1b8c66c6bcb2b6
Christopher Baines <mail <at> cbaines.net>:Leo Famulari <leo <at> famulari.name>:Message #28 received at 40178-done <at> debbugs.gnu.org (full text, mbox):
From: Christopher Baines <mail <at> cbaines.net> To: 40178-done <at> debbugs.gnu.org Subject: Re: [bug#40178] [PATCH 2/2] gnu: Remove qemu-minimal-2.10. Date: Tue, 31 Mar 2020 22:07:24 +0100
[Message part 1 (text/plain, inline)]
Leo Famulari <leo <at> famulari.name> writes: > On Sun, Mar 22, 2020 at 03:22:44PM +0100, Marius Bakke wrote: >> In any case LGTM, too! > > Thanks for the review! Pushed as 268efd97c255d75ac488c163ff1b8c66c6bcb2b6 Closing this bug, as I think the patches have been merged.
[signature.asc (application/pgp-signature, inline)]
Debbugs Internal Request <help-debbugs <at> gnu.org>
to internal_control <at> debbugs.gnu.org.
(Wed, 29 Apr 2020 11:24:07 GMT) Full text and rfc822 format available.
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.