From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 17299 invoked by alias); 16 Jan 2017 15:59:15 -0000 Mailing-List: contact zsh-workers-help@zsh.org; run by ezmlm Precedence: bulk X-No-Archive: yes List-Id: Zsh Workers List List-Post: List-Help: X-Seq: 40369 Received: (qmail 16422 invoked from network); 16 Jan 2017 15:59:15 -0000 X-Qmail-Scanner-Diagnostics: from mailout2.w1.samsung.com by f.primenet.com.au (envelope-from , uid 7791) with qmail-scanner-2.11 (clamdscan: 0.99.2/21882. spamassassin: 3.4.1. Clear:RC:0(210.118.77.12):SA:0(-7.5/5.0):. Processed in 2.035166 secs); 16 Jan 2017 15:59:15 -0000 X-Spam-Checker-Version: SpamAssassin 3.4.1 (2015-04-28) on f.primenet.com.au X-Spam-Level: X-Spam-Status: No, score=-7.5 required=5.0 tests=RCVD_IN_DNSWL_HI, RCVD_IN_MSPIKE_H3,RCVD_IN_MSPIKE_WL,RP_MATCHES_RCVD,SPF_HELO_PASS,URIBL_SBL, URIBL_SBL_A autolearn=unavailable autolearn_force=no version=3.4.1 X-Envelope-From: p.stephenson@samsung.com X-Qmail-Scanner-Mime-Attachments: | X-Qmail-Scanner-Zip-Files: | Received-SPF: none (ns1.primenet.com.au: domain at samsung.com does not designate permitted sender hosts) X-AuditID: cbfec7f4-f79716d000006f65-a8-587cedcaa40d Date: Mon, 16 Jan 2017 15:59:03 +0000 From: Peter Stephenson To: Zsh hackers list Subject: Re: PATH: autoload with explicit path Message-id: <20170116155903.3f40e475@pwslap01u.europe.root.pri> In-reply-to: Organization: Samsung Cambridge Solution Centre X-Mailer: Claws Mail 3.7.9 (GTK+ 2.22.0; i386-redhat-linux-gnu) MIME-version: 1.0 Content-type: text/plain; charset=US-ASCII Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFnrDIsWRmVeSWpSXmKPExsWy7djPc7qn3tZEGDzZL2dxsPkhkwOjx6qD H5gCGKO4bFJSczLLUov07RK4MvrnrmYqWMxX8ezUE+YGxnncXYycHBICJhJXTh9khbDFJC7c W8/WxcjFISSwlFHi/uf/jBBOL5PE4x2nWGE67s95xAJiCwksY5R4szYcomgak0TLw6NMEM4Z RokFG45DOWcZJY6enskE0sIioCpxbvFaZhCbTcBQYuqm2YwgtoiAlsSOkyfBaoQF9CR+fP0A toJXwF5i7fNGdhCbUyBYoq/9CVgvv4C+xNW/n5ggTrKXmHnlDCNEvaDEj8n3wHqZBXQktm17 zA5hy0tsXvOWGeQgCYH/bBL3nzwBSnAAObISmw4wQ8xxkTh+eg87hC0s8er4FihbRqKz4yDU rn5GiSfdvhBzZjBKnD6zgw0iYS3Rd/siI8QyPolJ26YzQ8znlehoE4Io8ZBY+vo6G0TYUeLl TtsJjIqzkFw9C8nVs5BcvYCReRWjSGppcW56arGJXnFibnFpXrpecn7uJkZgGjj97/iXHYyL j1kdYhTgYFTi4Z1xtyZCiDWxrLgy9xCjBAezkghv/AugEG9KYmVValF+fFFpTmrxIUZpDhYl cd49C66ECwmkJ5akZqemFqQWwWSZODilGhjjPQIVbxeICH2vu7jVz+ipys5De8rvf3Ha8zpx Q8qK5w+PfN2/TEmrc8vSfX8erF5zrv1rpVjf+QMrZRP2B9p7HwwJ8q6+tJFzhlyS3oXypC7u bNlPIgrv5HQndLVWJ83Jqmx6fIG9ets7/87nG2w43/ra6L8o3nxDynbr7EOZBTO4S6eEWvIo sRRnJBpqMRcVJwIADWGaev8CAAA= X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFlrGIsWRmVeSWpSXmKPExsVy+t/xa7pM72oiDD7t5rA42PyQyYHRY9XB D0wBjFFuNhmpiSmpRQqpecn5KZl56bZKoSFuuhZKCnmJuam2ShG6viFBSgpliTmlQJ6RARpw cA5wD1bSt0twy+ifu5qpYDFfxbNTT5gbGOdxdzFyckgImEjcn/OIBcIWk7hwbz1bFyMXh5DA EkaJ3Y1vmSCcGUwS8x/sZgOpEhI4xyjxutsZInGWUWL1gStg7SwCqhLnFq9lBrHZBAwlpm6a zQhiiwhoSew4eZIJxBYW0JP48fUDWD2vgL3E2ueN7CA2p0CwxJm985khhi5lkehuuAy2jV9A X+Lq309MEPfZS8y8coYRollQ4sfke2CDmIEWbN7WxAphy0tsXvOWGeJSdYkbd3ezT2AUnoWk ZRaSlllIWhYwMq9iFEktLc5Nzy021CtOzC0uzUvXS87P3cQIjKNtx35u3sF4aWPwIUYBDkYl Hl6H2zURQqyJZcWVuYcYJTiYlUR4418AhXhTEiurUovy44tKc1KLDzGaAkNmIrOUaHI+MMbz SuINTQzNLQ2NjC0szI2MlMR5Sz5cCRcSSE8sSc1OTS1ILYLpY+LglGpgrNm14OinObxW2wLY on+VZq5sklVe+NMyf8/pnQeMnzxYe+6FgqWkt0tp87nl33Ozdny+6vfKvVRbM3tF9oYe3ycx p9Nmss+R1L8849qUrcsloz/xzGo4y1ubvj9LNU340GrhpQpamY9tm/1UPT6/WjxBV6WgZwEj 35HNqjU32vLUg51vxXqfU2Ipzkg01GIuKk4EAChePFe5AgAA X-MTR: 20000000000000000@CPGS X-CMS-MailID: 20170116155906eucas1p1405c56922c0af46091350f90bf00d6e7 X-Msg-Generator: CA X-Sender-IP: 182.198.249.179 X-Local-Sender: =?UTF-8?B?UGV0ZXIgU3RlcGhlbnNvbhtTQ1NDLURhdGEgUGxhbmUb?= =?UTF-8?B?7IK87ISx7KCE7J6QG1ByaW5jaXBhbCBFbmdpbmVlciwgU29mdHdhcmU=?= X-Global-Sender: =?UTF-8?B?UGV0ZXIgU3RlcGhlbnNvbhtTQ1NDLURhdGEgUGxhbmUbU2Ft?= =?UTF-8?B?c3VuZyBFbGVjdHJvbmljcxtQcmluY2lwYWwgRW5naW5lZXIsIFNvZnR3YXJl?= X-Sender-Code: =?UTF-8?B?QzEwG0VIURtDMTBDRDA1Q0QwNTAwNTg=?= CMS-TYPE: 201P X-HopCount: 7 X-CMS-RootMailID: 20161212160617epcas2p16960e3d95c694147035f760090e6011b X-RootMTR: 20161212160617epcas2p16960e3d95c694147035f760090e6011b References: <20161211221844.5e51affe@ntlworld.com> <161212080550.ZM935@torch.brasslantern.com> <20161212163124.6654f077@pwslap01u.europe.root.pri> <20170110193102.7725620a@ntlworld.com> <20170111114232.04eedd08@pwslap01u.europe.root.pri> <20170111205122.2c47e89f@ntlworld.com> <20170113180423.12225591@pwslap01u.europe.root.pri> <20170116103708.46d2ee66@pwslap01u.europe.root.pri> On Mon, 16 Jan 2017 07:22:23 -0800 Bart Schaefer wrote: > On Mon, Jan 16, 2017 at 2:37 AM, Peter Stephenson > wrote: > > > > % whence -v my\ stupid\ function\ name > > my stupid function name is a shell function from /export/home/pws/src/zsh-git/code/'my stupid function name' > > > > You might even consider that a feature. > > When was this pushed? I don't get any quoting. > > % path+=(/tmp/one\ space) > % whence -v two\ space > two space is /tmp/one space/two space That's not a shell function, that's a command. Looks like that doesn't do quoting.; I presume that's just a historical oversight. This would handle the other verbose "is" cases. However, in this case the entire path is passed back by findcmd, so it's all quoted. Updating findcmd() would make it more efficient (don't need to copy the directory) and more consistent with the function case, but it's too minor for me to feel like doing. pws diff --git a/Src/builtin.c b/Src/builtin.c index a683032..b1b6e2e 100644 --- a/Src/builtin.c +++ b/Src/builtin.c @@ -3747,7 +3747,7 @@ bin_whence(char *nam, char **argv, Options ops, int func) } else { if (v && !csh) zputs(*argv, stdout), fputs(" is ", stdout); - zputs(buf, stdout); + quotedzputs(buf, stdout); if (OPT_ISSET(ops,'s') || OPT_ISSET(ops, 'S')) print_if_link(buf, OPT_ISSET(ops, 'S')); fputc('\n', stdout); @@ -3779,7 +3779,7 @@ bin_whence(char *nam, char **argv, Options ops, int func) } else { if (v && !csh) zputs(*argv, stdout), fputs(" is ", stdout); - zputs(cnam, stdout); + quotedzputs(cnam, stdout); if (OPT_ISSET(ops,'s') || OPT_ISSET(ops,'S')) print_if_link(cnam, OPT_ISSET(ops,'S')); fputc('\n', stdout);