From: Stefan Monnier <monnier@iro.umontreal.ca>
Cc: emacs-devel@gnu.org
Subject: Re: executable-find in files.el
Date: Wed, 11 May 2005 15:07:36 -0400 [thread overview]
Message-ID: <87fywt6128.fsf-monnier+emacs@gnu.org> (raw)
In-Reply-To: <01c55657$Blat.v2.4$7979dc20@zahav.net.il> (Eli Zaretskii's message of "Wed, 11 May 2005 21:29:12 +0300")
>> + (defun executable-find (command)
>> + "Search for COMMAND in `exec-path' and return the absolute file name.
>> + Return nil if COMMAND is not found anywhere in `exec-path'."
>> + ;; Use 1 rather than file-executable-p to better match the behavior of
>> + ;; call-process.
>> + (locate-file command exec-path exec-suffixes 1))
> Passing 1 as last arg of locate-file is subtly different from passing
> file-executable-p. I think the latter does a better job, so I think
> executable-find should use file-executable-p.
Have you read the comment you quoted? What do you reply to that?
Do you think it's more important to "do a subtly better job" or to "match
the behavior of call-process"?
In my view, the point of executable-find is to figure out whether there is
a command that we can run. If it tells us "I found /ssh:foo/bar/baz", but
then call-process fails because it doesn't work through Tramp, I think it's
a problem.
Stefan
next prev parent reply other threads:[~2005-05-11 19:07 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <E1DVuFh-0001h1-GE@monty-python.gnu.org>
2005-05-11 18:29 ` executable-find in files.el Eli Zaretskii
2005-05-11 19:07 ` Stefan Monnier [this message]
2005-05-11 22:56 ` Eli Zaretskii
2005-05-11 23:16 ` Stefan Monnier
2005-05-11 23:50 ` David Kastrup
2005-05-12 1:31 ` Stefan Monnier
2005-05-12 6:34 ` Eli Zaretskii
2005-05-12 12:42 ` Stefan Monnier
2005-05-12 13:14 ` Eli Zaretskii
2005-05-14 14:58 ` Michael Albinus
2005-05-15 0:21 ` Richard Stallman
2005-05-16 19:27 ` Michael Albinus
2005-05-16 22:38 ` Stefan Monnier
2005-05-17 22:03 ` Kim F. Storm
2005-05-17 22:23 ` Stefan Monnier
2005-05-18 13:08 ` Richard Stallman
2005-05-17 4:36 ` Richard Stallman
2005-05-17 16:32 ` Michael Albinus
2005-05-18 5:19 ` Richard Stallman
2005-05-18 13:08 ` Richard Stallman
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87fywt6128.fsf-monnier+emacs@gnu.org \
--to=monnier@iro.umontreal.ca \
--cc=emacs-devel@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.