Re: Build failed in Jenkins: cyrus-imapd-master #1059

2013-01-09 Thread Sébastien Michel
2013/1/9 Greg Banks g...@fastmail.fm: Those can be made cleaner anyway, by moving them into mboxevent.c. Then the mailbox.c code would always call mboxevent_* functions but those functions would, depending on the result of ./configure, either be a trivial stub that does do nothing or would

Re: Build failed in Jenkins: cyrus-imapd-master #1059

2013-01-08 Thread Greg Banks
On Tue, Jan 8, 2013, at 10:02 PM, Sébastien Michel wrote: 2013/1/3 Bron Gondwana br...@fastmail.fm: I vote for using the technically best library. It's probably small enough to that we can ship it inside the Cyrus tarballs if we need. It's only 132k in the src directory of the

Re: Build failed in Jenkins: cyrus-imapd-master #1059

2013-01-03 Thread Greg Banks
On Wed, Jan 2, 2013, at 09:31 PM, Sébastien Michel wrote: 2013/1/2 Bron Gondwana br...@fastmail.fm: Sounds good to me. I had a brief glance through today. It looks nice at a first reading. I have a couple of questions: Thank you. We can also warmly thank Greg who has taken some time

Re: Build failed in Jenkins: cyrus-imapd-master #1059

2013-01-03 Thread Greg Banks
On Thu, Jan 3, 2013, at 01:20 AM, Jeroen van Meeuwen (Kolab Systems) wrote: On 2013-01-02 15:07, Sébastien Michel wrote: I used initially json-c that is the most common. However, only the trunk offered support to 64bit integers. This is why I switched to jansson that is also valuable.

Re: Build failed in Jenkins: cyrus-imapd-master #1059

2013-01-02 Thread Sébastien Michel
2013/1/1 Jenkins do-not-re...@cyrusimap.org: See http://ci.cyrusimap.org/job/cyrus-imapd-master/1059/ checking for pkg-config... /usr/bin/pkg-config checking pkg-config is at least version 0.9.0... yes checking for JANSSON... no configure: error: jansson 2.0 or later was not found, but is

Re: Build failed in Jenkins: cyrus-imapd-master #1059

2013-01-02 Thread Sébastien Michel
2013/1/2 Jeroen van Meeuwen (Kolab Systems) vanmeeu...@kolabsys.com: On 2013-01-02 10:16, Sébastien Michel wrote: 3 options in this case: - Install jansson library on the jenkins server - Add --disable-event-notification in the Cyrus build script - Disable event notification by default at

Re: Build failed in Jenkins: cyrus-imapd-master #1059

2013-01-02 Thread Jeroen van Meeuwen (Kolab Systems)
On 2013-01-02 15:07, Sébastien Michel wrote: I used initially json-c that is the most common. However, only the trunk offered support to 64bit integers. This is why I switched to jansson that is also valuable. Unfortunately, it is less common than json-c, and for which the version 0.10 is now

Re: Build failed in Jenkins: cyrus-imapd-master #1059

2013-01-02 Thread Bron Gondwana
On Thu, Jan 3, 2013, at 01:20 AM, Jeroen van Meeuwen (Kolab Systems) wrote: On 2013-01-02 15:07, Sébastien Michel wrote: I used initially json-c that is the most common. However, only the trunk offered support to 64bit integers. This is why I switched to jansson that is also valuable.

Build failed in Jenkins: cyrus-imapd-master #1059

2013-01-01 Thread Jenkins
See http://ci.cyrusimap.org/job/cyrus-imapd-master/1059/ -- [...truncated 2198 lines...] autoconf/general.m4:2670: _AC_LINK_IFELSE is expanded from... autoconf/general.m4:2680: AC_LINK_IFELSE is expanded from... cmulocal/libtool.m4:1100: