[prev in list] [next in list] [prev in thread] [next in thread] 

List:       cyrus-devel
Subject:    Re: commit "mboxevent: Rewrite JSON formatting"
From:       Greg Banks <gnb () fastmail ! fm>
Date:       2012-08-24 20:23:41
Message-ID: FDDB35F1-DCE3-4725-958A-8529467CA5F0 () fastmail ! fm
[Download RAW message or body]



Sent from my iPhone

On 24/08/2012, at 19:49, Sébastien Michel <sebastien.michel@atos.net> wrote:

> 2012/8/24 Greg Banks <gnb@fastmail.fm>:
> > > If nobody else expresses opinion, whether to put xjson in libcyrus or \
> > > libcyrus_imap, it is up to you.  I just told you my opinion.
> > 
> > My 2c: JSON code is a utility and should go in libcyrus.
> > 
> Indeed.
> 
> As discussed on IRC, we decided to change the library to format to
> JSON, from libjson (that will support 64bit integer only in the next
> release) to jansson that is also a mature library, available on major
> Linux distro and already support 64bit.
> 

Cool.

> The last debate is on the bugzilla ticket #3605. It is about removing
> the internal xjson.[ch] json formatter and add a default option
> --disable-event-notification or --enable-event-notification and force
> requirement on jansson library if enabled.
> That involves decorating the code with a C macro in all source files
> that refer to mboxevent.h.

Personally I'd be happier with only one set of code to test.

Greg.


[prev in list] [next in list] [prev in thread] [next in thread]