Re: Object oriented binds for the Subversion client library
| From: | Chris Utz | Date: | Thu, 26 Mar 2009 02:39:36 +0000 |
| Subject: | Re: Object oriented binds for the Subversion client library | ||
| References: | 1 2 | Groups: | php.pecl.dev |
| Request: | Send a blank email to pecl-dev+get-6262@lists.php.net to get a copy of this message | ||
Hi Philip,
- I will start looking through the bugs for the existing svn extension and
ensure they don't exist in the one I am working on.
- The extension I am building currently provides support for the subversion
client library only, the repository and filesystem functions are not
exposed. I think that the client library functions provide what the
majority of people would be interested in. But, if it is decided that my
extension should replace the current extension, I could extend the scope to
include other parts of the SVN library.
- I committed the patch you sent me and ran the test suite over 5.3.0RC1.
Everything appears to be working properly with 5.3, at least so far.
- I am willing to write documentation for the extension.
Regards,
Chris
On Mon, Mar 23, 2009 at 11:40 PM, Philip Olson <philip@roshambo.org> wrote:
> Please let me know if this module is suitable for inclusion in PECL.
>>
>
> Hello Chris,
>
> Looks like a nice API. And granted it's easy for me to say... I'd be in
> favor of it replacing the current but some thoughts:
>
> - See what Scott has to say. He's the active maintainer and essentially
> took over to close bugs but since then has taken over more and more things
> in the php.net world (busy)
> - Go over current (and closed) pecl/svn bugs and be sure they don't exist
> in yours:
> --
> http://pecl.php.net/bugs/search.php?cmd=display&status=All&package_name
> []=svn
> - Ensure all features in current pecl/svn are covered
> - Promise to write documentation... :)
>
> Not sure if maintaining both APIs is worth it, I would say no. But maybe
> someone will create a wrapper so the current API would still work.
>
> And to build with 5_3 I had to apply this patch:
> - http://paste2.org/p/170144
>
> Regards,
> Philip
>
>