On Wed, September 17, 2008 12:19 am, Chuck Hagenbuch wrote:
> Quoting Diego Woitasen <[EMAIL PROTECTED]>:
>
>> Chuck/Ilia,
>>      Could you review this patch? It adds support for MYRIGHTS IMAP
>>      command to IMAP PHP extension.
>
> It's been a while since I've worked on this extension. What I can say
> is that I share the concern over using the re-use of the imap_acl_list
> global - why not have a new one for imap_myrights? Also, while adding
> this, any reason not to add LISTRIGHTS too?

No, no reason. I just add MYRIGHTS because it's what I need :). I'll add
LISTRIGHTS too.

>
> Have you checked what c-client versions have support for myrights? Do
> we need a configure check for it?

2000 support it. No configure needed.

>
> Otherwise, looks okay by my unpracticed C eyes, and obviously this
> would benefit Horde/Kolab, so I'd like to see this get in eventually.

Latest Horde/Kolab versions use it, right now.

>
> -chuck
>

So, Who is the mantainer of this extension? or who is the right person to
talk about the inclusion of this patch?


-- 
Diego Woitasen
XTECH - Soluciones Linux para empresas
(54) 011 5219-0678


-- 
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to