guix-devel
[Top][All Lists]
Advanced

[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

Attachment: signature.asc
Description: PGP signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]