[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: |
Maxim Cournoyer |
Subject: |
[bug#65866] [PATCH 0/8] Add built-in builder for Git checkouts |
Date: |
Mon, 25 Sep 2023 11:59:18 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux) |
Hello,
Ludovic Courtès <ludo@gnu.org> writes:
> Maxim Cournoyer <maxim.cournoyer@gmail.com> skribis:
>
>> Ludovic Courtès <ludo@gnu.org> writes:
>>
>>> * configure.ac: Check for ‘git’ and substitute ‘GIT’.
>>> * guix/config.scm.in (%git): New variable.
>>> * guix/self.scm (compiled-guix): Define ‘git’ and pass it to
>>> ‘make-config.scm’.
>>> (make-config.scm): Add #:git; emit a ‘%git’ variable.
>>> * doc/guix.texi (Requirements): Add it.
>>
>> I'm a bit confused; we *both* capture git from the build environment,
>> and reference git from the git-minimal Guix package -- why can't we
>> strictly rely on the captured Git from the environment?
>
> That’s because we have two build systems: Autotools and (guix self).
> It’s the same change semantically, but for each of these build systems.
Oof, thanks for explaining.
>> Nitpick: this commit should be ordered before the daemon changes that
>> requires it.
>
> I believe that’s the case.
Indeed, I should sort by subject in Gnus when reviewing to get the
correct ordering!
The series LGTM with the comments from Simon and myself in other replies
taken into account.
--
Thanks,
Maxim
- [bug#65866] [PATCH 0/8] Add built-in builder for Git checkouts, (continued)
[bug#65866] [PATCH 6/8] perform-git-download: Use the ‘git’ command captured at configure time., Ludovic Courtès, 2023/09/11
[bug#65866] [PATCH 8/8] tests: Assume ‘git’ is always available., Ludovic Courtès, 2023/09/11
[bug#65866] [PATCH 5/8] build: Add dependency on Git., Ludovic Courtès, 2023/09/11