You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Michael W Thelen <mi...@pietdepsi.com> on 2006/04/10 23:22:08 UTC

Re: [PATCH] minor fixes to Perl SVN::Fs module

Geoff Richards wrote:
> I've attached a small patch.  My first attempt at a patch to Subversion,
> so forgive any clumsiness.
> 
> I'm working on documentation for everything in SVN::Fs, but I don't know
> how soon I'll be able to get that into shape.  I'm still trying to
> figure out how some of this stuff is meant to work.
> 
> [[[
> Minor corrections to the Perl binding in SVN::Fs, with tests.
> 
> [in subversion/bindings/swig/perl/native]
> 
>  * Fs.pm:
>    (_p_svn_fs_t): add missing 'path' method to bind svn_fs_path.
>    (_p_svn_fs_root_t): add new method 'txn_name' as binding for
>    svn_fs_txn_root_name.
>  * t/2fs.t:
>    added tests for the new path() and txn_name() methods, and
>    also for the previously untested function SVN::Fs::type().
> ]]]

Thanks for the patch!  If no one does anything with it in the next few
days, I'll file an issue for it in the issue tracker.

-- 
Michael W Thelen
It is a mistake to think you can solve any major problems just with
potatoes.       -- Douglas Adams

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: [PATCH] minor fixes to Perl SVN::Fs module

Posted by Garrett Rooney <ro...@electricjellyfish.net>.
On 4/10/06, Michael W Thelen <mi...@pietdepsi.com> wrote:
> Geoff Richards wrote:
> > I've attached a small patch.  My first attempt at a patch to Subversion,
> > so forgive any clumsiness.
> >
> > I'm working on documentation for everything in SVN::Fs, but I don't know
> > how soon I'll be able to get that into shape.  I'm still trying to
> > figure out how some of this stuff is meant to work.
> >
> > [[[
> > Minor corrections to the Perl binding in SVN::Fs, with tests.
> >
> > [in subversion/bindings/swig/perl/native]
> >
> >  * Fs.pm:
> >    (_p_svn_fs_t): add missing 'path' method to bind svn_fs_path.
> >    (_p_svn_fs_root_t): add new method 'txn_name' as binding for
> >    svn_fs_txn_root_name.
> >  * t/2fs.t:
> >    added tests for the new path() and txn_name() methods, and
> >    also for the previously untested function SVN::Fs::type().
> > ]]]
>
> Thanks for the patch!  If no one does anything with it in the next few
> days, I'll file an issue for it in the issue tracker.

No need, committed in r19312, thanks for the reminder Michael, and
thanks for the patch Geoff!

-garrett

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org