GNU bug report logs -
#50072
[PATCH WIP 0/4] Add upstream updater for git-fetch origins.
Previous Next
Reported by: Sarah Morgensen <iskarian <at> mgsn.dev>
Date: Sun, 15 Aug 2021 23:17:02 UTC
Severity: normal
Tags: patch
Done: Ludovic Courtès <ludo <at> gnu.org>
Bug is archived. No further changes may be made.
Full log
View this message in rfc822 format
Hi Sarah,
I just noticed I hadn’t answered this message…
Sarah Morgensen <iskarian <at> mgsn.dev> skribis:
> Ludovic Courtès <ludo <at> gnu.org> writes:
[...]
>> Sarah Morgensen <iskarian <at> mgsn.dev> skribis:
>>
>>> * guix/git-download.scm (checkout-to-store): New procedure.
>>> * guix/upstream.scm (guess-version-transform)
>>> (package-update/git-fetch): New procedures.
>>> (%method-updates): Add GIT-FETCH mapping.
>>
>> This LGTM.
>
> Thanks. WDYT about pre-emptively adding support for non-url URIs as
> well? That is,
>
> 1. change "urls" in <upstream-source> to "uri"
>
> 2. in 'git-fetch'
>
> a) if the upstream-source-uri is a git-reference, just use it as-is
> rather than guessing the tag
>
> b) if it's not, return an 'upstream-source' with a git-reference URI
>
> 3. update 'upstream-source-compiler' to work for git-reference URIs.
>
> If there are no objections, I think I'll make those changes and send
> that as a proper patch.
That sounds like a good idea. We’ll need to check users of
‘upstream-source-urls’ & co. and see whether/how they can deal with
generalized “URIs”.
That said, perhaps it can come after this patch series, which I think
was mostly waiting on ‘package-definition-location’ initially?
Thanks,
Ludo’.
This bug report was last modified 3 years and 133 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.