[PATCH core-updates] gnu: findutils-boot0: Enable tests on arm.

  • Done
  • quality assurance status badge
Details
3 participants
  • Efraim Flashner
  • Maxim Cournoyer
  • Pierre Langlois
Owner
unassigned
Submitted by
Pierre Langlois
Severity
normal
P
P
Pierre Langlois wrote on 11 Feb 2021 20:38
(name . Guix-patches)(address . guix-patches@gnu.org)
87czx6wg2f.fsf@gmx.com
Hello Guix!

I noticed the recent rust changes on core-updates, so I was eager to try
and get rust working on aarch64 again, but I noticed I couldn't
bootstrap because the pthread tests from findutils-boot0 were failing.

I was just about to send a patch for that and then saw we had disabled
the tests all together. So, here's an updated patch that re-enables them
again, but disable only pthread tests.

Let me know what you think of this patch, I've tested it on aarch64 and
currently building rust 1.29, but I wasn't able to test it on x86, I'm
getting the following failure:

Toggle snippet (16 lines)
starting phase `compress-documentation'
compressing documentation in '/gnu/store/fl4xmk5pln6lw81ixjk3w24hhdn1zi26-findutils-boot0-4.8.0/share/man' with "gzip" and flags ("--best" "--no-name")
sterror: unknown error: /gnu/store/fl4xmk5pln6lw81ixjk3w24hhdn1zi26-findutils-boot0-4.8.0/share/man/man1/find.1
sterror: unknown error: /gnu/store/fl4xmk5pln6lw81ixjk3w24hhdn1zi26-findutils-boot0-4.8.0/share/man/man1/locate.1
sterror: unknown error: /gnu/store/fl4xmk5pln6lw81ixjk3w24hhdn1zi26-findutils-boot0-4.8.0/share/man/man1/updatedb.1
sterror: unknown error: /gnu/store/fl4xmk5pln6lw81ixjk3w24hhdn1zi26-findutils-boot0-4.8.0/share/man/man1/xargs.1
sterror: unknown error: /gnu/store/fl4xmk5pln6lw81ixjk3w24hhdn1zi26-findutils-boot0-4.8.0/share/man/man5/locatedb.5
error: in phase 'compress-documentation': uncaught exception:
srfi-34 #<condition &invoke-error [program: "gzip" arguments: ("--best" "--no-name" "/gnu/store/fl4xmk5pln6lw81ixjk3w24hhdn1zi26-findutils-boot0-4.8.0/share/man/man1/find.1" "/gnu/store/fl4xmk5pln6lw81ixjk3w24hhdn1zi26-findutils-boot0-4.8.0/share/man/man1/locate.1" "/gnu/store/fl4xmk5pln6lw81ixjk3w24hhdn1zi26-findutils-boot0-4.8.0/share/man/man1/updatedb.1" "/gnu/store/fl4xmk5pln6lw81ixjk3w24hhdn1zi26-findutils-boot0-4.8.0/share/man/man1/xargs.1" "/gnu/store/fl4xmk5pln6lw81ixjk3w24hhdn1zi26-findutils-boot0-4.8.0/share/man/man5/locatedb.5") exit-status: 1 term-signal: #f stop-signal: #f] 1222080>
phase `compress-documentation' failed after 0.0 seconds
command "gzip" "--best" "--no-name" "/gnu/store/fl4xmk5pln6lw81ixjk3w24hhdn1zi26-findutils-boot0-4.8.0/share/man/man1/find.1" "/gnu/store/fl4xmk5pln6lw81ixjk3w24hhdn1zi26-findutils-boot0-4.8.0/share/man/man1/locate.1" "/gnu/store/fl4xmk5pln6lw81ixjk3w24hhdn1zi26-findutils-boot0-4.8.0/share/man/man1/updatedb.1" "/gnu/store/fl4xmk5pln6lw81ixjk3w24hhdn1zi26-findutils-boot0-4.8.0/share/man/man1/xargs.1" "/gnu/store/fl4xmk5pln6lw81ixjk3w24hhdn1zi26-findutils-boot0-4.8.0/share/man/man5/locatedb.5" failed with status 1
builder for `/gnu/store/g64zjdljpk4wawimf44v7ysgf23ywg30-findutils-boot0-4.8.0.drv' failed with exit code 1
build of /gnu/store/g64zjdljpk4wawimf44v7ysgf23ywg30-findutils-boot0-4.8.0.drv failed
View build log at '/var/log/guix/drvs/g6/4zjdljpk4wawimf44v7ysgf23ywg30-findutils-boot0-4.8.0.drv.bz2'.

I'm a bit confused what's going on, anybody has a clue?

By the way, I'm glad we no longer have to pass all the tests for every
stage of the rust bootstrap, getting this working on different platform
was quite tedious :-).

Thanks,
Pierre
-----BEGIN PGP SIGNATURE-----

iQFMBAEBCgA2FiEEctU9gYy29KFyWDdMqPyeRH9PfVQFAmAlh6gYHHBpZXJyZS5s
YW5nbG9pc0BnbXguY29tAAoJEKj8nkR/T31UF9EH/jF3/YnF3foODsp++efxhQFa
iiEtKuJzIIEE5t2kCw6o9vogeCgH4pRouDYW+AQ6iIXeC56vEcgINz7OkjwFrvGH
bbyBWYiZJAiLmzegMW+yAfe63Xw3huAyDmZl1t2No0zw/4/epDJWwrxlS1ApxU/k
m/BBpIfz9msZ96y+nZSsnEkAIsSis10KIG6FTcjYPlZ7ooYh3g45qlVaO23quSe8
xKLIe+B3qYvaaOZfJeC/XJG/cKFp1+MKghVRyNNP/7deAVE+Grvw4r1GVNINLYkB
gBbINEzZEYbWoOhz9QarkAE5cjYgGnruvFF7+7CXlSFU066lKKx9JyGlKKOzEtE=
=0RhJ
-----END PGP SIGNATURE-----

From 6dffcb4a99909830cf0cecd2c9882867b379eca4 Mon Sep 17 00:00:00 2001
From: Pierre Langlois <pierre.langlois@gmx.com>
Date: Sat, 6 Feb 2021 21:23:56 +0000
Subject: [PATCH] gnu: findutils-boot0: Enable tests on arm.

* gnu/packages/commencement.scm (findutils-boot0)[arguments]: Enable tests.
Rename skip-fnmatch-test phase skip-tests. Only disable test-fnmatch on
x86_64 and i686. Disable pthread tests on other platforms.
---
gnu/packages/commencement.scm | 25 ++++++++++++++++---------
1 file changed, 16 insertions(+), 9 deletions(-)

Toggle diff (53 lines)
diff --git a/gnu/packages/commencement.scm b/gnu/packages/commencement.scm
index 890d57941f..ecb2307b05 100644
--- a/gnu/packages/commencement.scm
+++ b/gnu/packages/commencement.scm
@@ -10,6 +10,7 @@
;;; Copyright © 2020 Timothy Sample <samplet@ngyro.com>
;;; Copyright © 2020 Guy Fleury Iteriteka <gfleury@disroot.org>
;;; Copyright © 2021 Maxim Cournoyer <maxim.cournoyer@gmail.com>
+;;; Copyright © 2021 Pierre Langlois <pierre.langlois@gmx.com>
;;;
;;; This file is part of GNU Guix.
;;;
@@ -2490,8 +2491,6 @@ exec " gcc "/bin/" program
,@(%bootstrap-inputs+toolchain)))
(arguments
`(#:implicit-inputs? #f
- ;; Ignore test failure in gnulib for armhf/aarch64.
- #:tests? ,(not (target-arm?))
#:guile ,%bootstrap-guile
;; The build system assumes we have done a mistake when time_t is 32-bit
@@ -2509,14 +2508,22 @@ exec " gcc "/bin/" program
,flags))
((#:phases phases '%standard-phases)
`(modify-phases ,phases
- (add-before 'check 'skip-fnmatch-test
+ (add-before 'check 'skip-tests
(lambda _
- ;; 'test-fnmatch' fails when using glibc-mesboot@2.16, due
- ;; to incorrect handling of the [:alpha:] regexp character
- ;; class. Ignore it.
- (substitute* "gnulib-tests/Makefile"
- (("^XFAIL_TESTS =")
- "XFAIL_TESTS = test-fnmatch ")))))))))))
+ ,(match (%current-system)
+ ;; 'test-fnmatch' fails when using glibc-mesboot@2.16, due
+ ;; to incorrect handling of the [:alpha:] regexp character
+ ;; class. Ignore it.
+ ((or "x86_64-linux" "i686-linux")
+ '(substitute* "gnulib-tests/Makefile"
+ (("^XFAIL_TESTS =")
+ "XFAIL_TESTS = test-fnmatch ")))
+ (_
+ '(substitute* "gnulib-tests/Makefile"
+ (("test-pthread\\$\\(EXEEXT\\)") "")
+ (("test-pthread-thread\\$\\(EXEEXT\\)") "")
+ (("test-pthread_sigmask1\\$\\(EXEEXT\\)") "")
+ (("test-pthread_sigmask2\\$\\(EXEEXT\\)") "")))))))))))))
(define file
(package
--
2.30.0
E
E
Efraim Flashner wrote on 14 Feb 2021 13:27
(name . Pierre Langlois)(address . pierre.langlois@gmx.com)(address . 46440@debbugs.gnu.org)
YCkXQKTN2i3fZL2d@3900XT
On Thu, Feb 11, 2021 at 07:38:16PM +0000, Pierre Langlois wrote:
Toggle quote (40 lines)
> Hello Guix!
>
> I noticed the recent rust changes on core-updates, so I was eager to try
> and get rust working on aarch64 again, but I noticed I couldn't
> bootstrap because the pthread tests from findutils-boot0 were failing.
>
> I was just about to send a patch for that and then saw we had disabled
> the tests all together. So, here's an updated patch that re-enables them
> again, but disable only pthread tests.
>
> Let me know what you think of this patch, I've tested it on aarch64 and
> currently building rust 1.29, but I wasn't able to test it on x86, I'm
> getting the following failure:
>
> --8<---------------cut here---------------start------------->8---
> starting phase `compress-documentation'
> compressing documentation in '/gnu/store/fl4xmk5pln6lw81ixjk3w24hhdn1zi26-findutils-boot0-4.8.0/share/man' with "gzip" and flags ("--best" "--no-name")
> sterror: unknown error: /gnu/store/fl4xmk5pln6lw81ixjk3w24hhdn1zi26-findutils-boot0-4.8.0/share/man/man1/find.1
> sterror: unknown error: /gnu/store/fl4xmk5pln6lw81ixjk3w24hhdn1zi26-findutils-boot0-4.8.0/share/man/man1/locate.1
> sterror: unknown error: /gnu/store/fl4xmk5pln6lw81ixjk3w24hhdn1zi26-findutils-boot0-4.8.0/share/man/man1/updatedb.1
> sterror: unknown error: /gnu/store/fl4xmk5pln6lw81ixjk3w24hhdn1zi26-findutils-boot0-4.8.0/share/man/man1/xargs.1
> sterror: unknown error: /gnu/store/fl4xmk5pln6lw81ixjk3w24hhdn1zi26-findutils-boot0-4.8.0/share/man/man5/locatedb.5
> error: in phase 'compress-documentation': uncaught exception:
> srfi-34 #<condition &invoke-error [program: "gzip" arguments: ("--best" "--no-name" "/gnu/store/fl4xmk5pln6lw81ixjk3w24hhdn1zi26-findutils-boot0-4.8.0/share/man/man1/find.1" "/gnu/store/fl4xmk5pln6lw81ixjk3w24hhdn1zi26-findutils-boot0-4.8.0/share/man/man1/locate.1" "/gnu/store/fl4xmk5pln6lw81ixjk3w24hhdn1zi26-findutils-boot0-4.8.0/share/man/man1/updatedb.1" "/gnu/store/fl4xmk5pln6lw81ixjk3w24hhdn1zi26-findutils-boot0-4.8.0/share/man/man1/xargs.1" "/gnu/store/fl4xmk5pln6lw81ixjk3w24hhdn1zi26-findutils-boot0-4.8.0/share/man/man5/locatedb.5") exit-status: 1 term-signal: #f stop-signal: #f] 1222080>
> phase `compress-documentation' failed after 0.0 seconds
> command "gzip" "--best" "--no-name" "/gnu/store/fl4xmk5pln6lw81ixjk3w24hhdn1zi26-findutils-boot0-4.8.0/share/man/man1/find.1" "/gnu/store/fl4xmk5pln6lw81ixjk3w24hhdn1zi26-findutils-boot0-4.8.0/share/man/man1/locate.1" "/gnu/store/fl4xmk5pln6lw81ixjk3w24hhdn1zi26-findutils-boot0-4.8.0/share/man/man1/updatedb.1" "/gnu/store/fl4xmk5pln6lw81ixjk3w24hhdn1zi26-findutils-boot0-4.8.0/share/man/man1/xargs.1" "/gnu/store/fl4xmk5pln6lw81ixjk3w24hhdn1zi26-findutils-boot0-4.8.0/share/man/man5/locatedb.5" failed with status 1
> builder for `/gnu/store/g64zjdljpk4wawimf44v7ysgf23ywg30-findutils-boot0-4.8.0.drv' failed with exit code 1
> build of /gnu/store/g64zjdljpk4wawimf44v7ysgf23ywg30-findutils-boot0-4.8.0.drv failed
> View build log at '/var/log/guix/drvs/g6/4zjdljpk4wawimf44v7ysgf23ywg30-findutils-boot0-4.8.0.drv.bz2'.
> --8<---------------cut here---------------end--------------->8---
>
> I'm a bit confused what's going on, anybody has a clue?
>
> By the way, I'm glad we no longer have to pass all the tests for every
> stage of the rust bootstrap, getting this working on different platform
> was quite tedious :-).
>
> Thanks,
> Pierre

I had planned on re-enabling the tests with the next release of
findutils. I built out to findutils on aarch64 and armhf and they both
passed their test suites without any problems so I figured it was safe
enough to skip them for now. If you can skip just the pthread test(s)
that be better probably.

--
Efraim Flashner <efraim@flashner.co.il> ????? ?????
GPG key = A28B F40C 3E55 1372 662D 14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted
-----BEGIN PGP SIGNATURE-----

iQIzBAABCgAdFiEEoov0DD5VE3JmLRT3Qarn3Mo9g1EFAmApF0AACgkQQarn3Mo9
g1G6eg/+Im46BVKaN7PhpPoU9A/OY3t02aMWsomd1P5CfZR2VAO0lssw60JqhoCT
V9oqET4XGBFeiMh0TXMW10yMnO5AIQVeFJi34/f97XudqGt1NADX6O6j54X2VUjW
g3Y7NFlHUO8GUpRwMzZnYrqoLgXGplkOcFH1PJK36kiHKdneTEmtt/FPAxEBjQbN
+iIjPyngZXGSSbtLyO4lPn4Za+xD7SNyjv80O4ZgX754yYD8Zw4ZcSOkwBwwpWjm
ngx1bt73Y4P10Ps/voBi7kyBENxiqLo6DgHszDHXzkfiC8YJWv5NYUYPIrjVBUvs
lvLAD8XHvHa2cJTSY+DFQ/tQ3wCoAJd6B9AsblWqWeVzhkjT48iRBMLOAulHsgFU
JQxVOnZuFcnPdeVU/O534EcpKuNff5UEzFDr2jw0XHlYvz+mMP+8Mdrj8EA1VPB5
NN/81PoDeZ4vV72K7cLUQAS8s1GPojd1FEAsjocfGscyaBKMUF7xArF7FpBVQCeJ
8Gfwjq2NaP5FMlLtodnyaLnetvTsIUPIyVx/Qwmb3le7Rz/nwZYWgmRm7UsJGGTq
LDkxlw++7UXWdBOM4UMrund8JD6djrRA61dA8JfEuwk2aAiG3KpHF8HmL1PoxrY1
dZLM8gJwHImgpvNMe5FyfnOEkAG1qeKR1qlIYCa+Re3H65rc0hY=
=N7Po
-----END PGP SIGNATURE-----


M
M
Maxim Cournoyer wrote on 22 Jan 05:27 +0100
Re: bug#46440: [PATCH core-updates] gnu: findutils-boot0: Enable tests on arm.
(name . Efraim Flashner)(address . efraim@flashner.co.il)
87sf2q56wg.fsf_-_@gmail.com
Hi,

Efraim Flashner <efraim@flashner.co.il> writes:

[...]

Toggle quote (6 lines)
> I had planned on re-enabling the tests with the next release of
> findutils. I built out to findutils on aarch64 and armhf and they both
> passed their test suites without any problems so I figured it was safe
> enough to skip them for now. If you can skip just the pthread test(s)
> that be better probably.

Done with commit 3f4176ec37.

--
Thanks,
Maxim
Closed
?
Your comment

This issue is archived.

To comment on this conversation send an email to 46440@debbugs.gnu.org

To respond to this issue using the mumi CLI, first switch to it
mumi current 46440
Then, you may apply the latest patchset in this issue (with sign off)
mumi am -- -s
Or, compose a reply to this issue
mumi compose
Or, send patches to this issue
mumi send-email *.patch