Zsh Mailing List Archive
Messages sorted by:
Reverse Date,
Date,
Thread,
Author
Re: PATCH: perform expansion for precommand modifiers
- X-seq: zsh-workers 41038
- From: Daniel Shahaf <d.s@xxxxxxxxxxxxxxxxxx>
- To: zsh-workers@xxxxxxx
- Subject: Re: PATCH: perform expansion for precommand modifiers
- Date: Mon, 1 May 2017 18:31:07 +0000
- Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= daniel.shahaf.name; h=content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to:x-me-sender :x-me-sender:x-sasl-enc:x-sasl-enc; s=fm1; bh=J0KYp80UX4g8bTDNHn gtBDMVwyXhuX2FqNP7keVjZ40=; b=C9DELNno9N3j+oo4D1iWmavqBrSE6BKslG E21KECGa0kr/EH8Ox8sjlfka5GplRVdUZGletdKGGH7HrkgV/IliX52KuJuLVE15 BzdXT0pxozx7p5NpwFlxDLHAWFeRx5maNgyoJOqomC7JM/hU3il9/06kXOu9ExJQ ffus3uaHzp+R5T15APKO/16hLKH94+q/eZUtyazsuiZt9BnkfQHX0qdwzzPC5k+r cqoU+UnvP6wFzacQj7ac2d4K6qIqBelpl+MM2WZ8KmuNQDXHFHkTJLlyZGiSItDO URI8ULuYjLUcKhTxtVNPfeYoUkQuhtKc+UYRlnhjykVc4SlXOBcQ==
- Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to:x-me-sender :x-me-sender:x-sasl-enc:x-sasl-enc; s=fm1; bh=J0KYp80UX4g8bTDNHn gtBDMVwyXhuX2FqNP7keVjZ40=; b=C4i/zih+bkdeqOrVwruVEokZcv1K1vgl9s yUokFfJ0DSgF19izYqs+g4ohXlDebt9MU+t4oXM83rDJfQN4vohLa9sRkqTQ/cvZ hZQU98zn7+yIto9PCxZCN/A94M3YO7CI8JfpYbrbJeTjOYSxaqoxYTBt3rZgAS5Q lCDMLvZZLjfONpkTyjWMbd/JS04iaPvOgXTGWs8H0VYIkRL6H6wQqwPBSgHXjsng 9L0/9uiwczpxmjMsD2qqhPujotXWIdUD0Qiw6LMHdpZT44bdK7fLPdBGUd/vCyCT o7DRsSYUThfrDamufubhwfo9mHI5wuRX6Az6MyoederDtPg7Z95Q==
- In-reply-to: <20170428100338.27ef869c@pwslap01u.europe.root.pri>
- List-help: <mailto:zsh-workers-help@zsh.org>
- List-id: Zsh Workers List <zsh-workers.zsh.org>
- List-post: <mailto:zsh-workers@zsh.org>
- Mailing-list: contact zsh-workers-help@xxxxxxx; run by ezmlm
- References: <CGME20170425091533eucas1p26af70366557848059af3b7c8957f1d55@eucas1p2.samsung.com> <20170425101520.23910ede@pwslap01u.europe.root.pri> <20170426194517.GA12961@fujitsu.shahaf.local2> <170426142508.ZM19385@torch.brasslantern.com> <20170427094018.27eafa6a@pwslap01u.europe.root.pri> <170427161300.ZM23462@torch.brasslantern.com> <20170428100338.27ef869c@pwslap01u.europe.root.pri>
Peter Stephenson wrote on Fri, Apr 28, 2017 at 10:03:38 +0100:
> Bart Schaefer <schaefer@xxxxxxxxxxxxxxxx> wrote:
> > On Apr 27, 9:40am, Peter Stephenson wrote:
> > This isn't testing the right thing. The complaint was that the -c option
> > of exec was treated as a command name if the keyword was quoted.
>
> I think we can fix this by adding a test for that.
>
> > Similarly for "command -p" which now that I try it is still broken:
> >
> > torch% \command -p echo
> > zsh: command not found: -p
>
> That's nothing to do with the quoting. I assumed we had a test for -p
> without -v or -V, but apparently we don't, so I didn't notice there was
> an extra node to remove.
Updated docs, replaced 41013.
diff --git a/NEWS b/NEWS
index 5479251..568b160 100644
--- a/NEWS
+++ b/NEWS
@@ -4,6 +4,17 @@ CHANGES FROM PREVIOUS VERSIONS OF ZSH
Note also the list of incompatibilities in the README file.
+Changes from 5.3.1 to 5.4
+-------------------------
+
+The 'exec' and 'command' precommand modifiers, and options to them, are
+now parsed after parameter expansion. Previously, both the modifier and
+any options to it were parsed between alias expansion and parameter
+expansion (see zshexpn(1)), so they could neither be quoted nor be the
+result of parameter expansion. Examples: 's=command; $s -V ls' and
+'\command -V ls' now work as expected.
+
+
Changes from 5.2 to 5.3.1
-------------------------
Messages sorted by:
Reverse Date,
Date,
Thread,
Author