From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mp11.migadu.com ([2001:41d0:2:bcc0::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by ms0.migadu.com with LMTPS id aCHFKeH63WGQYgAAgWs5BA (envelope-from ) for ; Tue, 11 Jan 2022 22:47:13 +0100 Received: from aspmx1.migadu.com ([2001:41d0:2:bcc0::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by mp11.migadu.com with LMTPS id CIwwJ+H63WGJtAAA9RJhRA (envelope-from ) for ; Tue, 11 Jan 2022 22:47:13 +0100 Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by aspmx1.migadu.com (Postfix) with ESMTPS id A15B83B9CF for ; Tue, 11 Jan 2022 22:47:12 +0100 (CET) Received: from localhost ([::1]:34276 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1n7Oz1-00020e-QY for larch@yhetil.org; Tue, 11 Jan 2022 16:47:11 -0500 Received: from eggs.gnu.org ([209.51.188.92]:40178) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1n7Oys-00020R-Id for guix-patches@gnu.org; Tue, 11 Jan 2022 16:47:02 -0500 Received: from debbugs.gnu.org ([209.51.188.43]:34639) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1n7Oys-00085W-20 for guix-patches@gnu.org; Tue, 11 Jan 2022 16:47:02 -0500 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1n7Oyr-0003ec-PB for guix-patches@gnu.org; Tue, 11 Jan 2022 16:47:01 -0500 X-Loop: help-debbugs@gnu.org Subject: [bug#53040] [PATCH] gnu: bubblewrap: Update to 0.5.0. Resent-From: kiasoc5@tutanota.com Original-Sender: "Debbugs-submit" Resent-CC: guix-patches@gnu.org Resent-Date: Tue, 11 Jan 2022 21:47:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 53040 X-GNU-PR-Package: guix-patches X-GNU-PR-Keywords: patch To: Ludovic =?UTF-8?Q?Court=C3=A8s?= , 53040@debbugs.gnu.org Received: via spool by 53040-submit@debbugs.gnu.org id=B53040.164193761614028 (code B ref 53040); Tue, 11 Jan 2022 21:47:01 +0000 Received: (at 53040) by debbugs.gnu.org; 11 Jan 2022 21:46:56 +0000 Received: from localhost ([127.0.0.1]:55775 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1n7Oyf-0003dx-1J for submit@debbugs.gnu.org; Tue, 11 Jan 2022 16:46:56 -0500 Received: from w1.tutanota.de ([81.3.6.162]:45244) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1n7Oyd-0003da-6Z for 53040@debbugs.gnu.org; Tue, 11 Jan 2022 16:46:48 -0500 Received: from w3.tutanota.de (unknown [192.168.1.164]) by w1.tutanota.de (Postfix) with ESMTP id F0622FBF8AD; Tue, 11 Jan 2022 21:46:40 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; t=1641937600; s=s1; d=tutanota.com; h=From:From:To:To:Subject:Subject:Content-Description:Content-ID:Content-Type:Content-Type:Content-Transfer-Encoding:Cc:Date:Date:In-Reply-To:In-Reply-To:MIME-Version:MIME-Version:Message-ID:Message-ID:Reply-To:References:References:Sender; bh=6Sk4vrZrSg1u2YumKNlGGZxyQKVj5gGP3own9F/lIQc=; b=eUw2VTWNJx3qNX3tNJX5ioSWQrfEdCwTZZSrO0NWHls/adpVHKtOG5zx34KaHmJ4 5o6zKUlhuZcXDlsUFjSqhNrxU0fKaZLmZ1BByoYyRjhjsFw6FcUOyNt8h3Op0EHRVe0 j598Wxf8aJ81ccdNY3eyrlzaPvf+Aw5wT6lMb9NwSruZIEAPsI4PTyu4FDEO4w3XBem KG1hOzp+VfvuUemg+0OL4IPLl70XMOzjJs/X80cE5QUIyrfD+n2CtPepvpN6zjeNEWK gXt52BAV4QLUfT/nc9X7AqPwjEki2tgQPAcPDHpFgb7GbDBV8abVH/bHLV+FbGgpMwt FUT8McemwQ== Date: Tue, 11 Jan 2022 22:46:40 +0100 (CET) Message-ID: In-Reply-To: <87iluqtb11.fsf_-_@gnu.org> References: <87iluqtb11.fsf_-_@gnu.org> MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="----=_Part_460699_1100851437.1641937600975" X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-BeenThere: guix-patches@gnu.org List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+larch=yhetil.org@gnu.org Sender: "Guix-patches" Reply-to: kiasoc5@tutanota.com X-ACL-Warn: , kiasoc5--- via Guix-patches From: kiasoc5--- via Guix-patches via X-Migadu-Flow: FLOW_IN X-Migadu-Country: US ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=yhetil.org; s=key1; t=1641937632; h=from:from:sender:sender:reply-to:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:mime-version:mime-version: content-type:content-type:resent-cc:resent-from:resent-sender: resent-message-id:in-reply-to:in-reply-to:references:references: list-id:list-help:list-unsubscribe:list-subscribe:list-post: dkim-signature; bh=6Sk4vrZrSg1u2YumKNlGGZxyQKVj5gGP3own9F/lIQc=; b=X7tWajwECVxjSEZm3mN5j14+wlNAtH4Q6xxWX8isn1EzefizxBa41x1Zzai6H/ip8gcTya /0LLqYmaQJi/tVvyJpKp8kl8nE2gqAr1oQEUnyoDbObB1HJC4ZP1gLKRA0fh5e1ayV7I/j yLcUnZTJ6JvJHRWcusAjOA7xOcThReVdTorcsXWP/n12eF8s72M8I7NHIB6/jAL9apbl3C QzVgbS8//VJZAIdzWfD8As4QAQ7+cXZ5b5uxp8R/GO6hQ7nqE8nd/2oLwVt5/scIMMnvng ysf+8G33n0Z3i7tQO8oNoBsderd8JQh/52Ioh+a3INEwdl9/TBES/+xkkmyOpg== ARC-Seal: i=1; s=key1; d=yhetil.org; t=1641937632; a=rsa-sha256; cv=none; b=a1BBSAyzgWe+HhfqemWGpSyZcqBxaEqiuMMO7pHTPHevpY+OOAW3UpOHnZjDt8NhHIYIjf DvYE7yRDw+1Y3DC+ZWHVeyzTLAVYRBj0mbIZOCZ6IfV9zAY675xNQgN8aHLqpZlOroq4BT sTbPbkNlDmPCxd39lSwJxj4vp3b08Ps3GuasN+IwzpKIYpd9B68T1L1xNkySQy0Y0M6KTz m4rGqe5rW2KsEZ3hnEbtJx5AHl7xpA1+ZOVQrZ5gPXr3hfDvw5cgsFTKcYdWdansTFvxmV OcWGMkGpRl3VWx0jzBkLplmOcLpbtFzDLllkSTmrvlwMSItUpDY5YUyguayATg== ARC-Authentication-Results: i=1; aspmx1.migadu.com; dkim=fail ("headers rsa verify failed") header.d=tutanota.com header.s=s1 header.b=eUw2VTWN; dmarc=pass (policy=none) header.from=gnu.org; spf=pass (aspmx1.migadu.com: domain of "guix-patches-bounces+larch=yhetil.org@gnu.org" designates 209.51.188.17 as permitted sender) smtp.mailfrom="guix-patches-bounces+larch=yhetil.org@gnu.org" X-Migadu-Spam-Score: -4.22 Authentication-Results: aspmx1.migadu.com; dkim=fail ("headers rsa verify failed") header.d=tutanota.com header.s=s1 header.b=eUw2VTWN; dmarc=pass (policy=none) header.from=gnu.org; spf=pass (aspmx1.migadu.com: domain of "guix-patches-bounces+larch=yhetil.org@gnu.org" designates 209.51.188.17 as permitted sender) smtp.mailfrom="guix-patches-bounces+larch=yhetil.org@gnu.org" X-Migadu-Queue-Id: A15B83B9CF X-Spam-Score: -4.22 X-Migadu-Scanner: scn0.migadu.com X-TUID: rifT5WBO4XQo ------=_Part_460699_1100851437.1641937600975 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable I rebased the patch onto today's Guix commits, and Flatpak 1.12.1 and 1.12.= 2 passes the tests on my machine. But I'm currently running Guix on Arch an= d don't have a Guix system available to test further. All dependent packages except wpewebkit build successfully. wpewebkit build= log complains during cmake configuration that a required package is not fo= und, but it seems unrelated to bubblewrap, and it won't build on master any= ways. Jan 11, 2022, 17:52 by ludo@gnu.org: > Hi, > > kiasoc5@tutanota.com skribis: > >> Fixed locale in tests, thanks Ludo for the suggestion via email. Also fi= xed more test substitutions and simplified inputs. >> > >>From f5c56100d62d315c3ed860898998a7c8afc78d50 Mon Sep 17 00:00:00 2001 > >> From: kiasoc5 >> Date: Mon, 10 Jan 2022 17:58:35 -0500 >> Subject: [PATCH] gnu: bubblewrap: Update to 0.5.0. >> >> * gnu/packages/patches/bubblewrap-fix-locale-in-tests.patch: Add it. >> * gnu/local.mk (dist_patch_DATA): Add it. >> * gnu/packages/virtualization.scm (bubblewrap): Update to 0.5.0. >> [source]: Add patch. >> [arguments]: Adjust test substitutions. >> [inputs]: Remove labels. >> [native-inputs]: Remove labels. >> > > That LGTM. I tested direct dependents as shown by: > > ./pre-inst-env guix graph -M1 -t reverse-package bubblewrap | xdot -f fdp= - > > Unfortunately, the update breaks flatpak (both 1.12.1 and 1.12.2): > > --8<---------------cut here---------------start------------->8--- > make[3]: Nothing to be done for 'tests/org.flatpak.Authenticator.test.ser= vice.in'. > rm -rf tests/runtime-repo > PATH=3D/tmp/guix-build-flatpak-1.12.2.drv-0/flatpak-1.12.2:${PATH} ./test= s/make-test-runtime.sh tests/runtime-repo org.test.Platform master "" "" > guix > error: Writing content object: min-free-space-percent '3%' would be excee= ded, at least 36.9=C2=A0kB requested > make[3]: *** [Makefile:7804: tests/runtime-repo] Error 1 > make[3]: Leaving directory '/tmp/guix-build-flatpak-1.12.2.drv-0/flatpak-= 1.12.2' > make[2]: *** [Makefile:7063: check-am] Error 2 > make[2]: Leaving directory '/tmp/guix-build-flatpak-1.12.2.drv-0/flatpak-= 1.12.2' > make[1]: *** [Makefile:6255: check-recursive] Error 1 > make[1]: Leaving directory '/tmp/guix-build-flatpak-1.12.2.drv-0/flatpak-= 1.12.2' > make: *** [Makefile:7067: check] Error 2 > error: in phase 'check': uncaught exception: > %exception #<&invoke-error program: "make" arguments: ("check" "TESTS=3Dt= ests/test-basic.sh tests/test-config.sh testcommon") exit-status: 2 term-si= gnal: #f stop-signal: #f>=20 > phase `check' failed after 17.2 seconds > command "make" "check" "TESTS=3Dtests/test-basic.sh tests/test-config.sh = testcommon" failed with status 2 > --8<---------------cut here---------------end--------------->8--- > > Could you take a look? > > Apologies for the extra work! > > Thanks, > Ludo=E2=80=99. > ------=_Part_460699_1100851437.1641937600975 Content-Type: text/x-patch; charset=UTF-8; name=0001-gnu-bubblewrap-Update-to-0.5.0.patch Content-Transfer-Encoding: quoted-printable Content-Disposition: attachment; filename=0001-gnu-bubblewrap-Update-to-0.5.0.patch >From dc1d48162f3d9fdc2647cbbd4ed2b8eb57eca08f Mon Sep 17 00:00:00 2001 From: kiasoc5 Date: Mon, 10 Jan 2022 17:58:35 -0500 Subject: [PATCH] gnu: bubblewrap: Update to 0.5.0. * gnu/packages/patches/bubblewrap-fix-locale-in-tests.patch: Add it. * gnu/local.mk (dist_patch_DATA): Add it. * gnu/packages/virtualization.scm (bubblewrap): Update to 0.5.0. [source]: Add patch. [arguments]: Adjust test substitutions. [inputs]: Remove labels. [native-inputs]: Remove labels. --- gnu/local.mk | 1 + .../bubblewrap-fix-locale-in-tests.patch | 34 +++++++++++++++++++ gnu/packages/virtualization.scm | 24 ++++++++----- 3 files changed, 50 insertions(+), 9 deletions(-) create mode 100644 gnu/packages/patches/bubblewrap-fix-locale-in-tests.pat= ch diff --git a/gnu/local.mk b/gnu/local.mk index 7c75a1043b..756caeea44 100644 --- a/gnu/local.mk +++ b/gnu/local.mk @@ -911,6 +911,7 @@ dist_patch_DATA =3D=09=09=09=09=09=09\ %D%/packages/patches/binutils-mingw-w64-deterministic.patch=09\ %D%/packages/patches/bpftrace-disable-bfd-disasm.patch=09\ %D%/packages/patches/byobu-writable-status.patch=09=09\ + %D%/packages/patches/bubblewrap-fix-locale-in-tests.patch=09\ %D%/packages/patches/cabal-install-base16-bytestring1.0.patch=09=09\ %D%/packages/patches/cabal-install-ghc8.10.patch=09=09\ %D%/packages/patches/cairo-CVE-2018-19876.patch=09=09\ diff --git a/gnu/packages/patches/bubblewrap-fix-locale-in-tests.patch b/gn= u/packages/patches/bubblewrap-fix-locale-in-tests.patch new file mode 100644 index 0000000000..bd5924ef87 --- /dev/null +++ b/gnu/packages/patches/bubblewrap-fix-locale-in-tests.patch @@ -0,0 +1,34 @@ +From 9282223de4b511aeda3b7d2caf3810a56a865710 Mon Sep 17 00:00:00 2001 +From: kiasoc5 +Date: Mon, 10 Jan 2022 14:16:32 -0500 +Subject: [PATCH] Use C locale unconditionally for tests. +MIME-Version: 1.0 +Content-Type: text/plain; charset=3DUTF-8 +Content-Transfer-Encoding: 8bit + +The =E2=80=98en_US.utf8=E2=80=99 locale is available. However, the =E2=80= =98locale -a=E2=80=99 command won=E2=80=99t list it. +--- + tests/libtest-core.sh | 7 +------ + 1 file changed, 1 insertion(+), 6 deletions(-) + +diff --git a/tests/libtest-core.sh b/tests/libtest-core.sh +index 9632e90..82951dd 100644 +--- a/tests/libtest-core.sh ++++ b/tests/libtest-core.sh +@@ -41,12 +41,7 @@ assert_not_reached () { + # + # If we can't find the locale command assume we have support for C.UTF-8 + # (e.g. musl based systems) +-if type -p locale >/dev/null; then +- export LC_ALL=3D$(locale -a | grep -iEe '^(C|en_US)\.(UTF-8|utf8)$' |= head -n1 || true) +- if [ -z "${LC_ALL}" ]; then fatal "Can't find suitable UTF-8 locale";= fi +-else +- export LC_ALL=3DC.UTF-8 +-fi ++export LC_ALL=3Den_US.utf8 + # A GNU extension, used whenever LC_ALL is not C + unset LANGUAGE +=20 +--=20 +2.34.1 + diff --git a/gnu/packages/virtualization.scm b/gnu/packages/virtualization.= scm index 82cc02b738..cec1bce5f3 100644 --- a/gnu/packages/virtualization.scm +++ b/gnu/packages/virtualization.scm @@ -1926,7 +1926,7 @@ (define-public python-vagrant (define-public bubblewrap (package (name "bubblewrap") - (version "0.4.1") + (version "0.5.0") (source (origin (method url-fetch) (uri (string-append "https://github.com/containers/bubblewra= p/" @@ -1934,7 +1934,8 @@ (define-public bubblewrap version ".tar.xz")) (sha256 (base32 - "00ycgi6q2yngh06bnz50wkvar6r2jnjf3j158grhi9k13jdrpimr")))) + "0608l2sjwhnb1c0mslah1h6yjvqr17wk60by6i710qwxg4rszz8n")) + (patches (search-patches "bubblewrap-fix-locale-in-tests.pa= tch")))) (build-system gnu-build-system) (arguments `(#:phases @@ -1950,7 +1951,9 @@ (define-public bubblewrap (substitute* "tests/test-run.sh" (("/var/tmp") tmp-dir) ;; Tests create a temporary python script, so fix its she= bang. - (("/usr/bin/env python") (which "python")) + (("/usr/bin/env python3") (which "python3")) + ;; Tests call /usr/bin/env, so fix its path. + (("/usr/bin/env") (which "env")) ;; Some tests try to access /usr, but that doesn't exist. ;; Give them /gnu instead. (("/usr") "/gnu") @@ -1959,18 +1962,21 @@ (define-public bubblewrap (("--ro-bind /lib /lib") "--ro-bind /gnu /lib") ((" */bin/bash") (which "bash")) (("/bin/sh") (which "sh")) - (("findmnt") (which "findmnt")))) + (("findmnt") (which "findmnt"))) + (substitute* "tests/libtest.sh" + (("/var/tmp") tmp-dir) + (("/usr") "/gnu") + (("--ro-bind /bin /bin") "--ro-bind /gnu /bin") + (("--ro-bind /sbin /sbin") "--ro-bind /gnu /sbin") + (("--ro-bind /lib /lib") "--ro-bind /gnu /lib"))) #t)) ;; Remove the directory we gave to tests to have a clean package. (add-after 'check 'remove-tmp-dir (lambda* (#:key outputs #:allow-other-keys) (delete-file-recursively (string-append (assoc-ref outputs "o= ut") "/tmp")) #t))))) - (inputs - (list libcap)) - (native-inputs - `(("python" ,python-wrapper) - ("util-linux" ,util-linux))) + (inputs (list libcap)) + (native-inputs (list python-wrapper util-linux)) (home-page "https://github.com/containers/bubblewrap") (synopsis "Unprivileged sandboxing tool") (description "Bubblewrap is aimed at running applications in a sandbox= , base-commit: 993bc8f06cc0662632618e49f6940d5036479c8b --=20 2.34.1 ------=_Part_460699_1100851437.1641937600975--