git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Bash prompt "namespace" issue
@ 2014-05-13  6:18 szeder
  2014-05-13 18:52 ` Junio C Hamano
  0 siblings, 1 reply; 2+ messages in thread
From: szeder @ 2014-05-13  6:18 UTC (permalink / raw)
  To: Junio C Hamano, Felipe Contreras; +Cc: git

Hi,

Commit 59d3924fb (prompt: fix missing file errors in zsh) added the  
helper function eread() to git-prompt.sh.  That function should be in  
the git "namespace", i.e. prefixed with __git_, otherwise it might  
collide with a function of the same name in the user's environment.

It's already in master and I don't have the means to send a patch  
fixing this ATM, sorry.

Best,
Gábor

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: Bash prompt "namespace" issue
  2014-05-13  6:18 Bash prompt "namespace" issue szeder
@ 2014-05-13 18:52 ` Junio C Hamano
  0 siblings, 0 replies; 2+ messages in thread
From: Junio C Hamano @ 2014-05-13 18:52 UTC (permalink / raw)
  To: szeder; +Cc: Felipe Contreras, git

szeder@ira.uka.de writes:

> Commit 59d3924fb (prompt: fix missing file errors in zsh) added the  
> helper function eread() to git-prompt.sh.  That function should be in  
> the git "namespace", i.e. prefixed with __git_, otherwise it might  
> collide with a function of the same name in the user's environment.
>
> It's already in master and I don't have the means to send a patch  
> fixing this ATM, sorry.

Thanks.  I think the patch Felipe posted to rename it to __git_eread
is a reasonable regression fix, so I'll queue it on top of the
problematic commit 59d3924f (prompt: fix missing file errors in zsh,
2014-04-11) and merge the result to 'master'.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2014-05-13 18:52 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-05-13  6:18 Bash prompt "namespace" issue szeder
2014-05-13 18:52 ` Junio C Hamano

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).