Re: [Evolution-hackers] Camel IMAPX RFC5464 compliance

2010-08-21 Thread David Woodhouse
Please don't drop me from Cc when replying to my messages.
See http://david.woodhou.se/reply-to-list.html

On Wed, 2010-08-18 at 10:30 +0200, Christian Hilberg wrote:
> Hi everyone,
> 
> On Thursday 05 August 2010 David Woodhouse wrote:
> > On Wed, 2010-08-04 at 12:22 +0200, Christian Hilberg wrote:
> > > Now, I would like to know how we should deal with the issue. We (the
> > > evolution-kolab developers) could patch the 2.30 version of IMAPX only to
> > > get things running. In this case, would our additions be pulled
> > > upstream?
> > [...] 
> > I would strongly recommend that you do it in the development branch
> > first, then we can backport it to gnome-2-30.
> > I've been backporting most IMAPX changes from master to the 2.30 branch;
> > I see no particular reason why we shouldn't backport METADATA support
> > too, as long as you're careful not to add new user-visible strings that
> > would need translation.
> 
> Okay, let's say, we will patch upstream IMAPX to support RFC5464. The patch 
> gets reviewed, and after being polished it will (hopefully :-) be accepted in 
> upstream.
> 
> How long do you think it would take you to backport such a patch to 2.30, 
> assuming we heed to the aforementioned implementation recommendations?

Just to clarify -- I would expect you to do the backport yourself.

When I said I've been backporting most IMAPX changes from master to
2.30... that's because I'd been making most of them. I wasn't
volunteering to backport *your* changes too. :)

It shouldn't be that hard. Probably even less than a day -- I suspect
Chen was a little pessimistic in his estimate.

-- 
David WoodhouseOpen Source Technology Centre
david.woodho...@intel.com  Intel Corporation

___
evolution-hackers mailing list
evolution-hackers@gnome.org
To change your list options or unsubscribe, visit ...
http://mail.gnome.org/mailman/listinfo/evolution-hackers


Re: [Evolution-hackers] Camel IMAPX RFC5464 compliance

2010-08-18 Thread chen
On Wed, 2010-08-18 at 10:30 +0200, Christian Hilberg wrote:
> Hi everyone,
> 
> On Thursday 05 August 2010 David Woodhouse wrote:
> > On Wed, 2010-08-04 at 12:22 +0200, Christian Hilberg wrote:
> > > Now, I would like to know how we should deal with the issue. We (the
> > > evolution-kolab developers) could patch the 2.30 version of IMAPX only to
> > > get things running. In this case, would our additions be pulled
> > > upstream?
> > [...] 
> > I would strongly recommend that you do it in the development branch
> > first, then we can backport it to gnome-2-30.
> > I've been backporting most IMAPX changes from master to the 2.30 branch;
> > I see no particular reason why we shouldn't backport METADATA support
> > too, as long as you're careful not to add new user-visible strings that
> > would need translation.
> 
> Okay, let's say, we will patch upstream IMAPX to support RFC5464. The patch 
> gets reviewed, and after being polished it will (hopefully :-) be accepted in 
> upstream.
> 
> How long do you think it would take you to backport such a patch to 2.30, 
> assuming we heed to the aforementioned implementation recommendations?

It should just take a day or two once its approved. You just need to
keep in mind that its not possible to add string/UI changes in
gnome-2-30 branch. I am assuming that string changes would not be need
since you would just check for server capabilities and add the meta
data..


- Chenthill.
> 
> Best regards,
> 
>   Christian
> 
> ___
> evolution-hackers mailing list
> evolution-hackers@gnome.org
> To change your list options or unsubscribe, visit ...
> http://mail.gnome.org/mailman/listinfo/evolution-hackers



___
evolution-hackers mailing list
evolution-hackers@gnome.org
To change your list options or unsubscribe, visit ...
http://mail.gnome.org/mailman/listinfo/evolution-hackers


Re: [Evolution-hackers] Camel IMAPX RFC5464 compliance

2010-08-18 Thread Christian Hilberg
Hi everyone,

On Thursday 05 August 2010 David Woodhouse wrote:
> On Wed, 2010-08-04 at 12:22 +0200, Christian Hilberg wrote:
> > Now, I would like to know how we should deal with the issue. We (the
> > evolution-kolab developers) could patch the 2.30 version of IMAPX only to
> > get things running. In this case, would our additions be pulled
> > upstream?
> [...] 
> I would strongly recommend that you do it in the development branch
> first, then we can backport it to gnome-2-30.
> I've been backporting most IMAPX changes from master to the 2.30 branch;
> I see no particular reason why we shouldn't backport METADATA support
> too, as long as you're careful not to add new user-visible strings that
> would need translation.

Okay, let's say, we will patch upstream IMAPX to support RFC5464. The patch 
gets reviewed, and after being polished it will (hopefully :-) be accepted in 
upstream.

