Message ID | cover.1689690897.git.janneke@gnu.org |
---|---|
Headers |
Return-Path: <guix-patches-bounces+patchwork=mira.cbaines.net@gnu.org> X-Original-To: patchwork@mira.cbaines.net Delivered-To: patchwork@mira.cbaines.net Received: by mira.cbaines.net (Postfix, from userid 113) id F2E1327BBEA; Tue, 18 Jul 2023 15:39:24 +0100 (BST) X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on mira.cbaines.net X-Spam-Level: X-Spam-Status: No, score=-3.7 required=5.0 tests=BAYES_00,DKIM_INVALID, DKIM_SIGNED,MAILING_LIST_MULTI,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS, URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6 Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mira.cbaines.net (Postfix) with ESMTPS id 5F00727BBE2 for <patchwork@mira.cbaines.net>; Tue, 18 Jul 2023 15:39:23 +0100 (BST) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from <guix-patches-bounces@gnu.org>) id 1qLlqy-0005SK-5u; Tue, 18 Jul 2023 10:39:04 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from <Debian-debbugs@debbugs.gnu.org>) id 1qLlqx-0005Rm-20 for guix-patches@gnu.org; Tue, 18 Jul 2023 10:39:03 -0400 Received: from debbugs.gnu.org ([2001:470:142:5::43]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from <Debian-debbugs@debbugs.gnu.org>) id 1qLlqw-0008NF-Q9 for guix-patches@gnu.org; Tue, 18 Jul 2023 10:39:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from <Debian-debbugs@debbugs.gnu.org>) id 1qLlqw-0006Rq-Lw for guix-patches@gnu.org; Tue, 18 Jul 2023 10:39:02 -0400 X-Loop: help-debbugs@gnu.org Subject: [bug#64711] [PATCH 00/45] Fix builds and skip failing tests for the Hurd. Resent-From: Janneke Nieuwenhuizen <janneke@gnu.org> Original-Sender: "Debbugs-submit" <debbugs-submit-bounces@debbugs.gnu.org> Resent-CC: guix-patches@gnu.org Resent-Date: Tue, 18 Jul 2023 14:39:02 +0000 Resent-Message-ID: <handler.64711.B.168969110924740@debbugs.gnu.org> Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: report 64711 X-GNU-PR-Package: guix-patches X-GNU-PR-Keywords: patch To: 64711@debbugs.gnu.org X-Debbugs-Original-To: guix-patches@gnu.org Received: via spool by submit@debbugs.gnu.org id=B.168969110924740 (code B ref -1); Tue, 18 Jul 2023 14:39:02 +0000 Received: (at submit) by debbugs.gnu.org; 18 Jul 2023 14:38:29 +0000 Received: from localhost ([127.0.0.1]:53705 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from <debbugs-submit-bounces@debbugs.gnu.org>) id 1qLlqP-0006Qx-6H for submit@debbugs.gnu.org; Tue, 18 Jul 2023 10:38:29 -0400 Received: from lists.gnu.org ([2001:470:142::17]:60198) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from <janneke@gnu.org>) id 1qLlqK-0006Qe-D5 for submit@debbugs.gnu.org; Tue, 18 Jul 2023 10:38:27 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from <janneke@gnu.org>) id 1qLlqD-0005K9-U1 for guix-patches@gnu.org; Tue, 18 Jul 2023 10:38:17 -0400 Received: from fencepost.gnu.org ([2001:470:142:3::e]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from <janneke@gnu.org>) id 1qLlqC-0008Hw-O3; Tue, 18 Jul 2023 10:38:16 -0400 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=gnu.org; s=fencepost-gnu-org; h=MIME-Version:Date:Subject:To:From:in-reply-to: references; bh=zjeA7GExngr1MBTDkvIC3gSNj+gei+HabS4SK1Yq2NY=; b=Zd3ZEIaAzOqWEH SIhYt1o9snW3Mof3J31V+r/VfCznjauw4LHW6nQ4iBd9AOdR7/UEhcC5gZp5568iNWc2EaEbu3HGQ /eFyKdPqGile6tYHi9Nw+WaTmCkCZNgziPHjkpKaaamJXa1WBS9L5KxLQ+M+JBZwwpFblcq5Uyu2R c1ns0/RKLm+o2GupaMFa8/pFGhnKPobiBan9nasFqMowrG0oYW4+Oh96hcnymx87XCdfIDjpfscaC UEQXLly5q039FBFh/Oy58ls29HuNsY3/l9KgmN26CH618gjdWd3VRv5oLBdjJIfWYU0Z9nIfPQLaP 1grhyZHS4OjWZcHJF7Dg==; Received: from 2a02-a462-da03-1-aeb0-2f5e-e96a-dccb.fixed6.kpn.net ([2a02:a462:da03:1:aeb0:2f5e:e96a:dccb] helo=drakenpad.fritz.box) by fencepost.gnu.org with esmtpa (Exim 4.90_1) (envelope-from <janneke@gnu.org>) id 1qLlqB-0004EV-OW; Tue, 18 Jul 2023 10:38:16 -0400 From: Janneke Nieuwenhuizen <janneke@gnu.org> Date: Tue, 18 Jul 2023 16:38:12 +0200 Message-Id: <cover.1689690897.git.janneke@gnu.org> X-Mailer: git-send-email 2.40.1 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-BeenThere: guix-patches@gnu.org List-Id: <guix-patches.gnu.org> List-Unsubscribe: <https://lists.gnu.org/mailman/options/guix-patches>, <mailto:guix-patches-request@gnu.org?subject=unsubscribe> List-Archive: <https://lists.gnu.org/archive/html/guix-patches> List-Post: <mailto:guix-patches@gnu.org> List-Help: <mailto:guix-patches-request@gnu.org?subject=help> List-Subscribe: <https://lists.gnu.org/mailman/listinfo/guix-patches>, <mailto:guix-patches-request@gnu.org?subject=subscribe> Errors-To: guix-patches-bounces+patchwork=mira.cbaines.net@gnu.org Sender: guix-patches-bounces+patchwork=mira.cbaines.net@gnu.org X-getmail-retrieved-from-mailbox: Patches |
Series |
Fix builds and skip failing tests for the Hurd.
|
|
Message
Janneke Nieuwenhuizen
July 18, 2023, 2:38 p.m. UTC
Hi! Now that native builds work again on the Hurd, we need builds to succeed and test suites to pass. Sadly, quite some packages have several problems with their test suite. Notably e2fsprogs (+62), c-ares (+37), coreutils (+21), python (+17), elfutils (+12), cmake (+11), curl (+9). And then there's a problem with gnulib's stack tests that we see some recurrences of. Some good news, previous commits that skipped tests for sed and libunistring can now be reverted, because their test suites pass. Also, most probably problematic tests in the guile test suite were fixed on guile master. As I understand, Josselin is working on this, so way may want to skip the patch for guile (or apply it anyway and revert it later). Because many patches are almost trivial and similar, you may want to give some specific comments to the non-trivial patches and possibly some general comments for the trivial ones instead of a LGTM on every individual patch? Greetings, Janneke Janneke Nieuwenhuizen (43): Revert "gnu: libunistring: Fix make check for the Hurd." gnu: guile: Skip hanging and failing pipe tests on the Hurd. gnu: coreutils: Skip hanging and failing test on the Hurd. gnu: grep: Update hanging and failing tests on the Hurd. Revert "gnu: sed: Skip failing test on GNU/Hurd." gnu: findutils: Move test-strerror_r from XFAIL to skip on the Hurd. gnu: diffutils: Remove test-perror2 from XFAIL_TESTS on the Hurd. gnu: gettext-minimal: Remove XFAIL_TESTS for the Hurd. gnu: m4: Skip gnulib stack-overflow tests for the Hurd. gnu: openssl: Build fix for the Hurd. gnu: python: Support native build on the Hurd. gnu: mpfr: Skip failing test on the Hurd. gnu: elfutils: Skip failing tests on the Hurd. gnu: libbsd: Skip failing test on the Hurd. gnu: c-ares: Skip failing tests on the Hurd. gnu: libgpg-error: Skip failing test on the Hurd. gnu: libgcrypt: Skip hanging benchmark tests on the Hurd. gnu: tcl: Remove failing tests on the Hurd.. gnu: curl: Skip failing test on the Hurd. gnu: git: Skip failing test on the Hurd. gnu: emacs: Depend on libc-for-target. gnu: procps: Skip linux-version test for the Hurd. gnu: e2fsprogs: Skip failing tests on the Hurd. gnu: parted: Disable tests for the Hurd. gnu: libpaper: Disable tests for the Hurd. gnu: cairo: Support building for the Hurd. gnu: openssl-1.1: Fix shared build for the Hurd. gnu: ruby-2.6: Skip test on the Hurd. gnu: tcsh: Skip substitution tests on the Hurd. gnu: swig: Skip tests when building for the Hurd. gnu: zstd: Skip tests when building for the Hurd. gnu: cmake-bootstrap: Fix build for the Hurd. gnu: cmake-minimal: Skip tests on the Hurd. gnu: doxygen: Fix build for the Hurd. gnu: graphite2: Skip test on the Hurd. gnu: guile-git: Skip http proxy test on the Hurd. gnu: glib: Disable tests for the Hurd. gnu: ghostscript: Fix build for the Hurd. gnu: harfbuzz: Support build for the Hurd. gnu: pango: Support build for the Hurd. gnu: fontforge: Support build on the Hurd. gnu: po4a: Skip failing test on the Hurd. gnu: guile-2.0: Skip failing tests on the Hurd. gnu/local.mk | 5 +- gnu/packages/adns.scm | 68 +++++++++-- gnu/packages/base.scm | 101 +++++++++------- gnu/packages/cmake.scm | 48 +++++--- gnu/packages/compression.scm | 5 +- gnu/packages/curl.scm | 19 ++- gnu/packages/disk.scm | 1 + gnu/packages/documentation.scm | 12 +- gnu/packages/elf.scm | 23 ++++ gnu/packages/emacs.scm | 4 +- gnu/packages/fontutils.scm | 108 +++++++++++++----- gnu/packages/gettext.scm | 14 ++- gnu/packages/ghostscript.scm | 20 +++- gnu/packages/glib.scm | 3 +- gnu/packages/gnupg.scm | 92 +++++++++------ gnu/packages/gtk.scm | 39 ++++--- gnu/packages/guile.scm | 40 ++++++- gnu/packages/libbsd.scm | 12 +- gnu/packages/libunistring.scm | 8 +- gnu/packages/linux.scm | 85 +++++++++++++- gnu/packages/m4.scm | 12 +- gnu/packages/multiprecision.scm | 12 ++ gnu/packages/patches/doxygen-hurd.patch | 30 +++++ gnu/packages/patches/fontforge-hurd.patch | 82 +++++++++++++ .../patches/ghostscript-leptonica-hurd.patch | 78 +++++++++++++ gnu/packages/python.scm | 27 ++++- gnu/packages/ruby.scm | 9 +- gnu/packages/shells.scm | 9 +- gnu/packages/swig.scm | 3 + gnu/packages/tcl.scm | 10 +- gnu/packages/tls.scm | 18 ++- gnu/packages/version-control.scm | 11 +- 32 files changed, 820 insertions(+), 188 deletions(-) create mode 100644 gnu/packages/patches/doxygen-hurd.patch create mode 100644 gnu/packages/patches/fontforge-hurd.patch create mode 100644 gnu/packages/patches/ghostscript-leptonica-hurd.patch base-commit: 61835f33c6fee35457c7277e3216c10c15d0a391
Comments
Hey! Janneke Nieuwenhuizen <janneke@gnu.org> skribis: > Also, most probably problematic tests in the guile test suite were fixed on > guile master. As I understand, Josselin is working on this, so way may want > to skip the patch for guile (or apply it anyway and revert it later). > > Because many patches are almost trivial and similar, you may want to give some > specific comments to the non-trivial patches and possibly some general > comments for the trivial ones instead of a LGTM on every individual patch? So modulo the few comments I sent, I think this looks good. There’s a couple of things to check (which I haven’t do): • Make sure x86_64-linux derivations (for instance) are unchanged. That is, ‘./pre-inst-env guix build libreoffice -d’ should produce the same thing before and after this patch series, on x86_64-linux. (Pro tip: replace ‘-d’ with ‘-n’; if there are substitutes, it means it hasn’t changed. :-)) • Make sure ‘guix build sed --target=i586-pc-gnu’ still works. Probably the derivation should be unchanged, but maybe it’s OK if it changes (sometimes avoiding a cross-rebuild would require extra boilerplate that we’d rather not have). • And well, ‘guix build sed -s i586-gnu’ should work as well, but that’s the whole point of this series! Thanks a lot for all the work! Ludo’.
Ludovic Courtès writes: Hi, > Janneke Nieuwenhuizen <janneke@gnu.org> skribis: > >> Also, most probably problematic tests in the guile test suite were fixed on >> guile master. As I understand, Josselin is working on this, so way may want >> to skip the patch for guile (or apply it anyway and revert it later). >> >> Because many patches are almost trivial and similar, you may want to give some >> specific comments to the non-trivial patches and possibly some general >> comments for the trivial ones instead of a LGTM on every individual patch? > > So modulo the few comments I sent, I think this looks good. \o/ > There’s a couple of things to check (which I haven’t do): > > • Make sure x86_64-linux derivations (for instance) are unchanged. > That is, ‘./pre-inst-env guix build libreoffice -d’ should produce > the same thing before and after this patch series, on x86_64-linux. > (Pro tip: replace ‘-d’ with ‘-n’; if there are substitutes, it means > it hasn’t changed. :-)) Ah, that's a nice trick. Checked! What I did, was after developing each patch (only) check that x86_64 package didn't get rebuilt. > • Make sure ‘guix build sed --target=i586-pc-gnu’ still works. > Probably the derivation should be unchanged, but maybe it’s OK if it > changes (sometimes avoiding a cross-rebuild would require extra > boilerplate that we’d rather not have). Checked! (It changed, of course). > • And well, ‘guix build sed -s i586-gnu’ should work as well, but > that’s the whole point of this series! Checked! -- but here I cheated a bit, I've only checked this before rebasing. > Thanks a lot for all the work! Thank you (and other reviewers) for all the help! Pushed to master as 6547d3852b11966271535962110d9e68f50c95fc Janneke