GNU bug report logs - #63545
[PATCH 0/3] gnu: heimdal: Drop obsolete user tools.

Previous Next

Full log

Message #17 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Josselin Poiret <dev <at> jpoiret.xyz>
To: Felix Lechner via Guix-patches via <guix-patches <at> gnu.org>,
 63545 <at> debbugs.gnu.org
Cc: Felix Lechner <felix.lechner <at> lease-up.com>
Subject: Re: [bug#63545] [PATCH 1/3] gnu: heimdal: Build from Git.
Date: Wed, 17 May 2023 22:20:19 +0200

[Message part 1 (text/plain, inline)]

Hi Felix,

I am no user of Heimdal, but this looks good to me apart from some
nitpicks below.

Felix Lechner via Guix-patches via <guix-patches <at> gnu.org> writes:

>  (define-public heimdal
> +  (let ((commit "a6cf94577c0d1e5bca5304342e4ddffb18255afe")
> +        (revision "1"))

No need to use commit+revision for fixed releases, unless upstream has
the bad habit of moving their upstream tags. I see the tag
heimdal-7.8.0, which you should use in (commit ...), possibly with
(commit (string-append "heimdall-" version)).

>    (package
>      (name "heimdal")
> -    (version "7.8.0")
> +    (version (git-version "7.8.0" revision commit))

This would lead to a very ugly version string for what is actually
7.8.0, no need to change this.

The other patches seem fine, although I think you don't need to go as
far as make the indentation change its own patch (and by dropping the
commit+revision thing, there shouldn't be one).

I haven't tried building this yet though, will let QA go over it.

Best,
-- 
Josselin Poiret

[signature.asc (application/pgp-signature, inline)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.