Hi Julien, I do appreciate all the work Zentyal is doing on both OpenChange and the SOGo backend, but my question is still: Why is OpenChange breaking the API with every release? Isn't it possible to provide some API stability and do changes in a way that keep things backwards compatible? I can understand that you don't want to keep compatibility code forever, but that can be avoided by doing a major release once in a while that will drop backwards compatibility or do it more gradually by deprecating the old interface in release x.y and removing it in release x.y+1.
Kind regards, Jeroen Dekkers At Thu, 13 Nov 2014 20:31:22 +0100, Julien Kerihuel wrote: > > Hi Jeroen, > > I understand your frustration, but you need to put things back into > their original context. > > Most of the API breakage you are referring are pushed as pull request to > inverse-inc repo since mid August. We are now in mid-November and there > is still no packaged version out with the proposed change. What is in > cause here is therefore not a late breakage of the API but a lack of > consistent schedule of upstream releases, not to say none. > > The truth is that today all the fixes and new features on the OpenChange > SOGo backend are contributed by Zentyal and OpenChange master primarily > synchronize and works with github.com/Zentyal/sogo. > > Cheers, > Julien. > > On 13/11/14 20:07, Jeroen Dekkers wrote: > > Why does OpenChange keep breaking the API? The API breakage in 2.2 > > almost caused SOGo to not be in Debian jessie because Jelmer uploaded > > it without knowing the API broke and SOGo didn't have the changes for > > the new API yet. > > > > Kind regards, > > > > Jeroen Dekkers > > > > At Thu, 13 Nov 2014 17:41:33 +0100, > > Kamen Mazdrashki wrote: > >> Hi Serge, > >> > >> You should be using Zentyal fork for SOGo with current Openchange. > >> https://github.com/Zentyal/sogo > >> At this point I think that not all patches from zentyal/sogo are merged > >> into inverse/sogo - so problems > >> as you have described above should be expected. > >> > >> Cheers, > >> Kamen > >> > >> On Thu, Nov 13, 2014 at 5:06 PM, Serge NOEL <serge.noel2...@gmail.com> > >> wrote: > >> > >> Sorry for missing details : > >> > >> OS : Description: Ubuntu 14.04.1 LTS with apt-get update/upgrade > >> git version : 1.9.1 > >> Samba : git version from git.samba.org/samba.git > >> Openchange : git version from github.com/openchange/openchange.git > >> Sogo : git version from github/inverse.inc/sogo > >> > >> Serge > >> > >> Le 13/11/2014 14:12, Jason Pyeron a écrit : > >> > >> -----Original Message----- > >> From: devel [mailto:devel-boun...@lists.openchange.org] On > >> Behalf Of Serge NOEL > >> Sent: Thursday, November 13, 2014 6:38 > >> To: devel@lists.openchange.org > >> Subject: [openchange][devel] Error compiling sogo/Openchange > >> > >> Hi all, > >> > >> i have installed one brand new virtual machine, and i try to > >> > >> Which > >> OS: > >> Version: > >> git ref? > >> > >> compile everything from source (from git), i get following > >> errors : > >> > >> MAPIStoreContext.m: In function '-[MAPIStoreContext > >> getNewChangeNumber]': > >> MAPIStoreContext.m:552:3: error: passing argument 2 of > >> 'openchangedb_get_new_changeNumber' from incompatible pointer > >> type [-Werror] > >> if (openchangedb_get_new_changeNumber > >> (connInfo->oc_ctx, &newVersionNumber) > >> ^ > >> In file included from MAPIStoreContext.m:57:0: > >> /usr/include/libmapiproxy.h:203:17: note: expected > >> 'const char *' but argument is of type 'uint64_t *' > >> enum MAPISTATUS > >> openchangedb_get_new_changeNumber(struct openchangedb_context > >> *, const char *, uint64_t *); > >> ^ > >> MAPIStoreContext.m:552:3: error: too few arguments to > >> function 'openchangedb_get_new_changeNumber' > >> if (openchangedb_get_new_changeNumber > >> (connInfo->oc_ctx, &newVersionNumber) > >> ^ > >> In file included from MAPIStoreContext.m:57:0: > >> /usr/include/libmapiproxy.h:203:17: note: declared here > >> enum MAPISTATUS > >> openchangedb_get_new_changeNumber(struct openchangedb_context > >> *, const char *, uint64_t *); > >> ^ > >> MAPIStoreContext.m: In function '-[MAPIStoreContext > >> getNewChangeNumbers:]': > >> MAPIStoreContext.m:571:43: error: passing argument 3 of > >> 'openchangedb_get_new_changeNumbers' makes pointer from > >> integer without a cast [-Werror] > >> memCtx, max, > >> &numbers) > >> ^ > >> In file included from MAPIStoreContext.m:57:0: > >> /usr/include/libmapiproxy.h:204:17: note: expected > >> 'const char *' but argument is of type 'uint64_t' > >> enum MAPISTATUS > >> openchangedb_get_new_changeNumbers(struct > >> openchangedb_context *, TALLOC_CTX *, const char *, uint64_t, > >> struct UI8Array_r **); > >> ^ > >> MAPIStoreContext.m:571:43: error: passing argument 4 of > >> 'openchangedb_get_new_changeNumbers' makes integer from > >> pointer without a cast [-Werror] > >> memCtx, max, > >> &numbers) > >> ^ > >> In file included from MAPIStoreContext.m:57:0: > >> /usr/include/libmapiproxy.h:204:17: note: expected > >> 'uint64_t' but argument is of type 'struct UI8Array_r **' > >> enum MAPISTATUS > >> openchangedb_get_new_changeNumbers(struct > >> openchangedb_context *, TALLOC_CTX *, const char *, uint64_t, > >> struct UI8Array_r **); > >> ^ > >> MAPIStoreContext.m:571:43: error: too few arguments to > >> function 'openchangedb_get_new_changeNumbers' > >> memCtx, max, > >> &numbers) > >> ^ > >> In file included from MAPIStoreContext.m:57:0: > >> /usr/include/libmapiproxy.h:204:17: note: declared here > >> enum MAPISTATUS > >> openchangedb_get_new_changeNumbers(struct > >> openchangedb_context *, TALLOC_CTX *, const char *, uint64_t, > >> struct UI8Array_r **); > >> ^ > >> cc1obj: all warnings being treated as errors > >> make[3]: *** > >> [obj/SOGoBackend.obj/MAPIStoreContext.m.o] Erreur 1 > >> make[2]: *** [internal-bundle-run-compile-submake] > >> Erreur 2 > >> make[1]: *** [SOGoBackend.all.bundle.variables] Erreur > >> 2 > >> make: *** [internal-all] Erreur 2 > >> > >> > >> Any help would be welcome > >> Serge > >> > >> -- > >> -=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=- > >> - - > >> - Jason Pyeron PD Inc. http://www.pdinc.us - > >> - Principal Consultant 10 West 24th Street #100 - > >> - +1 (443) 269-1555 x333 Baltimore, Maryland 21218 - > >> - - > >> -=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=- > >> This message is copyright PD Inc, subject to license 20080407P00. > >> > >> _______________________________________________ > >> devel mailing list > >> devel@lists.openchange.org > >> http://mailman.openchange.org/listinfo/devel > >> > >> _______________________________________________ > >> devel mailing list > >> devel@lists.openchange.org > >> http://mailman.openchange.org/listinfo/devel > > _______________________________________________ > > devel mailing list > > devel@lists.openchange.org > > http://mailman.openchange.org/listinfo/devel > > -- > Julien Kerihuel > j.kerih...@openchange.org > OpenChange Project Founder > > Twitter: http://twitter.com/jkerihuel > > GPG Fingerprint: 0B55 783D A781 6329 108A B609 7EF6 FE11 A35F 1F79 _______________________________________________ devel mailing list devel@lists.openchange.org http://mailman.openchange.org/listinfo/devel
