From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 9620 invoked by alias); 29 Sep 2014 13:26:37 -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: 33278 Received: (qmail 10301 invoked from network); 29 Sep 2014 13:26:34 -0000 X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on f.primenet.com.au X-Spam-Level: X-Spam-Status: No, score=-6.9 required=5.0 tests=BAYES_00,RCVD_IN_DNSWL_HI, SPF_HELO_PASS autolearn=ham version=3.3.2 X-AuditID: cbfec7f5-b7f776d000003e54-e9-54295e0500cf Date: Mon, 29 Sep 2014 14:26:28 +0100 From: Peter Stephenson To: zsh-workers@zsh.org Subject: Re: [PATCH] Fix VCS_INFO_reposub's command expansion Message-id: <20140929142628.27e15e54@pwslap01u.europe.root.pri> In-reply-to: <1411993408-31305-1-git-send-email-ft@bewatermyfriend.org> References: <20140928171033.GA11234@cheyenne> <1411993408-31305-1-git-send-email-ft@bewatermyfriend.org> 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+NgFupmluLIzCtJLcpLzFFi42I5/e/4VV3WOM0Qg+d9ahYHmx8yOTB6rDr4 gSmAMYrLJiU1J7MstUjfLoEr49jNNywFncwVnzfuYWlg3MjUxcjBISFgIjGry6WLkRPIFJO4 cG89WxcjF4eQwFJGiSvT9jNCOMuZJM6dmcIOUsUioCpxctdTFhCbTcBQYuqm2YwgtoiAuMTZ tefB4sICthLPNz1nA7F5Bewlmo9fBrM5Bdwlvk9oZANZLCSQJbHtnTZImF9AX+Lq309MEEfY S8y8coYRolVQ4sfke2AjmQW0JDZva2KFsOUlNq95yzyBUWAWkrJZSMpmISlbwMi8ilE0tTS5 oDgpPddIrzgxt7g0L10vOT93EyMkAL/uYFx6zOoQowAHoxIPL8cKjRAh1sSy4srcQ4wSHMxK Irx2tpohQrwpiZVVqUX58UWlOanFhxiZODilGhgzp1Ufu/pCvGZlWuJU4VOWJ04YqJdk5G/5 HH7w4/9+3Vu8LwWvJZQxmNSpnUyrVS3SXTbzCceSxf+mZj569GDCduOb/+rz3q+2D0lcteI3 ywSLhT28Ye1i1otvVSlcXCljEl3H/4ljRfVuWWe1/mPVRoxG7qq7LqsFrp9qWfm0onmTWe3t PQuUWIozEg21mIuKEwHp9kS+HgIAAA== On Mon, 29 Sep 2014 14:23:28 +0200 Frank Terbeck wrote: > -[[ $(pwd -P) == ${base}/* ]] || { > +tmp="$(pwd -P)" > +[[ $tmp == ${base}/* ]] || { There's no problem with this change, but actually you get away with it anyway here since the [[ ... ]] context knows it should expand as a single word. [[ $(print eurgh yuk) == "eurgh yuk" ]] && print OK pws