How long do you think it would take you to backport such a patch to 2.30, 
assuming we heed to the aforementioned implementation recommendations?

Best regards,

Christian

-- 
kernel concepts GbRTel: +49-271-771091-14
Sieghuetter Hauptweg 48Fax: +49-271-771091-19
D-57072 Siegen
http://www.kernelconcepts.de/


signature.asc
Description: This is a digitally signed message part.
___
evolution-hackers mailing list
evolution-hackers@gnome.org
To change your list options or unsubscribe, visit ...
http://mail.gnome.org/mailman/listinfo/evolution-hackers


Re: [Evolution-hackers] Camel IMAPX RFC5464 compliance

2010-08-05 Thread David Woodhouse
On Wed, 2010-08-04 at 12:22 +0200, Christian Hilberg wrote:
> Now, I would like to know how we should deal with the issue. We (the 
> evolution-kolab developers) could patch the 2.30 version of IMAPX only to get 
> things running. In this case, would our additions be pulled upstream?
>   As an alternative, would anyone like to implement RFC5464 in the current 
> upstream IMAPX so we could try and backport the changes into 2.30?

I would strongly recommend that you do it in the development branch
first, then we can backport it to gnome-2-30.

I've been backporting most IMAPX changes from master to the 2.30 branch;
I see no particular reason why we shouldn't backport METADATA support
too, as long as you're careful not to add new user-visible strings that
would need translation.

-- 
David WoodhouseOpen Source Technology Centre
david.woodho...@intel.com  Intel Corporation


___
evolution-hackers mailing list
evolution-hackers@gnome.org
To change your list options or unsubscribe, visit ...
http://mail.gnome.org/mailman/listinfo/evolution-hackers


Re: [Evolution-hackers] Camel IMAPX RFC5464 compliance

2010-08-04 Thread Christian Hilberg
Hi again.

On Monday 26 July 2010 Christian Hilberg wrote:
> while I suspect the answer will most likely be "no", just to be sure I'd
> like to put the question here anyway (if only for the record):
> Does the Camel IMAPX implementation comply with RFC5464 "The IMAP METADATA
> Extension" [1] ?
> [...]
> [1] http://www.faqs.org/rfcs/rfc5464.html

After taking a closer look at the IMAPX implementation (and since there was no 
veto here), it seems clear that the 2.30 IMAPX does not support the RFC5464 
IMAP protocol extension.

Now, we need this functionality in our evolution-kolab plugin to avoid ugly 
workarounds (like scanning all folder contents in order to find out the folder 
type) when working with Kolab IMAP (PIM) Folders.

We could patch the IMAPX implementation to add RFC5464 functionality. This 
would mean that IMAPX needed to be extended by two new IMAP commands 
(SETMETADATA and GETMETADATA), and one new response (METADATA). The 
GETMETADATA command has two options, MAXSIZE and DEPTH. The METADATA response 
may carry values. For further details, please see RFC5464.

In all, it does not seem to be overly complicated. However, apart from 
implementing the protocol extension itself, it would mean to also extend the 
IMAPX API. This should be possible to implement just as an extension to the 
existing API so we would not break anything, right?

Now, I would like to know how we should deal with the issue. We (the 
evolution-kolab developers) could patch the 2.30 version of IMAPX only to get 
things running. In this case, would our additions be pulled upstream?
  As an alternative, would anyone like to implement RFC5464 in the current 
upstream IMAPX so we could try and backport the changes into 2.30?

Best regards,

Christian

-- 
kernel concepts GbRTel: +49-271-771091-14
Sieghuetter Hauptweg 48Fax: +49-271-771091-19
D-57072 Siegen
http://www.kernelconcepts.de/


signature.asc
Description: This is a digitally signed message part.
___
evolution-hackers mailing list
evolution-hackers@gnome.org
To change your list options or unsubscribe, visit ...
http://mail.gnome.org/mailman/listinfo/evolution-hackers


[Evolution-hackers] Camel IMAPX RFC5464 compliance

2010-07-26 Thread Christian Hilberg
Hi everyone,

while I suspect the answer will most likely be "no", just to be sure I'd like 
to put the question here anyway (if only for the record):

Does the Camel IMAPX implementation comply with RFC5464 "The IMAP METADATA 
Extension" [1] ?

Best regards,

Christian

[1] http://www.faqs.org/rfcs/rfc5464.html

-- 
kernel concepts GbRTel: +49-271-771091-14
Sieghuetter Hauptweg 48Fax: +49-271-771091-19
D-57072 Siegen
http://www.kernelconcepts.de/


signature.asc
Description: This is a digitally signed message part.
___
evolution-hackers mailing list
evolution-hackers@gnome.org
To change your list options or unsubscribe, visit ...
http://mail.gnome.org/mailman/listinfo/evolution-hackers