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

Re: lhw@xxxxxxxx, 611175@xxxxxxxxxxxxxxx



On 2011-02-11 at 07:34 +0100, Frank Terbeck wrote:
> ...now that CVS is back...
> 
> Here's a fix for an issue with vcs_info's subversion detection, which
> was reported in the debian BTS (issue #611175¹).

That entry complains that .svn/ on its own is not sufficient and a more
thorough check is needed.

Your patch adds dead code, because if .svn already exists as a
directory, the function will have returned true already.

I suspect that you should be *replacing* the [[ -d ".svn" ]] check.

-Phil



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