guix-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug#35518] gnu: guile-pfds bugfix


From: Ludovic Courtès
Subject: [bug#35518] gnu: guile-pfds bugfix
Date: Mon, 06 May 2019 10:05:52 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/26.2 (gnu/linux)

Hi Amirouche,

address@hidden skribis:

> On 2019-05-01 08:20, Ricardo Wurmus wrote:
>>
>> I’d prefer not to add a patch file here but to add a build phase after
>> unpack that uses substitute* to patch the file.
>>
>
> new patch
>
> From de22b734f37c2701121b990fe1fda5ead67ece31 Mon Sep 17 00:00:00 2001
> From: Amirouche <address@hidden>
> Date: Wed, 1 May 2019 13:41:59 +0200
> Subject: [PATCH] gnu: guile-pfds bugfix
>
> - gnu/packages/guile-xyz.scm(guile-pfds)[version]: upgrade to 0.3-0
> - gnu/packages/guile-xyz.scm(guile-pfds)[commit]: hardcode v0.3
> - gnu/packages/guile-xyz.scm(guile-pfds)[arguments]: New phase 'patch'

[...]

> +                  (add-after 'unpack 'patch
> +                    (lambda _
> +                      (substitute* "hamts.sls"
> +                        (("subtrie-vector vector")
> +                         "subtrie-vector trie"))
> +                        #t))

Like Ricardo wrote: “there should be a note about the upstream status of
this change.”  Someone reading this cannot understand what this change
is about nor what upstream thinks about it.

Could you address this issue?

Bonus points if you change it to follow our conventions for the commit
log and for the Git checkouts.  :-)

  https://www.gnu.org/software/guix/manual/en/html_node/Submitting-Patches.html
  https://www.gnu.org/software/guix/manual/en/html_node/Version-Numbers.html

Thanks in advance,
Ludo’.





reply via email to

[Prev in Thread] Current Thread [Next in Thread]