commit "mboxevent: Rewrite JSON formatting"

Greg Banks gnb at
Fri Aug 24 16:23:41 EDT 2012

Sent from my iPhone

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

> 2012/8/24 Greg Banks <gnb at>:
>>> 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.


> 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.


More information about the Cyrus-devel mailing list