Subject: Re: [bug#67903] [PATCH] gnu: libcgroup: Update to 3.1.0.

From: Zheng Junjie <[email-masked]>
Date: Fri, 05 Jan 2024 13:45:36 +0800

Hilton Chain <[email-masked]> writes:

> Hi,
>
> On Sun, 24 Dec 2023 13:27:13 +0800,
> 郑俊杰 wrote:
>>
>>
>>
>> > 在 2023年12月19日，22:51，Hilton Chain via Guix-patches via <[email-masked]> 写道：
>> >
>> > * gnu/packages/linux.scm (libcgroup): Update to 3.1.0.
>> > [arguments]<#:configure-flags>: Disable systemd.
>> >
>> > Change-Id: I67fe380ebdf91ca254b3365b7f237a3d9ad4f0a5
>> > ---
>> > gnu/packages/linux.scm | 7 ++++---
>> > 1 file changed, 4 insertions(+), 3 deletions(-)
>> >
>> > diff --git a/gnu/packages/linux.scm b/gnu/packages/linux.scm
>> > index 3ca756dead..c4c366e21f 100644
>> > --- a/gnu/packages/linux.scm
>> > +++ b/gnu/packages/linux.scm
>> > @@ -8798,7 +8798,7 @@ (define-public fbgrab
>> > (define-public libcgroup
>> >   (package
>> >     (name "libcgroup")
>> > -    (version "2.0.2")
>> > +    (version "3.1.0")
>> >     (home-page "https://github.com/libcgroup/libcgroup")
>> >     (source
>> >      (origin
>> > @@ -8806,14 +8806,15 @@ (define-public libcgroup
>> >        (uri (string-append home-page "/releases/download/v"
>> >                            version "/" name "-" version ".tar.gz"))
>>
>> Can we switch to git-fetch?
>
> Is there any good reason?

if use git-fetch, allow `guix build libcgroup
--with-{branch,commit,git-url} ...` work.

>
>> And don’t use home-page in source.
>
> No problem.
>
>> >        (sha256
>> > -        (base32 "1y0c9ncsawamj77raiw6qkbm5cdsyvhjb2mvgma1kxmgw0r3pxlf"))))
>> > +        (base32 "0n0jkvmagw14vgwx3j5b6vv5h25lasrg2a7xihq9h11ww2qw8vlp"))))
>> >     (build-system gnu-build-system)
>> >     (arguments
>> >      ;; Tests are virtualized with lxc, it is not very feasible
>> >      ;; to make them executable under guix build. Also, note that
>> >      ;; origin is using source tarball release which is prepared
>> >      ;; after testing.
>> > -     `(#:tests? #f))
>> > +     `(#:tests? #f
>> > +       #:configure-flags '("--disable-systemd")))
>>
>> Maybe use gexp.
>
> Will add in a later commit.
>
>> >     (native-inputs
>> >      (list bison flex))
>> >     (inputs
>> >
>> > base-commit: fa92a7c2fe0c5b9e9923ad5f53bafd41aa6dbe66
>> > --
>> > 2.41.0
>>
>> Other LGTM
>
> Thanks for the review!