[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 3/3] gnu: Add vcsh
From: |
Christopher Baines |
Subject: |
Re: [PATCH 3/3] gnu: Add vcsh |
Date: |
Thu, 18 Feb 2016 21:04:11 +0000 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Icedove/38.6.0 |
On 18/02/16 20:01, Andreas Enge wrote:
> On Wed, Feb 17, 2016 at 07:11:44PM -0500, Leo Famulari wrote:
>> I missed this earlier. We don't usually propagate this sort of "end-user
>> program" dependencies. For example rsync does not propagate ssh; the
>> user must install ssh. [0]
>
> I think the cleanest approach would be to patch all calls to "git" in
> the vcsh shell script by calls to "/gnu/store/xxxx-git-.../bin/git",
> where the base name is taken as "(assoc-ref %build-inputs "git")" or the like.
>
> What do you think?
>
> It might be as simple as a call in a phase to "substitute*", replacing
> "git " by the path+" ". The script is a bit annoying in that it scatters
> calls to git all over the place. It would be cleaner to define "GIT=git"
> at the start of the file, and then always call "$(GIT)". Maybe that is
> something to suggest to the upstream author?
=
Yep, I can look in to doing this.
signature.asc
Description: OpenPGP digital signature
- Re: [PATCH 2/3] gnu: Don't use guix build utils, (continued)
[PATCH 3/3] gnu: Add vcsh, Christopher Baines, 2016/02/13
- Re: [PATCH 3/3] gnu: Add vcsh, Leo Famulari, 2016/02/13
- Re: [PATCH 3/3] gnu: Add vcsh, Christopher Baines, 2016/02/14
- Re: [PATCH 3/3] gnu: Add vcsh, Andreas Enge, 2016/02/17
- Re: [PATCH 3/3] gnu: Add vcsh, Christopher Baines, 2016/02/17
- Re: [PATCH 3/3] gnu: Add vcsh, Leo Famulari, 2016/02/17
- Re: [PATCH 3/3] gnu: Add vcsh, Christopher Baines, 2016/02/17
- Re: [PATCH 3/3] gnu: Add vcsh, Andreas Enge, 2016/02/18
- Re: [PATCH 3/3] gnu: Add vcsh,
Christopher Baines <=
- Re: [PATCH 3/3] gnu: Add vcsh, Leo Famulari, 2016/02/18
- Re: [PATCH 3/3] gnu: Add vcsh, Christopher Baines, 2016/02/18
- Re: [PATCH 3/3] gnu: Add vcsh, Andreas Enge, 2016/02/18