From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mp10.migadu.com ([2001:41d0:2:bcc0::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by ms5.migadu.com with LMTPS id iEU4GJmA8mJRnwAAbAwnHQ (envelope-from ) for ; Tue, 09 Aug 2022 17:43:21 +0200 Received: from aspmx1.migadu.com ([2001:41d0:2:bcc0::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by mp10.migadu.com with LMTPS id IMkdF5mA8mK9LgEAG6o9tA (envelope-from ) for ; Tue, 09 Aug 2022 17:43:21 +0200 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 D9F0831F00 for ; Tue, 9 Aug 2022 17:43:20 +0200 (CEST) Received: from localhost ([::1]:52644 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1oLRO3-00016o-O8 for larch@yhetil.org; Tue, 09 Aug 2022 11:43:19 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:57144) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1oLRIw-0002Fo-J2 for guix-patches@gnu.org; Tue, 09 Aug 2022 11:38:02 -0400 Received: from debbugs.gnu.org ([209.51.188.43]:54766) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1oLRIw-0004Es-9h for guix-patches@gnu.org; Tue, 09 Aug 2022 11:38:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1oLRIv-0004JT-VE for guix-patches@gnu.org; Tue, 09 Aug 2022 11:38:01 -0400 X-Loop: help-debbugs@gnu.org Subject: [bug#56858] [PATCH] gnu: libcgroup: Update to 2.0.2. Resent-From: Ludovic =?UTF-8?Q?Court=C3=A8s?= Original-Sender: "Debbugs-submit" Resent-CC: guix-patches@gnu.org Resent-Date: Tue, 09 Aug 2022 15:38:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 56858 X-GNU-PR-Package: guix-patches X-GNU-PR-Keywords: patch To: "(" Cc: muradm , 56858@debbugs.gnu.org Received: via spool by 56858-submit@debbugs.gnu.org id=B56858.166005943916528 (code B ref 56858); Tue, 09 Aug 2022 15:38:01 +0000 Received: (at 56858) by debbugs.gnu.org; 9 Aug 2022 15:37:19 +0000 Received: from localhost ([127.0.0.1]:44515 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1oLRI9-0004IQ-U4 for submit@debbugs.gnu.org; Tue, 09 Aug 2022 11:37:19 -0400 Received: from eggs.gnu.org ([209.51.188.92]:44078) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1oLRI7-0004IE-OG for 56858@debbugs.gnu.org; Tue, 09 Aug 2022 11:37:12 -0400 Received: from fencepost.gnu.org ([2001:470:142:3::e]:40794) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1oLRI2-00047Y-2l; Tue, 09 Aug 2022 11:37:06 -0400 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=gnu.org; s=fencepost-gnu-org; h=MIME-Version:In-Reply-To:Date:References:Subject:To: From; bh=ga56tLovIV1LqrDpRLer1PonCpwLXYEwO7lt2V0/Cdg=; b=LDWRxyEmlElvYd2nSmVC LaVTL2nYFDpPPI25Z6RVRho6eu7n6Ax1VW2Aro7+ZIPYaSHaEc0/wGYwBh58Q0cHe4PTwiE/1+WV8 YNORudej97bQAo48gaXK7PttO8u797SGWAz9wSp5lOTLafEGUHt83r/4YvlbWyhG5g30n+vpPEkRx rbX4u4IKeJcIbJdYimVdTQND8qSDtg0vnVGGUqs7TRSIGplj/otO/RmSvfggT92JP5gGn4rzkp4OH jsZbyLiqhnkcOZF4Ox/SZqMWYfuQO4Utjj/jkU1cUxDP68oP/q0H+uUnnl/RvmTywtkhM4DX5v34x 61N3mTDdY5Kv3A==; Received: from [193.50.110.201] (port=56142 helo=ribbon) by fencepost.gnu.org with esmtpsa (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1oLRI1-00044l-IB; Tue, 09 Aug 2022 11:37:05 -0400 From: Ludovic =?UTF-8?Q?Court=C3=A8s?= References: <20220731152938.28770-1-mail@muradm.net> <87sfmgplez.fsf@muradm.net> Date: Tue, 09 Aug 2022 17:37:02 +0200 In-Reply-To: (paren@disroot.org's message of "Mon, 01 Aug 2022 10:02:30 +0100") Message-ID: <87bkst5syp.fsf_-_@gnu.org> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/28.1 (gnu/linux) MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="=-=-=" 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" X-Migadu-Flow: FLOW_IN X-Migadu-To: larch@yhetil.org X-Migadu-Country: US ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=yhetil.org; s=key1; t=1660059801; h=from:from:sender:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc: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=ga56tLovIV1LqrDpRLer1PonCpwLXYEwO7lt2V0/Cdg=; b=gpUFkq6NWGT0q9f6wx6o/jQeHZ7f88mj62ebepvjwZjLCj6EiXrw7IKF53uWRk2joCjfN7 J2vwRaOK0Sxrc0z0uDx1SVFdjTxT4KEXJceQa5XxUO36LMPHp0BMS7dEezjvpr5YLa1N8G r0tZ8aeMg7Ni2c7hlUO3SEe+3NG/7VNFJL/yXYJGtmQtyu7QE6jCyfPoFBVo+NjJmx/yvW kapATIlj0/hrmeI71KGSHkGXiCXVh2tKn+1a/TTWgqwJzV9xQS2JK5YKNApE7Xfs9ZOIHH I+hXkm3C11GL1RYKprmPbcdHULtKhnNonPqBhKAm1LEeg+pqkEt5MxOarFbKGw== ARC-Seal: i=1; s=key1; d=yhetil.org; t=1660059801; a=rsa-sha256; cv=none; b=peHrGBP/5hZwNwM8YhXf1S6Ijf0OQCcqBIIIqhFnF5KS5hpVlo4FBYHyQBFlPCARHkqVwk i74IZnqkk9zu8ttR+MmeQMjzVJZJuWh0kJeEZf+/OmfinKe80bYjpEgkOKJIfDNl5ku5fW 9G0NwW1pXoV9joKORP5dzt/95sgpptd00ddJr05yN+5Ad3o9bDtv/DepVut6ltc6eltv3b +V2G9BbMto4J7N6k4RLb+kFnFL28HqtcvpU+3/bNBLtP5pgMfh++lot1jpEsN7XRdyEqe/ uX+YkN0daKvVn/KgKSUWkZsIv5us3rF8Ogh7KAcOPMdtEPGm2w6fYFdX9cc90Q== ARC-Authentication-Results: i=1; aspmx1.migadu.com; dkim=fail ("headers rsa verify failed") header.d=gnu.org header.s=fencepost-gnu-org header.b=LDWRxyEm; 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: -3.69 Authentication-Results: aspmx1.migadu.com; dkim=fail ("headers rsa verify failed") header.d=gnu.org header.s=fencepost-gnu-org header.b=LDWRxyEm; 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: D9F0831F00 X-Spam-Score: -3.69 X-Migadu-Scanner: scn0.migadu.com X-TUID: rWtecT+b7EwJ --=-=-= Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Hi, "(" skribis: > On Mon Aug 1, 2022 at 6:42 AM BST, muradm wrote: >> Is there a guideline where home-page should be? I find it more >> DRY when reusing. > Almost every package in Guix puts it directly above synopsis, though > you do have a point about DRY. Yeah IMO it=E2=80=99s a good idea to reuse =E2=80=98home-page=E2=80=99 like= this patch does. >> Their repo is not suitable for building from, instead >> they release sources manually. > Looks okay to me: > But there is a problem with vendoring googletest, it seems. You'll want > to use the system's googletest, and (recursive? #t) in the git-reference > to clone the tests repo. Also, the tarball seems to contain pregenerated > autotools files, which are discouraged by Guix. So, first things first. The priority should be to run tests and remove the bundled googletest, which I tried with the patch below. Now, tests won=E2=80=99t run because apparently they try to use =E2=80=98su= do=E2=80=99 (!): --8<---------------cut here---------------start------------->8--- Traceback (most recent call last): File "/tmp/guix-build-libcgroup-2.0.2.drv-0/libcgroup-2.0.2/tests/ftests/= ./ftests.py", line 353, in sys.exit(main(config)) File "/tmp/guix-build-libcgroup-2.0.2.drv-0/libcgroup-2.0.2/tests/ftests/= ./ftests.py", line 337, in main setup(config, record_time=3DTrue) File "/tmp/guix-build-libcgroup-2.0.2.drv-0/libcgroup-2.0.2/tests/ftests/= ./ftests.py", line 170, in setup Run.run(['sudo', 'lxd', 'init', '--auto']) File "/tmp/guix-build-libcgroup-2.0.2.drv-0/libcgroup-2.0.2/tests/ftests/= run.py", line 38, in run subproc =3D subprocess.Popen(command, shell=3Dshell_bool, File "/gnu/store/65i3nhcwmz0p8rqbg48gaavyky4g4hwk-python-3.9.9/lib/python= 3.9/subprocess.py", line 951, in __init__ self._execute_child(args, executable, preexec_fn, close_fds, File "/gnu/store/65i3nhcwmz0p8rqbg48gaavyky4g4hwk-python-3.9.9/lib/python= 3.9/subprocess.py", line 1821, in _execute_child raise child_exception_type(errno_num, err_msg, err_filename) FileNotFoundError: [Errno 2] No such file or directory: 'sudo' FAIL ftests.sh (exit status: 1) --8<---------------cut here---------------end--------------->8--- Could you check whether they can run at all without root privileges, and if not, document it in a comment above #:tests? #f? Thanks, Ludo=E2=80=99. --=-=-= Content-Type: text/x-patch Content-Disposition: inline diff --git a/gnu/packages/linux.scm b/gnu/packages/linux.scm index e70d8d1a90..7eb309dfa5 100644 --- a/gnu/packages/linux.scm +++ b/gnu/packages/linux.scm @@ -8177,14 +8177,18 @@ (define-public libcgroup (uri (string-append home-page "/releases/download/v" version "/" name "-" version ".tar.gz")) (sha256 - (base32 "1y0c9ncsawamj77raiw6qkbm5cdsyvhjb2mvgma1kxmgw0r3pxlf")))) + (base32 "1y0c9ncsawamj77raiw6qkbm5cdsyvhjb2mvgma1kxmgw0r3pxlf")) + (modules '((guix build utils))) + (snippet + ;; Delete the bundled copy of googletest, that even includes a .so. + '(delete-file-recursively "googletest")))) (build-system gnu-build-system) - (arguments - `(#:tests? #f)) + ;; (arguments + ;; `(#:tests? #f)) (native-inputs - (list bison flex)) + (list bison flex python-wrapper)) (inputs - (list linux-pam)) + (list linux-pam googletest)) (synopsis "Control groups management tools") (description "Control groups is Linux kernel method for process resource restriction, permission handling and more. This package provides userspace --=-=-=--