svnwcsub.py - various pylint errors

[sebb]

pylint reports quite a few warnings and errors:

svnwcsub.py:55:0: W0311: Bad indentation. Found 2 spaces, expected 4
(bad-indentation)
svnwcsub.py:57:0: W0311: Bad indentation. Found 2 spaces, expected 4
(bad-indentation)
svnwcsub.py:61:0: W0311: Bad indentation. Found 2 spaces, expected 4
(bad-indentation)
svnwcsub.py:63:0: W0311: Bad indentation. Found 2 spaces, expected 4
(bad-indentation)
svnwcsub.py:67:0: W0311: Bad indentation. Found 2 spaces, expected 4
(bad-indentation)
svnwcsub.py:69:0: W0311: Bad indentation. Found 2 spaces, expected 4
(bad-indentation)
svnwcsub.py:87:4: W0612: Unused variable 'output' (unused-variable)
svnwcsub.py:115:12: W0612: Unused variable 'x' (unused-variable)
svnwcsub.py:439:38: W0613: Unused argument 'event_arg' (unused-argument)
svnwcsub.py:46:0: W0611: Unused import errno (unused-import)
svnwcsub.py:58:0: W0611: Unused import time (unused-import)
svnwcsub.py:65:0: W0611: Unused import functools (unused-import)
svnwcsub.py:67:2: W0611: Unused import urlparse (unused-import)
svnwcsub.py:69:2: W0611: Unused urllib.parse imported as urlparse
(unused-import)

svnwcsub.py:378:18: E0602: Undefined variable 'fname' (undefined-variable)
(should be self.fname)

Sebb

svnpubsub/commit-hook.py:65 - variable url assigned but not used

[sebb]

As the subject says

watcher.py no longer uses urlparse

[sebb]

As the subject says.

svnwcsub.py does not work across different SVN trees

[sebb]

It looks like svnwcsub.py is not always able to update the workspace
if the source URL is updated in the configuration file.

I am seeing an error of the form:

svn: E195012: Path '/www/ace.apache.org' does not share common version
control ancestry with the requested switch location.  Use
--ignore-ancestry to disable this check.
svn: E195012: 'https://svn-master.apache.org/repos/infra/sites/ace'
shares no common ancestry with '/www/ace.apache.org

Is there any reason why the svn switch command does not include the
suggested option?
Seems to me it ought to be possible to replace the checkout with a
different SVN repo without needing to manually remove the existing
checkout.

Sebb.