Re: [dev] [abduco] [patch] more descriptive execvp error message

From: Marc André Tanner <mat_AT_brain-dump.org>
Date: Mon, 10 Nov 2014 13:59:14 +0100

On Sat, Nov 08, 2014 at 12:56:36PM -0500, Brandon Mulcahy wrote:
> By default abduco attempts to launch dvtm if `abduco -c` is run with no
> command argument. I didn't have dvtm installed, so the resulting error
> message was:
>
> server-execvp: No such file or directory
>
> My first thought was that abduco was implicitly relying on a program
> called 'server-execvp'. Anyway, I made a simple fix to include the
> command name given to execvp(3) as part of the error message.
>
> $ abduco -c test
> server-execvp: dvtm: No such file or directory
> $ abduco -c test badname
> server-execvp: badname: No such file or directory

Thanks applied! I agree, the messages have still potential for
improvement. Not sure whether it makes sense to include the function
name as a prefix.

-- 
 Marc André Tanner >< http://www.brain-dump.org/ >< GPG key: CF7D56C0
Received on Mon Nov 10 2014 - 13:59:14 CET

This archive was generated by hypermail 2.3.0 : Mon Nov 10 2014 - 14:00:15 CET