Re: [PATCH] edit: Friendlier error message.

2015-11-15 Thread Andreas Enge
On Wed, Nov 11, 2015 at 11:01:30PM +0100, Ludovic Courtès wrote: > I’ve committed a change that honors $VISUAL first, $EDITOR second. Great, it works in my setting where $EDITOR is empty and $VISUAL equals vim (without me setting either of them, if I remember correctly; some debian magic must have

Re: [PATCH] edit: Friendlier error message.

2015-11-11 Thread Ludovic Courtès
Mathieu Lirzin skribis: > l...@gnu.org (Ludovic Courtès) writes: > >> Andreas Enge skribis: >> >>> On Thu, Nov 05, 2015 at 12:08:35PM +0100, Ludovic Courtès wrote: > +(unless (search-path (parse-path (getenv "PATH")) (%editor)) > + (leave (_ "Couldn't find editor '~a'. Please

Re: [PATCH] edit: Friendlier error message.

2015-11-07 Thread Mathieu Lirzin
l...@gnu.org (Ludovic Courtès) writes: > Andreas Enge skribis: > >> On Thu, Nov 05, 2015 at 12:08:35PM +0100, Ludovic Courtès wrote: >>> > +(unless (search-path (parse-path (getenv "PATH")) (%editor)) >>> > + (leave (_ "Couldn't find editor '~a'. Please check $EDITOR and >>> > $PATH.")

Re: [PATCH] edit: Friendlier error message.

2015-11-07 Thread Ludovic Courtès
Andreas Enge skribis: > On Thu, Nov 05, 2015 at 12:08:35PM +0100, Ludovic Courtès wrote: >> > +(unless (search-path (parse-path (getenv "PATH")) (%editor)) >> > + (leave (_ "Couldn't find editor '~a'. Please check $EDITOR and >> > $PATH.") >> > + (%editor))) > > An additiona

Re: [PATCH] edit: Friendlier error message.

2015-11-06 Thread Andreas Enge
On Thu, Nov 05, 2015 at 12:08:35PM +0100, Ludovic Courtès wrote: > > +(unless (search-path (parse-path (getenv "PATH")) (%editor)) > > + (leave (_ "Couldn't find editor '~a'. Please check $EDITOR and > > $PATH.") > > + (%editor))) An additional, heterodox suggestion: Maybe al

Re: [PATCH] edit: Friendlier error message.

2015-11-05 Thread Ludovic Courtès
be...@bmevers.de skribis: > From: Benno Evers > > * guix/scripts/edit.scm (guix-edit): Check if %editor exists. > --- > These are my first lines of guile, so probably there is a much > better way to do this :D > > The previous error message ("execlp: No such file or directory") > had me confused

[PATCH] edit: Friendlier error message.

2015-11-04 Thread benno
From: Benno Evers * guix/scripts/edit.scm (guix-edit): Check if %editor exists. --- These are my first lines of guile, so probably there is a much better way to do this :D The previous error message ("execlp: No such file or directory") had me confused for a while, though. guix/scripts/edit.sc