> Isn't this already fixed on the 2.5 branch? Yep, commit 8a41e54 As for this: > > > *) htmlstrip.c gets installed as source to /usr/lib/cyrus-imapd. It seems > > > it doesn't get compiled at all and the source gets installed instead. That's weird. htmlstrip is a tool used in the build process only, it shouldn't get installed anywhere (nor should its source). Will see if I can reproduce it and work out why it happens. Cheers, ellie On Mon, Aug 22, 2016, at 09:14 AM, ellie timoney via Info-cyrus wrote: > Isn't this already fixed on the 2.5 branch? > > On Mon, Aug 22, 2016, at 02:17 AM, Ken Murchison via Info-cyrus wrote: > > I'm guessing we should steal json_support.h from master and add it to > > the 2.5 branch. > > > > > > On 08/20/2016 10:40 AM, Wolfgang Breyha via Info-cyrus wrote: > > > After modifying my spec file from 2.5.8 to 2.5.9 I ended up with two new > > > issues: > > > > > > *) jcal.c can't be compiled with jansson < 2.7 (eg. every RHEL) because > > > json_boolean_value() is missing. I fixed it with: > > > ------------- > > > --- cyrus-imapd-2.5.9/imap/jcal.h.orig 2016-08-20 16:20:52.049359274 +0200 > > > +++ cyrus-imapd-2.5.9/imap/jcal.h 2016-08-20 16:21:03.719723892 +0200 > > > @@ -50,6 +50,10 @@ > > > > > > #include "util.h" > > > > > > +#if (JANSSON_MINOR_VERSION < 7) > > > +#define json_boolean_value json_is_true > > > +#endif > > > + > > > extern char *icalcomponent_as_jcal_string(icalcomponent* comp); > > > extern icalcomponent *jcal_string_as_icalcomponent(const char *str); > > > extern const char *begin_jcal(struct buf *buf); > > > ------------ > > > > > > *) htmlstrip.c gets installed as source to /usr/lib/cyrus-imapd. It seems > > > it doesn't get compiled at all and the source gets installed instead. > > > > > > Greetings, Wolfgang > > > > -- > > Kenneth Murchison > > Principal Systems Software Engineer > > Carnegie Mellon University > > > > ---- > > Cyrus Home Page: http://www.cyrusimap.org/ > > List Archives/Info: http://lists.andrew.cmu.edu/pipermail/info-cyrus/ > > To Unsubscribe: > > https://lists.andrew.cmu.edu/mailman/listinfo/info-cyrus > ---- > Cyrus Home Page: http://www.cyrusimap.org/ > List Archives/Info: http://lists.andrew.cmu.edu/pipermail/info-cyrus/ > To Unsubscribe: > https://lists.andrew.cmu.edu/mailman/listinfo/info-cyrus ---- Cyrus Home Page: http://www.cyrusimap.org/ List Archives/Info: http://lists.andrew.cmu.edu/pipermail/info-cyrus/ To Unsubscribe: https://lists.andrew.cmu.edu/mailman/listinfo/info-cyrus
