[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] gnu: Add asunder.
From: |
John Darrington |
Subject: |
Re: [PATCH] gnu: Add asunder. |
Date: |
Sat, 24 Dec 2016 08:07:41 +0100 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Fri, Dec 23, 2016 at 06:11:44PM -0800, Chris Marusich wrote:
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.
FWIW, I think wrappers are bit of a nasty solution and should be avoided
if feasible. Sometimes however there is no reasonable way to avoid them.
It sounds as if this might be such as case. J'
--
Avoid eavesdropping. Send strong encrypted email.
PGP Public key ID: 1024D/2DE827B3
fingerprint = 8797 A26D 0854 2EAB 0285 A290 8A67 719C 2DE8 27B3
See http://sks-keyservers.net or any PGP keyserver for public key.
signature.asc
Description: Digital 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, 2016/12/23
- Re: [PATCH] gnu: Add asunder.,
John Darrington <=
- 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