[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#65866] [PATCH 0/8] Add built-in builder for Git checkouts
From: |
Simon Tournier |
Subject: |
[bug#65866] [PATCH 0/8] Add built-in builder for Git checkouts |
Date: |
Mon, 25 Sep 2023 14:37:17 +0200 |
Hi,
On Mon, 25 Sep 2023 at 11:23, Ludovic Courtès <ludo@gnu.org> wrote:
> Simon Tournier <zimon.toutoune@gmail.com> skribis:
>
>> On Sat, 23 Sep 2023 at 00:28, Ludovic Courtès <ludo@gnu.org> wrote:
>>
>>> +(define* (git-fetch/built-in ref hash-algo hash
>>> + #:optional name
>>> + #:key (system (%current-system)))
>>> + "Return a fixed-output derivation without any dependency that performs a
>>> Git
>>> +checkout of REF, using the \"builtin:git-download\" derivation builder."
>>
>> I do not understand what means “without any dependency” here.
>
> It means the derivation does not depend on anything (it has zero
> “sources” and zero “input derivations”). That’s fundamental here, which
> is why I made it explicit.
Aaah :-) Yeah that makes sense.
Well, from my understanding, « fixed-output derivation without any
dependency » is not clear at first (we are always dependent on something
else ;-)) and I am sure that weeks or months later, I will not remember
that it means: « zero “sources” and zero “input derivations” ».
Hum, once it is known, it is hard to find a better wording though. ;-)
I do not know if it is better, at least, it appears clearer from my
point of view:
"Return a fixed-output derivation that performs a Git checkout of REF,
using the \"builtin:git-download\" derivation builder, thus without any
dependency other than from builtin."
Cheers,
simon
- [bug#65866] [PATCH v2 8/8] tests: Assume ‘git’ is always available., (continued)
- [bug#65866] [PATCH v2 8/8] tests: Assume ‘git’ is always available., Ludovic Courtès, 2023/09/22
- [bug#65866] [PATCH v2 6/8] perform-download: Use the ‘git’ command captured at configure time., Ludovic Courtès, 2023/09/22
- [bug#65866] [PATCH v2 4/8] daemon: Add “git-download” built-in builder., Ludovic Courtès, 2023/09/22
- [bug#65866] [PATCH v2 7/8] git-download: Use “builtin:git-download” when available., Ludovic Courtès, 2023/09/22
- [bug#65866] [PATCH v2 7/8] git-download: Use “builtin:git-download” when available., Simon Tournier, 2023/09/25
- [bug#65866] [PATCH 0/8] Add built-in builder for Git checkouts, Ludovic Courtès, 2023/09/25
- [bug#65866] [PATCH 0/8] Add built-in builder for Git checkouts, Simon Tournier, 2023/09/25
- [bug#65866] [PATCH 0/8] Add built-in builder for Git checkouts, Maxim Cournoyer, 2023/09/25
- bug#65866: [PATCH 0/8] Add built-in builder for Git checkouts, Ludovic Courtès, 2023/09/26
- [bug#65866] [PATCH 0/8] Add built-in builder for Git checkouts, Simon Tournier, 2023/09/26
- [bug#65866] [PATCH 0/8] Add built-in builder for Git checkouts,
Simon Tournier <=
- [bug#65866] [PATCH v2 5/8] build: Add dependency on Git., Ludovic Courtès, 2023/09/22
- [bug#65866] [PATCH v2 5/8] build: Add dependency on Git., Simon Tournier, 2023/09/25
- [bug#65866] Bootstrapping without the daemon and all that, Ludovic Courtès, 2023/09/26
- [bug#65866] Bootstrapping without the daemon and all that, Simon Tournier, 2023/09/26
[bug#65866] [PATCH 2/8] git-download: Honor the ‘GUIX_DOWNLOAD_FALLBACK_TEST’ environment variable., Ludovic Courtès, 2023/09/11
[bug#65866] [PATCH 3/8] perform-download: Remove unused one-argument clause., Ludovic Courtès, 2023/09/11
[bug#65866] [PATCH 1/8] git-download: Move fallback code to (guix build git)., Ludovic Courtès, 2023/09/11