Subject: Re: [bug#64746] [PATCH v2 2/3] pull: Tag commit argument with 'tag-or-commit.

From: Simon Tournier <[email-masked]>
Date: Thu, 17 Aug 2023 16:45:45 +0200

Hi Maxim,

On Wed, 16 Aug 2023 at 14:47, Maxim Cournoyer <[email-masked]> wrote:

>>>           (option '("commit") #t #f
>>>                   (lambda (opt name arg result)
>>> -                   (alist-cons 'ref `(commit . ,arg) result)))
>>> +                   (alist-cons 'ref `(tag-or-commit . ,arg) result)))

[...]

>                       (match ref
> -                       (('commit . commit)
> +                       ((or ('commit . commit)
> +                            ('tag-or-commit . commit))

> The reason is to standardize the API of (guix pull) and (guix git),
> whose procedure had a different expectation for 'ref' objects.

My point is that this ’or’ is useless, IIUC.  Well, I have removed it in
the series fixing the annoyance with the network access of “guix
time-machine”.

Cheers,
simon