Zsh Mailing List Archive Messages sorted by: Reverse Date, Date, Thread, Author

Re: [PATCH] Fix VCS_INFO_reposub's command expansion

X-seq: zsh-workers 33278
From: Peter Stephenson <p.stephenson@xxxxxxxxxxx>
To: zsh-workers@xxxxxxx
Subject: Re: [PATCH] Fix VCS_INFO_reposub's command expansion
Date: Mon, 29 Sep 2014 14:26:28 +0100
In-reply-to: <1411993408-31305-1-git-send-email-ft@bewatermyfriend.org>
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
Organization: Samsung Cambridge Solution Centre
References: <20140928171033.GA11234@cheyenne> <1411993408-31305-1-git-send-email-ft@bewatermyfriend.org>

On Mon, 29 Sep 2014 14:23:28 +0200
Frank Terbeck <ft@xxxxxxxxxxxxxxxxxxx> 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

References:
- [patch] Fix VCS_INFO_reposub's whitespace handling
  - From: Marco Hinz
- [PATCH] Fix VCS_INFO_reposub's command expansion
  - From: Frank Terbeck

Messages sorted by: Reverse Date, Date, Thread, Author