GNU bug report logs - #67755
[PATCH] gnu: Add imhex.

Previous Next

Full log

View this message in rfc822 format

From: Sergio Pastor Pérez <sergio.pastorperez <at> outlook.es>
To: Sergio Pastor Pérez <liliana.prikler <at> gmail.com>, 67755 <at> debbugs.gnu.org
Subject: [bug#67755] [PATCH v6 7/8] gnu: Add imhex-pattern-language.
Date: Wed, 19 Mar 2025 21:25:45 +0100

On 3/19/25 18:00, Sergio Pastor Pérez wrote:
> * gnu/packages/hexedit.scm (imhex-pattern-language): New variable.
> 
> Change-Id: I641bebed0bea9b2c8aca87f54d0fb457064a07d7
> ---
> Changes from v5:
> * Use system fmt (fmt-11).

This is fine.

> * Use a keep list rather than a delete list.

Looks good to me.

> I think the header-only libraries should be provided as regular inputs,
> but haven't done that yet.

As you think goes more inline with Guix philosophy.

> +(define-public imhex-pattern-language
> [...]
> +       (snippet #~(begin
> +                    ;; NOTE: the `throwing-ptr' hasn't been updated in 6 years
> +                    ;; and the testsuite expects to use an outdated version of

I wrote bad english here:
> +                    ;; Conan, since this library if not interesting to have it
> +                    ;; in Guix it will remain bundled.

It should probably be:
`since this library is not interesting for Guix, it will remain bundled.`


Thanks for your work!


Best regards,
Sergio.

Previous Next

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