[PATCH] [PATCH RESEND] convert/svn: Abort operation when python bindings are not available

[Azhagu Selvan SP]

On Wed, 2011-02-23 at 22:42 +0530, Azhagu Selvan SP wrote:
> On Wed, 2011-02-23 at 11:42 +0100, Mads Kiilerich wrote:
> 
> > >
> > > But why the tests didn't fail? Should we update the tests?
> > 
> > The svn convert tests are skipped if the svn libraries not are available 
> > - see hghave.
> > 
> > We could perhaps add some tests that hacked PYTHONPATH and/or provided a 
> > stub svn.py in such a way that we got some tests of these failures even 
> > when svn was available. But ... that would probably be complex and 
> > fragile and add very little value.
> 
> Interestingly, in my debian squeeze machine I have these two lines in
> svn_sink class constructor,
> 
> <snip>
> 
> checktool('svn',debname='subversion')                                                                               
> checktool('svnadmin',debname='subversion') 
> 
> </snip>                                                                         
> 
> I have mercurial version 1.6.4. But this is not present in the source
> tree? May be debian guys have applied this patch?

Yes, this is a debian specific patch

http://patch-tracker.debian.org/patch/series/view/mercurial/1.6.4-1/deb_specific__optional-dependencies




-- 
Azhagu Selvan SP

http://azhaguselvan.co.cc