From: Andreas Enge <andreas@enge.fr>
To: guix-devel@gnu.org
Subject: [PATCH] build-system/perl: Add handling of flags for parallel build and testing.
Date: Wed, 27 Aug 2014 12:23:47 +0200 [thread overview]
Message-ID: <20140827102347.GA19935@debian> (raw)
The patch below should do what is says... I needed to add the flags in two
places, actually. Can I push it?
Andreas
* guix/build-system/perl.scm (perl-build): Add support for variables
parallel-build? and parallel-tests?.
* guix/build-system/gnu.scm (gnu-build): Add a line break for clarity.
---
guix/build-system/gnu.scm | 3 ++-
guix/build-system/perl.scm | 4 ++++
2 files changed, 6 insertions(+), 1 deletion(-)
diff --git a/guix/build-system/gnu.scm b/guix/build-system/gnu.scm
index b2b184d..2c41ce0 100644
--- a/guix/build-system/gnu.scm
+++ b/guix/build-system/gnu.scm
@@ -268,7 +268,8 @@ System: GCC, GNU Make, Bash, Coreutils, etc."
(out-of-source? #f)
(tests? #t)
(test-target "check")
- (parallel-build? #t) (parallel-tests? #t)
+ (parallel-build? #t)
+ (parallel-tests? #t)
(patch-shebangs? #t)
(strip-binaries? #t)
(strip-flags ''("--strip-debug"))
diff --git a/guix/build-system/perl.scm b/guix/build-system/perl.scm
index 5695491..e8fe2d5 100644
--- a/guix/build-system/perl.scm
+++ b/guix/build-system/perl.scm
@@ -47,6 +47,8 @@
(perl (default-perl))
(search-paths '())
(tests? #t)
+ (parallel-build? #t)
+ (parallel-tests? #t)
(make-maker-flags ''())
(phases '(@ (guix build perl-build-system)
%standard-phases))
@@ -79,6 +81,8 @@ provides a `Makefile.PL' file as its build system."
#:system ,system
#:test-target "test"
#:tests? ,tests?
+ #:parallel-build? ,parallel-build?
+ #:parallel-tests? ,parallel-tests?
#:outputs %outputs
#:inputs %build-inputs)))
--
1.8.4
next reply other threads:[~2014-08-27 10:24 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-08-27 10:23 Andreas Enge [this message]
2014-08-27 12:23 ` [PATCH] build-system/perl: Add handling of flags for parallel build and testing Ludovic Courtès
2014-10-03 20:25 ` Andreas Enge
2014-10-03 20:56 ` Eric Bavier
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
List information: https://guix.gnu.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20140827102347.GA19935@debian \
--to=andreas@enge.fr \
--cc=guix-devel@gnu.org \
/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 public inbox
https://git.savannah.gnu.org/cgit/guix.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).