Re: [PATCH] prompt: fix tracked files for zsh

2013-04-21 Thread Felipe Contreras
On Sun, Apr 21, 2013 at 5:00 PM, Junio C Hamano wrote: > Felipe Contreras writes: > >> On Sun, Apr 21, 2013 at 2:02 PM, Andreas Schwab >> wrote: >>> Junio C Hamano writes: >>> Andreas Schwab writes: > Felipe Contreras writes: > >> + if [ -n "${

Re: [PATCH] prompt: fix tracked files for zsh

2013-04-21 Thread Junio C Hamano
Felipe Contreras writes: > On Sun, Apr 21, 2013 at 2:02 PM, Andreas Schwab wrote: >> Junio C Hamano writes: >> >>> Andreas Schwab writes: >>> Felipe Contreras writes: > + if [ -n "${ZSH_VERSION-}" ]; then > + u="%%"

Re: [PATCH] prompt: fix tracked files for zsh

2013-04-21 Thread Felipe Contreras
On Sun, Apr 21, 2013 at 2:02 PM, Andreas Schwab wrote: > Junio C Hamano writes: > >> Andreas Schwab writes: >> >>> Felipe Contreras writes: >>> + if [ -n "${ZSH_VERSION-}" ]; then + u="%%" + els

Re: [PATCH] prompt: fix tracked files for zsh

2013-04-21 Thread Andreas Schwab
Junio C Hamano writes: > Andreas Schwab writes: > >> Felipe Contreras writes: >> >>> + if [ -n "${ZSH_VERSION-}" ]; then >>> + u="%%" >>> + else >>> + u="%" >>> +

Re: [PATCH] prompt: fix tracked files for zsh

2013-04-21 Thread Junio C Hamano
Andreas Schwab writes: > Felipe Contreras writes: > >> +if [ -n "${ZSH_VERSION-}" ]; then >> +u="%%" >> +else >> +u="%" >> +fi > > aka u="%$

Re: [PATCH] prompt: fix tracked files for zsh

2013-04-21 Thread Junio C Hamano
Felipe Contreras writes: > When this option is enabled, the prompt gets totally screwed in zsh > because all the codes start with % in zsh (like they are \ in bas). So > we need to escape the % character. It is not that I do not believe that $u needs to duplicate %% in ZSH, but I hate it when I

Re: [PATCH] prompt: fix tracked files for zsh

2013-04-21 Thread Junio C Hamano
Felipe Contreras writes: > When this option is enabled, the prompt gets totally screwed in zsh > because all the codes start with % in zsh (like they are \ in bas). So > we need to escape the % character. It is not that I do not believe that $u needs to duplicate %% in ZSH, but I hate it when I

Re: [PATCH] prompt: fix tracked files for zsh

2013-04-21 Thread Eric Sunshine
On Sun, Apr 21, 2013 at 4:11 AM, Felipe Contreras wrote: > When this option is enabled, the prompt gets totally screwed in zsh > because all the codes start with % in zsh (like they are \ in bas). So s/bas/bash/ > we need to escape the % character. > > Signed-off-by: Felipe Contreras -- To unsu

Re: [PATCH] prompt: fix tracked files for zsh

2013-04-21 Thread Andreas Schwab
Felipe Contreras writes: > + if [ -n "${ZSH_VERSION-}" ]; then > + u="%%" > + else > + u="%" > + fi aka u="%${ZSH_VERSION:+%}" Andreas. --

[PATCH] prompt: fix tracked files for zsh

2013-04-21 Thread Felipe Contreras
When this option is enabled, the prompt gets totally screwed in zsh because all the codes start with % in zsh (like they are \ in bas). So we need to escape the % character. Signed-off-by: Felipe Contreras --- contrib/completion/git-prompt.sh | 6 +- 1 file changed, 5 insertions(+), 1 deleti