Re: commit mboxevent: Rewrite JSON formatting

2012-08-24 Thread Greg Banks
Sent from my iPhone On 16/08/2012, at 0:46, Bron Gondwana br...@fastmail.fm wrote: On Wed, Aug 15, 2012, at 01:41 PM, Leena Heino wrote: On 14.8.2012 20:07, Sébastien Michel wrote: I would say the minimal version of pkg-config. Indeed, version 0.24 or greater is required in order to

Re: commit mboxevent: Rewrite JSON formatting

2012-08-24 Thread Greg Banks
Sent from my iPhone On 15/08/2012, at 2:40, Дилян Палаузовdilyan.palau...@aegee.org wrote: Hello Michel, according to my understanding, there are no strict rules what goes in libcyrus, libcyrus_min and libcyrus_imap . I am not aware of a guidelines, describing in which library to

Re: commit mboxevent: Rewrite JSON formatting

2012-08-24 Thread Sébastien Michel
2012/8/24 Greg Banks g...@fastmail.fm: I wouldn't describe RHEL 5 and 6 as old yet, their end of production dates are in 2017 and 2020 respectively. https://access.redhat.com/support/policy/updates/errata/ You're right. Running such systems out to a large fraction of their vendors'

Re: commit mboxevent: Rewrite JSON formatting

2012-08-24 Thread Sébastien Michel
2012/8/24 Greg Banks g...@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

Re: commit mboxevent: Rewrite JSON formatting

2012-08-24 Thread Greg Banks
Sent from my iPhone On 24/08/2012, at 19:49, Sébastien Michel sebastien.mic...@atos.net wrote: 2012/8/24 Greg Banks g...@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

Re: commit mboxevent: Rewrite JSON formatting

2012-08-16 Thread Sébastien Michel
That requirement would rule out systems like RHEL 5, RHEL 6. It is thus preferable to not require such version. On older versions of pkg-config (0.24), the macro will not call AC_SUBST on generated variables prefix_LIBS and prefix_CFLAGS . Running it twice, though, will not cause problems. I

Re: commit mboxevent: Rewrite JSON formatting

2012-08-15 Thread Leena Heino
On 14.8.2012 20:07, Sébastien Michel wrote: I would say the minimal version of pkg-config. Indeed, version 0.24 or greater is required in order to avoid the duplicate AC_SUBST macro calls for PKG_CHECK_MODULES substitutions. This version is available since May 2010. That requirement would rule

Re: commit mboxevent: Rewrite JSON formatting

2012-08-15 Thread Bron Gondwana
On Wed, Aug 15, 2012, at 01:41 PM, Leena Heino wrote: On 14.8.2012 20:07, Sébastien Michel wrote: I would say the minimal version of pkg-config. Indeed, version 0.24 or greater is required in order to avoid the duplicate AC_SUBST macro calls for PKG_CHECK_MODULES substitutions. This

Re: commit mboxevent: Rewrite JSON formatting

2012-08-14 Thread Sébastien Michel
please delete imap/Makefile.in : all the build rules are in /Makefile.am . It's already done. Sorry, I just forgot to mention it. As libjson supports the .pc format, you can detect libjson in configure.ac with PKG_CHECK_MODULES ([libjson], [json = 0.10], [check_libjson=yes],

Re: commit mboxevent: Rewrite JSON formatting

2012-08-14 Thread Дилян Палаузов
Hello Michel, according to my understanding, there are no strict rules what goes in libcyrus, libcyrus_min and libcyrus_imap . I am not aware of a guidelines, describing in which library to include what kind of file. While working on getting Automake/libtool in cyrus-imapd, I moved some

Re: commit mboxevent: Rewrite JSON formatting

2012-08-14 Thread Sébastien Michel
2012/8/14 Дилян Палаузов dilyan.palau...@aegee.org: lib/parseaddr.c is used within libcyrus_sieve and imap/, so having it in the common library libcyrus seems logical. I do not know, why iostat.c is part of libcyrus. If nobody else expresses opinion, whether to put xjson in libcyrus or

RE: commit mboxevent: Rewrite JSON formatting

2012-08-13 Thread Michel Sébastien
...@aegee.org] Envoyé : dimanche 1 juillet 2012 16:37 À : Michel Sébastien Objet : commit mboxevent: Rewrite JSON formatting Hello Sebastien, you have put the support for -ljson in libcyrus_min . According to my understanding, in libcyrus_min come only the files needed by the master process