You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Jelmer Vernooij <je...@vernstok.nl> on 2011/12/14 12:49:49 UTC

Re: svn commit: r25805 - trunk

On Sat, 2007-07-21 at 14:24 -0700, Blair Zajac wrote:
> jrvernooij@tigris.org wrote:
> > Author: jrvernooij
> > Date: Fri Jul 20 17:13:48 2007
> > New Revision: 25805
> > 
> > Log:
> > Fix Python bindings for svn.ra.lock().
> > 
> > * subversion/bindings/swig/include/svn_containers.swg: Add Python 
> >   typemap for apr_hash_t *path_revs.
> > 
> > * subversion/bindings/swig/svn_ra.i
> >   (svn_ra_lock_callback_t): Add Python typemap
> > 
> > * subversion/bindings/swig/python/libsvn_swig_py/swigutil_py.c
> >   (svn_swig_py_path_revs_hash_from_dict): Add function for converting 
> >   from dictionaries to apr_hash_t * mapping paths to revnums.
> >  
> >   (svn_swig_py_ra_lock_callback): Add Python wrapper for 
> >   svn_ra_lock_callback_t
> > 
> > * subversion/bindings/swig/python/libsvn_swig_py/swigutil_py.c
> >   (svn_swig_py_path_revs_hash_from_dict): Add prototype.
> > 
> >   (svn_swig_py_ra_lock_callback): Add prototype.
> > 
> > * subversion/bindings/swig/python/tests/ra.py
> >   (test_lock): Add test for svn.ra.lock()
> > 
> > 
> > Modified:
> >    trunk/   (props changed)
> 
> Was this meant to be an empty commit?
No, it most certainly wasn't. I've properly committed this patch now,
will update the log message of r25805. 

Thanks for pointing out.

Cheers,

Jelmer

Re: svn commit: r25805 - trunk

Posted by Jelmer Vernooij <je...@vernstok.nl>.
On Mon, 2007-07-30 at 08:34 -0700, Daniel Rall wrote:
> On Sun, 29 Jul 2007, Peter Samuelson wrote:
> > [Jelmer Vernooij]
> > > On Sat, 2007-07-21 at 14:24 -0700, Blair Zajac wrote:
> > > > Was this meant to be an empty commit?
> > > No, it most certainly wasn't. I've properly committed this patch now,
> > > will update the log message of r25805. 
> > 
> > Someone want to do that sometime?  The new rev is r25828.
> Jelmer, you want to take care of this?
Whoops, forgot about this. Thanks for reminding, it's fixed now. 

Cheers,

Jelmer

Re: svn commit: r25805 - trunk

Posted by Daniel Rall <dl...@collab.net>.
On Sun, 29 Jul 2007, Peter Samuelson wrote:

> 
> [Jelmer Vernooij]
> > On Sat, 2007-07-21 at 14:24 -0700, Blair Zajac wrote:
> > > Was this meant to be an empty commit?
> > No, it most certainly wasn't. I've properly committed this patch now,
> > will update the log message of r25805. 
> 
> Someone want to do that sometime?  The new rev is r25828.

Jelmer, you want to take care of this?

Thanks, Dan

Re: svn commit: r25805 - trunk

Posted by Peter Samuelson <pe...@p12n.org>.
[Jelmer Vernooij]
> On Sat, 2007-07-21 at 14:24 -0700, Blair Zajac wrote:
> > Was this meant to be an empty commit?
> No, it most certainly wasn't. I've properly committed this patch now,
> will update the log message of r25805. 

Someone want to do that sometime?  The new rev is r25828.
-- 
Peter Samuelson | org-tld!p12n!peter | http://p12n.org/