> > + # ctx.config.config (ctx.config is opaque).
> What do you mean by that?
> % PYTHONPATH=$prefix/svn-t1/lib/svn-python/ \
> python -c 'import svn, svn.core, svn.client;
> print svn.core.svn_config_get_config(None, None)["config"]'
> <libsvn.core.svn_config_t; proxy of <Swig Object of type 'svn_config_t *' at 0xb734d998> >
I meant this:
matthijs_at_grubby:~/docs/src/upstream/subversion-trunk$ python -c 'import svn, svn.core, svn.client;print svn.core.svn_config_get_config(None, None)["config"]'
Traceback (most recent call last):
File "<string>", line 1, in <module>
File "/usr/lib/pymodules/python2.7/libsvn/core.py", line 5494, in __getattr__
value = _swig_getattr(self, self.__class__, name)
File "/usr/lib/pymodules/python2.7/libsvn/core.py", line 51, in _swig_getattr
On trunk, your code seems to work (but there, ctx.config still seems to be None, though).
Would using svn.core.svn_config_get_config(None, None)["config"] here be the
> > +++ subversion-trunk/subversion/bindings/swig/python/tests/client.py 2012-02-07 16:20:36.914353499 +0100
> > @@ -374,6 +374,11 @@
> > self.assertEqual(readme_text, 'This is a test.\n')
> > + def test_platform_providers(self):
> > + providers = core.svn_auth_get_platform_specific_client_providers(None, None)
> > + # Not much more we can test in this minimal environment.
> > + self.assert_(isinstance(providers, list))
> > +
> You could assert
> all(map(lambda x: isinstance(x, 'svn_auth_provider_object_t *'),
> like the Perl revision you refer to does.
I couldn't figure out how to do that in the face of swig types. Your suggestion
doesn't work either:
TypeError: isinstance() arg 2 must be a class, type, or tuple of classes and types
> I've committed this patch. (I had to rewrite the log message to conform
> to our style of mentioning every symbol.) Please send followups as
> patches against the new HEAD.
Thanks! I thought I had a policy-compliant log message already, but it seems
you've been even more thorough. I'll keep that in mind for further patches.
Received on 2012-02-07 19:30:13 CET