* [bug#30601] [PATCH staging] gnu: ninja: Disable tests on armhf.
@ 2018-02-25 3:54 Marius Bakke
2018-02-26 17:49 ` Ludovic Courtès
0 siblings, 1 reply; 5+ messages in thread
From: Marius Bakke @ 2018-02-25 3:54 UTC (permalink / raw)
To: 30601
* gnu/packages/ninja.scm (ninja)[arguments]: Set #:tests? #f on armhf. Adjust
CHECK-PHASE accordingly.
---
gnu/packages/ninja.scm | 20 +++++++++++++++-----
1 file changed, 15 insertions(+), 5 deletions(-)
diff --git a/gnu/packages/ninja.scm b/gnu/packages/ninja.scm
index adb236484..cff2ac38b 100644
--- a/gnu/packages/ninja.scm
+++ b/gnu/packages/ninja.scm
@@ -42,7 +42,15 @@
(build-system gnu-build-system)
(native-inputs `(("python" ,python-2)))
(arguments
- '(#:phases
+ ;; FIXME: On armhf-linux, Subprocess.TestWithLots fails like this:
+ ;; *** Failure in src/subprocess_test.cc:241
+ ;; ExitSuccess == procs[i]->Finish()
+ ;; See <https://hydra.gnu.org/build/2515522/nixlog/1/raw>.
+ ;; Disable for now.
+ `(#:tests? (let ((system ,(or (%current-target-system)
+ (%current-system))))
+ (not (string-prefix? "arm" system)))
+ #:phases
(modify-phases %standard-phases
(replace 'configure
(lambda _
@@ -53,10 +61,12 @@
(lambda _
(invoke "./configure.py" "--bootstrap")))
(replace 'check
- (lambda _
- (invoke "./configure.py")
- (invoke "./ninja" "ninja_test")
- (invoke "./ninja_test")))
+ (lambda* (#:key tests? #:allow-other-keys)
+ (if tests?
+ (and (invoke "./configure.py")
+ (invoke "./ninja" "ninja_test")
+ (invoke "./ninja_test"))
+ (display "test suite not run\n"))))
(replace 'install
(lambda* (#:key outputs #:allow-other-keys)
(let* ((out (assoc-ref outputs "out"))
--
2.16.2
^ permalink raw reply related [flat|nested] 5+ messages in thread* [bug#30601] [PATCH staging] gnu: ninja: Disable tests on armhf.
2018-02-25 3:54 [bug#30601] [PATCH staging] gnu: ninja: Disable tests on armhf Marius Bakke
@ 2018-02-26 17:49 ` Ludovic Courtès
2018-02-26 19:40 ` bug#30601: " Marius Bakke
0 siblings, 1 reply; 5+ messages in thread
From: Ludovic Courtès @ 2018-02-26 17:49 UTC (permalink / raw)
To: Marius Bakke; +Cc: 30601
Hi!
Marius Bakke <mbakke@fastmail.com> skribis:
> * gnu/packages/ninja.scm (ninja)[arguments]: Set #:tests? #f on armhf. Adjust
> CHECK-PHASE accordingly.
> ---
> gnu/packages/ninja.scm | 20 +++++++++++++++-----
> 1 file changed, 15 insertions(+), 5 deletions(-)
>
> diff --git a/gnu/packages/ninja.scm b/gnu/packages/ninja.scm
> index adb236484..cff2ac38b 100644
> --- a/gnu/packages/ninja.scm
> +++ b/gnu/packages/ninja.scm
> @@ -42,7 +42,15 @@
> (build-system gnu-build-system)
> (native-inputs `(("python" ,python-2)))
> (arguments
> - '(#:phases
> + ;; FIXME: On armhf-linux, Subprocess.TestWithLots fails like this:
> + ;; *** Failure in src/subprocess_test.cc:241
> + ;; ExitSuccess == procs[i]->Finish()
> + ;; See <https://hydra.gnu.org/build/2515522/nixlog/1/raw>.
> + ;; Disable for now.
Sounds reasonable. Could you also report it upstream? :-)
Thanks,
Ludo’.
^ permalink raw reply [flat|nested] 5+ messages in thread* bug#30601: [PATCH staging] gnu: ninja: Disable tests on armhf.
2018-02-26 17:49 ` Ludovic Courtès
@ 2018-02-26 19:40 ` Marius Bakke
2018-02-26 20:02 ` [bug#30601] " Marius Bakke
0 siblings, 1 reply; 5+ messages in thread
From: Marius Bakke @ 2018-02-26 19:40 UTC (permalink / raw)
To: Ludovic Courtès; +Cc: 30601-done
[-- Attachment #1: Type: text/plain, Size: 1423 bytes --]
Ludovic Courtès <ludo@gnu.org> writes:
> Hi!
>
> Marius Bakke <mbakke@fastmail.com> skribis:
>
>> * gnu/packages/ninja.scm (ninja)[arguments]: Set #:tests? #f on armhf. Adjust
>> CHECK-PHASE accordingly.
>> ---
>> gnu/packages/ninja.scm | 20 +++++++++++++++-----
>> 1 file changed, 15 insertions(+), 5 deletions(-)
>>
>> diff --git a/gnu/packages/ninja.scm b/gnu/packages/ninja.scm
>> index adb236484..cff2ac38b 100644
>> --- a/gnu/packages/ninja.scm
>> +++ b/gnu/packages/ninja.scm
>> @@ -42,7 +42,15 @@
>> (build-system gnu-build-system)
>> (native-inputs `(("python" ,python-2)))
>> (arguments
>> - '(#:phases
>> + ;; FIXME: On armhf-linux, Subprocess.TestWithLots fails like this:
>> + ;; *** Failure in src/subprocess_test.cc:241
>> + ;; ExitSuccess == procs[i]->Finish()
>> + ;; See <https://hydra.gnu.org/build/2515522/nixlog/1/raw>.
>> + ;; Disable for now.
>
> Sounds reasonable. Could you also report it upstream? :-)
Actually, now that I'm more awake, I found the problem. On most
machines this test is skipped because it requires 1025 open files.
However, on Hydras armhf machines, this limit is apparently higher.
The problem is that the test calls out to /bin/echo. I was able to
reproduce it on x86_64 simply by raising the softlimit to 4096.
So, I pushed a one-line fix as d0de4c40e24ee52e4382ee2ac82088932ab49099.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 487 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread* [bug#30601] [PATCH staging] gnu: ninja: Disable tests on armhf.
2018-02-26 19:40 ` bug#30601: " Marius Bakke
@ 2018-02-26 20:02 ` Marius Bakke
2018-02-27 9:05 ` Ludovic Courtès
0 siblings, 1 reply; 5+ messages in thread
From: Marius Bakke @ 2018-02-26 20:02 UTC (permalink / raw)
To: Ludovic Courtès; +Cc: 30601-done
[-- Attachment #1: Type: text/plain, Size: 1707 bytes --]
Marius Bakke <mbakke@fastmail.com> writes:
> Ludovic Courtès <ludo@gnu.org> writes:
>
>> Hi!
>>
>> Marius Bakke <mbakke@fastmail.com> skribis:
>>
>>> * gnu/packages/ninja.scm (ninja)[arguments]: Set #:tests? #f on armhf. Adjust
>>> CHECK-PHASE accordingly.
>>> ---
>>> gnu/packages/ninja.scm | 20 +++++++++++++++-----
>>> 1 file changed, 15 insertions(+), 5 deletions(-)
>>>
>>> diff --git a/gnu/packages/ninja.scm b/gnu/packages/ninja.scm
>>> index adb236484..cff2ac38b 100644
>>> --- a/gnu/packages/ninja.scm
>>> +++ b/gnu/packages/ninja.scm
>>> @@ -42,7 +42,15 @@
>>> (build-system gnu-build-system)
>>> (native-inputs `(("python" ,python-2)))
>>> (arguments
>>> - '(#:phases
>>> + ;; FIXME: On armhf-linux, Subprocess.TestWithLots fails like this:
>>> + ;; *** Failure in src/subprocess_test.cc:241
>>> + ;; ExitSuccess == procs[i]->Finish()
>>> + ;; See <https://hydra.gnu.org/build/2515522/nixlog/1/raw>.
>>> + ;; Disable for now.
>>
>> Sounds reasonable. Could you also report it upstream? :-)
>
> Actually, now that I'm more awake, I found the problem. On most
> machines this test is skipped because it requires 1025 open files.
> However, on Hydras armhf machines, this limit is apparently higher.
>
> The problem is that the test calls out to /bin/echo. I was able to
> reproduce it on x86_64 simply by raising the softlimit to 4096.
>
> So, I pushed a one-line fix as d0de4c40e24ee52e4382ee2ac82088932ab49099.
Derp. Apparently I wasn't awake enough and managed to patch the wrong
file when making the commit. The followup in
6d796220c417a34100c282a7fa1f9afb6ac00339 fixes it, sorry for the noise.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 487 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread* [bug#30601] [PATCH staging] gnu: ninja: Disable tests on armhf.
2018-02-26 20:02 ` [bug#30601] " Marius Bakke
@ 2018-02-27 9:05 ` Ludovic Courtès
0 siblings, 0 replies; 5+ messages in thread
From: Ludovic Courtès @ 2018-02-27 9:05 UTC (permalink / raw)
To: Marius Bakke; +Cc: 30601-done
Marius Bakke <mbakke@fastmail.com> skribis:
> Marius Bakke <mbakke@fastmail.com> writes:
>
>> Ludovic Courtès <ludo@gnu.org> writes:
>>
>>> Hi!
>>>
>>> Marius Bakke <mbakke@fastmail.com> skribis:
>>>
>>>> * gnu/packages/ninja.scm (ninja)[arguments]: Set #:tests? #f on armhf. Adjust
>>>> CHECK-PHASE accordingly.
>>>> ---
>>>> gnu/packages/ninja.scm | 20 +++++++++++++++-----
>>>> 1 file changed, 15 insertions(+), 5 deletions(-)
>>>>
>>>> diff --git a/gnu/packages/ninja.scm b/gnu/packages/ninja.scm
>>>> index adb236484..cff2ac38b 100644
>>>> --- a/gnu/packages/ninja.scm
>>>> +++ b/gnu/packages/ninja.scm
>>>> @@ -42,7 +42,15 @@
>>>> (build-system gnu-build-system)
>>>> (native-inputs `(("python" ,python-2)))
>>>> (arguments
>>>> - '(#:phases
>>>> + ;; FIXME: On armhf-linux, Subprocess.TestWithLots fails like this:
>>>> + ;; *** Failure in src/subprocess_test.cc:241
>>>> + ;; ExitSuccess == procs[i]->Finish()
>>>> + ;; See <https://hydra.gnu.org/build/2515522/nixlog/1/raw>.
>>>> + ;; Disable for now.
>>>
>>> Sounds reasonable. Could you also report it upstream? :-)
>>
>> Actually, now that I'm more awake, I found the problem. On most
>> machines this test is skipped because it requires 1025 open files.
>> However, on Hydras armhf machines, this limit is apparently higher.
>>
>> The problem is that the test calls out to /bin/echo. I was able to
>> reproduce it on x86_64 simply by raising the softlimit to 4096.
>>
>> So, I pushed a one-line fix as d0de4c40e24ee52e4382ee2ac82088932ab49099.
>
> Derp. Apparently I wasn't awake enough and managed to patch the wrong
> file when making the commit. The followup in
> 6d796220c417a34100c282a7fa1f9afb6ac00339 fixes it, sorry for the noise.
Heh, thank you!
Ludo’.
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2018-02-27 9:06 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-02-25 3:54 [bug#30601] [PATCH staging] gnu: ninja: Disable tests on armhf Marius Bakke
2018-02-26 17:49 ` Ludovic Courtès
2018-02-26 19:40 ` bug#30601: " Marius Bakke
2018-02-26 20:02 ` [bug#30601] " Marius Bakke
2018-02-27 9:05 ` Ludovic Courtès
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).