Hi Sebastian,

I think the attached patch should restore compatibility with autoconf
version 2.63 -- are you able to confirm?

Cheers,

ellie

On Mon, Aug 8, 2016, at 10:07 AM, ellie timoney via Cyrus-devel wrote:
> Yep, it's due to the use of the 'm4_esyscmd_s' macro.  I've seen a few
> other projects achieve 2.63 compatibility just by detecting whether this
> macro is available, and providing their own replacement if not.  I plan
> to do something like this too, hopefully before next release, but
> haven't yet looked into it deeply.  Would welcome a patch in the
> meantime!
> 
> On Thu, Aug 4, 2016, at 10:26 PM, Sebastian Hagedorn wrote:
> > Hi Ellie,
> > 
> > your commit f5d8e1c77d2ed40ee734eb0c5627f425c595f33f requires Autoconf 
> > version 2.68 or higher, but CentOS 6 only has Autoconf 2.63 by default. 
> > There is a separate RPM for Autoconf 2.68, and after I installed that it 
> > works from within the git repository if I use "autoreconf268 -vi"
> > manually.
> > 
> > I would also note that the dependencies here list Autoconf 2.63:
> > 
> > <http://cyrusimap.org/imap/installation/diy.html>
> > 
> > I have a related, but separate issue, for which I will write another
> > mail.
> > 
> > Cheers
> > Sebastian
> > -- 
> >     .:.Sebastian Hagedorn - Weyertal 121 (Gebäude 133), Zimmer 2.02.:.
> >                  .:.Regionales Rechenzentrum (RRZK).:.
> >    .:.Universität zu Köln / Cologne University - ✆ +49-221-470-89578.:.
> > Email had 1 attachment:
> > + Attachment2
> >   1k (application/pgp-signature)
diff --git a/configure.ac b/configure.ac
index a773fa6..5665b5b 100644
--- a/configure.ac
+++ b/configure.ac
@@ -45,11 +45,11 @@ dnl
 
 
 AC_INIT([cyrus-imapd],
-        m4_esyscmd_s(git describe --dirty=-dirty | sed -e 's/^cyrus-imapd-//'),
+        m4_esyscmd(printf '%s' $(git describe --dirty=-dirty | sed -e 's/^cyrus-imapd-//')),
         [https://github.com/cyrusimap/cyrus-imapd/issues],
         ,
         [https://www.cyrusimap.org])
-AC_PREREQ([2.68])
+AC_PREREQ([2.63])
 AC_CONFIG_SRCDIR([imap/imapd.c])
 AC_CONFIG_HEADERS(config.h)
 AC_CONFIG_MACRO_DIR([cmulocal])

Reply via email to