[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] gnu: Add asunder.
From: |
Chris Marusich |
Subject: |
Re: [PATCH] gnu: Add asunder. |
Date: |
Fri, 23 Dec 2016 18:11:44 -0800 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux) |
Leo Famulari <address@hidden> writes:
> On Fri, Dec 23, 2016 at 12:16:24AM -0800, Chris Marusich wrote:
>> Hi Leo,
>>
>> I've responded to the feedback from you, Hartmut, and John. How does
>> the latest patch look? It's available here:
>>
>> https://lists.gnu.org/archive/html/guix-devel/2016-12/msg00707.html
>
> Thanks for the reminder! I pushed as
> 426e6083ae9d4569005dab8edf948485e5979171.
>
> I think it would be useful to figure out if it's possible to avoid the
> wrapper somehow, but I didn't look into this closely.
I'm sure we could avoid the wrapper by patching the source, but why
would that be better? The wrapper is a simple and robust solution, and
in this case I can't see any drawbacks to using it.
For context, Asunder assumes that various tools will be made available
via the PATH environment variable. It refuses to function in certain
cases when a tool that it needs can't be found in the PATH. I'm sure we
could patch this mechanism, but it seems simpler to just create a
wrapper that puts the directories containing the tools onto the PATH.
--
Chris
signature.asc
Description: PGP signature
- Add Asunder, Chris Marusich, 2016/12/17
- [PATCH] gnu: Add asunder., Chris Marusich, 2016/12/17
- Re: [PATCH] gnu: Add asunder., John Darrington, 2016/12/17
- Re: [PATCH] gnu: Add asunder., Hartmut Goebel, 2016/12/17
- Re: [PATCH] gnu: Add asunder., Leo Famulari, 2016/12/17
- Re: [PATCH] gnu: Add asunder., Chris Marusich, 2016/12/18
- Re: [PATCH] gnu: Add asunder., Chris Marusich, 2016/12/23
- Re: [PATCH] gnu: Add asunder., Leo Famulari, 2016/12/23
- Re: [PATCH] gnu: Add asunder.,
Chris Marusich <=
- Re: [PATCH] gnu: Add asunder., John Darrington, 2016/12/24
- Re: [PATCH] gnu: Add asunder., Chris Marusich, 2016/12/24
- Re: [PATCH] gnu: Add asunder., John Darrington, 2016/12/25
- Re: [PATCH] gnu: Add asunder., Leo Famulari, 2016/12/25