From: Logarithmus <Logarithmus@users.noreply.github.com>
To: ml@inbox.vuxu.org
Subject: Re: [PR PATCH] [Updated] [RFC] common/build-style/: enable running tests in parallel
Date: Wed, 14 Jul 2021 17:10:18 +0200 [thread overview]
Message-ID: <20210714151018.FiCb2YIukJ6yqM8yMM8xuTEZuEjb5CTVRc6jQKmZHcE@z> (raw)
In-Reply-To: <gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-31811@inbox.vuxu.org>
[-- Attachment #1: Type: text/plain, Size: 413 bytes --]
There is an updated pull request by Logarithmus against master on the void-packages repository
https://github.com/Logarithmus/void-packages common-parallel-tests
https://github.com/void-linux/void-packages/pull/31811
[RFC] common/build-style/: enable running tests in parallel
What do you think? Are there any caveats?
A patch file from https://github.com/void-linux/void-packages/pull/31811.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-common-parallel-tests-31811.patch --]
[-- Type: text/x-diff, Size: 3962 bytes --]
From 09906118bbeb4699637ab689bbf8812cd6a24832 Mon Sep 17 00:00:00 2001
From: Artur Sinila <freesoftware@logarithmus.dev>
Date: Mon, 5 Jul 2021 03:44:08 +0300
Subject: [PATCH] common/build-style/: enable running tests in parallel
---
common/build-style/cmake.sh | 6 +++++-
common/build-style/configure.sh | 6 +++++-
common/build-style/gnu-configure.sh | 6 +++++-
common/build-style/gnu-makefile.sh | 6 +++++-
common/build-style/perl-module.sh | 6 +++++-
common/environment/setup/sourcepkg.sh | 2 +-
6 files changed, 26 insertions(+), 6 deletions(-)
diff --git a/common/build-style/cmake.sh b/common/build-style/cmake.sh
index a7627e7dcf21..c3f6143d928a 100644
--- a/common/build-style/cmake.sh
+++ b/common/build-style/cmake.sh
@@ -113,7 +113,11 @@ do_check() {
: ${make_check_target:=test}
- ${make_cmd} ${make_check_args} ${make_check_target}
+ if [ -z "$disable_parallel_check" ]; then
+ ${make_cmd} ${makejobs} ${make_check_args} ${make_check_target}
+ else
+ ${make_cmd} ${make_check_args} ${make_check_target}
+ fi
}
do_install() {
diff --git a/common/build-style/configure.sh b/common/build-style/configure.sh
index 0963fc347934..b846a1a77572 100644
--- a/common/build-style/configure.sh
+++ b/common/build-style/configure.sh
@@ -29,7 +29,11 @@ do_check() {
: ${make_cmd:=make}
: ${make_check_target:=check}
- ${make_cmd} ${make_check_args} ${make_check_target}
+ if [ -z "$disable_parallel_check" ]; then
+ ${make_cmd} ${makejobs} ${make_check_args} ${make_check_target}
+ else
+ ${make_cmd} ${make_check_args} ${make_check_target}
+ fi
}
do_install() {
diff --git a/common/build-style/gnu-configure.sh b/common/build-style/gnu-configure.sh
index 8121136cd121..b09de442b05b 100644
--- a/common/build-style/gnu-configure.sh
+++ b/common/build-style/gnu-configure.sh
@@ -30,7 +30,11 @@ do_check() {
: ${make_cmd:=make}
: ${make_check_target:=check}
- ${make_cmd} ${make_check_args} ${make_check_target}
+ if [ -z "$disable_parallel_check" ]; then
+ ${make_cmd} ${makejobs} ${make_check_args} ${make_check_target}
+ else
+ ${make_cmd} ${make_check_args} ${make_check_target}
+ fi
}
do_install() {
diff --git a/common/build-style/gnu-makefile.sh b/common/build-style/gnu-makefile.sh
index d7ea148fc483..0f7afa7d3695 100644
--- a/common/build-style/gnu-makefile.sh
+++ b/common/build-style/gnu-makefile.sh
@@ -30,7 +30,11 @@ do_check() {
: ${make_cmd:=make}
: ${make_check_target:=check}
- ${make_cmd} ${make_check_args} ${make_check_target}
+ if [ -z "$disable_parallel_check" ]; then
+ ${make_cmd} ${makejobs} ${make_check_args} ${make_check_target}
+ else
+ ${make_cmd} ${make_check_args} ${make_check_target}
+ fi
}
do_install() {
diff --git a/common/build-style/perl-module.sh b/common/build-style/perl-module.sh
index b9a01b13cfa9..d47945b33d36 100644
--- a/common/build-style/perl-module.sh
+++ b/common/build-style/perl-module.sh
@@ -79,7 +79,11 @@ do_check() {
: ${make_cmd:=make}
: ${make_check_target:=test}
- ${make_cmd} ${make_check_args} ${make_check_target}
+ if [ -z "$disable_parallel_check" ]; then
+ ${make_cmd} ${makejobs} ${make_check_args} ${make_check_target}
+ else
+ ${make_cmd} ${make_check_args} ${make_check_target}
+ fi
}
do_install() {
diff --git a/common/environment/setup/sourcepkg.sh b/common/environment/setup/sourcepkg.sh
index bc06f745b78a..b6caa797138f 100644
--- a/common/environment/setup/sourcepkg.sh
+++ b/common/environment/setup/sourcepkg.sh
@@ -3,7 +3,7 @@
## VARIABLES
unset -v pkgname version revision short_desc homepage license maintainer
-unset -v archs distfiles checksum build_style build_helper nocross broken
+unset -v archs distfiles checksum build_style build_helper nocross disable_parallel_check broken
unset -v configure_script configure_args wrksrc build_wrksrc create_wrksrc
unset -v make_build_args make_check_args make_install_args
unset -v make_build_target make_check_target make_install_target
next prev parent reply other threads:[~2021-07-14 15:10 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-07-05 1:29 [PR PATCH] " Logarithmus
2021-07-05 23:37 ` [RFC] " Logarithmus
2021-07-05 23:46 ` ericonr
2021-07-06 1:51 ` q66
2021-07-14 14:51 ` [PR PATCH] [Updated] " Logarithmus
2021-07-14 15:09 ` Logarithmus
2021-07-14 15:09 ` Chocimier
2021-07-14 15:09 ` Logarithmus
2021-07-14 15:10 ` Logarithmus [this message]
2021-07-14 15:12 ` Logarithmus
2021-07-14 15:24 ` Chocimier
2021-07-14 15:24 ` [PR REVIEW] " sgn
2021-07-14 15:29 ` sgn
2021-07-24 12:01 ` Logarithmus
2021-07-24 12:04 ` Logarithmus
2021-07-24 12:16 ` sgn
2021-07-24 12:30 ` [PR PATCH] [Updated] " Logarithmus
2021-07-24 12:31 ` Logarithmus
2021-07-25 19:53 ` [PR REVIEW] " Logarithmus
2021-07-25 19:54 ` Logarithmus
2021-07-30 20:43 ` Chocimier
2022-05-24 2:13 ` github-actions
2022-06-08 2:11 ` [PR PATCH] [Closed]: " github-actions
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=20210714151018.FiCb2YIukJ6yqM8yMM8xuTEZuEjb5CTVRc6jQKmZHcE@z \
--to=logarithmus@users.noreply.github.com \
--cc=ml@inbox.vuxu.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.
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 NNTP newsgroup(s).