Subject: Re: bug#24471: 25.1.50; Error on empty PATH component

From: Eli Zaretskii <[email-masked]>
Date: Mon, 17 Oct 2016 20:36:17 +0300

> From: Achim Gratz <[email-masked]>
> Date: Mon, 17 Oct 2016 18:25:17 +0200
> 
> Eli Zaretskii writes:
> > Thanks.  Does the patch below fix the problem?
> >
> > diff --git a/lisp/cus-start.el b/lisp/cus-start.el
> > index 001d638..bd2b280 100644
> > --- a/lisp/cus-start.el
> > +++ b/lisp/cus-start.el
> > @@ -174,8 +174,10 @@ minibuffer-prompt-properties--setter
> >                          nil
> >                          :standard
> >                          (mapcar 'directory-file-name
> > -                                (append (parse-colon-path (getenv "PATH"))
> > -                                        (list exec-directory))))
> > +                                (delq nil
> > +                                      (append
> > +                                       (parse-colon-path (getenv "PATH"))
> > +                                       (list exec-directory)))))
> >  	     (exec-suffixes execute (repeat string))
> >  	     ;; charset.c
> >  	     (charset-map-path installation
> 
> I've tried to patch this in-place, first by recompiling the affected
> custom files, then by removing all the .elc files.  I failed, somehow
> this eval never changes to include the delq nil introduced with the
> patch and I can't seem to find the code path that it takes to get to the
> non-changed version.  I can't recompile Emacs on Windows.

You must re-dump Emacs for this to take effect.

Anyway, the change that was committed is different, so the above is no
longer pertinent.

(I don't think I understand what you meant by your comment about "nil
meaning default-directory being wrong for PATH and POSIX".)