From: Peter Stephenson <p.w.stephenson@ntlworld.com>
To: Zsh Hackers' List <zsh-workers@zsh.org>
Subject: Re: 'whence' question
Date: Thu, 6 Nov 2014 21:10:17 +0000 [thread overview]
Message-ID: <20141106211017.11b8848a@pws-pc.ntlworld.com> (raw)
In-Reply-To: <141105204330.ZM2973@torch.brasslantern.com>
On Wed, 05 Nov 2014 20:43:30 -0800
Bart Schaefer <schaefer@brasslantern.com> wrote:
> On Nov 5, 6:00pm, Peter Stephenson wrote:
> } Subject: Re: 'whence' question
> }
> } On Wed, 05 Nov 2014 18:49:23 +0100
> } Oliver Kiddle <okiddle@yahoo.co.uk> wrote:
> } > Looking at the source, it seems that whence doesn't support both -a and
> } > -m together though it doesn't produce an error.
> }
> } You're right: I think *that's* a bug.
>
> Hmm. This is more complicated than it at first appears.
Moved to zsh-workers.
Don't we want something like this?
diff --git a/Src/builtin.c b/Src/builtin.c
index 5b711ed..101c19a 100644
--- a/Src/builtin.c
+++ b/Src/builtin.c
@@ -3228,11 +3228,39 @@ bin_whence(char *nam, char **argv, Options ops, int func)
scanmatchtable(builtintab, pprog, 1, 0, DISABLED,
builtintab->printnode, printflags);
}
- /* Done search for `internal' commands, if the -p option *
- * was not used. Now search the path. */
- cmdnamtab->filltable(cmdnamtab);
- scanmatchtable(cmdnamtab, pprog, 1, 0, 0,
- cmdnamtab->printnode, printflags);
+ if (all) {
+ char **pp, *buf, *fn;
+ DIR *od;
+
+ pushheap();
+ for (pp = path; *pp; pp++) {
+ if (!**pp)
+ continue;
+ od = opendir(*pp);
+ if (!od)
+ continue;
+
+ while ((fn = zreaddir(od, 0))) {
+ if (!pattry(pprog, fn))
+ continue;
+
+ buf = zhtricat(*pp, "/", fn);
+
+ if (iscom(buf)) {
+ zputs(buf, stdout);
+ fputc('\n', stdout);
+ }
+ }
+ closedir(od);
+ }
+ popheap();
+ } else {
+ /* Done search for `internal' commands, if the -p option *
+ * was not used. Now search the path. */
+ cmdnamtab->filltable(cmdnamtab);
+ scanmatchtable(cmdnamtab, pprog, 1, 0, 0,
+ cmdnamtab->printnode, printflags);
+ }
unqueue_signals();
}
--
Peter Stephenson <p.w.stephenson@ntlworld.com>
Web page now at http://homepage.ntlworld.com/p.w.stephenson/
next parent reply other threads:[~2014-11-06 21:15 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <545A6D66.3080500@eastlink.ca>
[not found] ` <1458.1415209763@thecus.kiddle.eu>
[not found] ` <20141105180035.22f6e9b1@pwslap01u.europe.root.pri>
[not found] ` <141105204330.ZM2973@torch.brasslantern.com>
2014-11-06 21:10 ` Peter Stephenson [this message]
2014-11-06 21:58 ` Bart Schaefer
2014-11-08 20:41 ` Peter Stephenson
2014-11-09 18:51 ` Bart Schaefer
2014-11-10 5:15 ` 'whence' question and others Ray Andrews
2014-11-10 8:20 ` Bart Schaefer
2014-11-10 8:23 ` Bart Schaefer
2014-11-10 16:57 ` Ray Andrews
2014-11-10 19:53 ` Vin Shelton
2014-11-11 16:52 ` 'whence' question Ray Andrews
2014-11-11 19:14 ` Bart Schaefer
2014-11-11 19:38 ` Bart Schaefer
2014-11-11 20:22 ` Ray Andrews
2014-11-11 18:16 ` Ray Andrews
2014-11-11 19:33 ` Bart Schaefer
2014-11-11 20:40 ` Ray Andrews
2014-11-08 21:55 ` Bart Schaefer
2014-11-10 10:04 ` Peter Stephenson
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=20141106211017.11b8848a@pws-pc.ntlworld.com \
--to=p.w.stephenson@ntlworld.com \
--cc=zsh-workers@zsh.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 public inbox
https://git.vuxu.org/mirror/zsh/
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).