[Clamav-devel] Linux, configure.ac: automake 1.16, AC_SYS_LARGEFILE

[Micah Snyder (micasnyd)]

Sergey,

I realize it's a bit of a delay since we talked about it, but we just merged the change you recommended:
https://github.com/Cisco-Talos/clamav-devel/commit/c72b1839a6b32827ba4fb18fd245f646b2c930e9

-Micah

On 5/8/19, 11:37 AM, "clamav-devel on behalf of Sergey" <clamav-devel-bounces at lists.clamav.net on behalf of a_s_y at sama.ru> wrote:

    On Friday 26 April 2019, Micah Snyder (micasnyd) wrote:
    
    >    > 1. automake 1.16
    >    > 
    >    > ClamAV isn't built if autoreconf was used. AC_CONFIG_AUX_DIR([config])
    >    > should be placed after AC_INIT before all other.
    >     
    > Do you suggest this? 
    > 
    > diff --git a/configure.ac b/configure.ac
    > index b333e3eb2..fbb62526c 100644
    > --- a/configure.ac
    > +++ b/configure.ac
    > @@ -23,13 +23,13 @@ AC_PREREQ([2.59])
    >  dnl For a release change [devel] to the real version [0.xy]
    >  dnl also change VERSION below
    >  AC_INIT([ClamAV], [0.102.0-devel], [https://bugzilla.clamav.net/], [clamav], [https://www.clamav.net/])
    > +AC_CONFIG_AUX_DIR([config])
    > 
    >  dnl enable C++
    >  AC_PROG_CXX()
    > 
    >  AH_BOTTOM([#include "platform.h"])
    >  dnl put configure auxiliary into config
    > -AC_CONFIG_AUX_DIR([config])
    >  AC_CONFIG_HEADERS([clamav-config.h libclammspack/config.h:libclammspack/config.h.in])
    > 
    >  dnl safety check, this used to be a parameter to AC_INIT
    
    Yes. And with comment "dnl put configure auxiliary into config"
    
    -- 
    Regards,
    Sergey
    _______________________________________________
    
    clamav-devel mailing list
    clamav-devel at lists.clamav.net
    https://lists.clamav.net/mailman/listinfo/clamav-devel
    
    Please submit your patches to our Bugzilla: http://bugzilla.clamav.net
    
    Help us build a comprehensive ClamAV guide:
    https://github.com/vrtadmin/clamav-faq
    
    http://www.clamav.net/contact.html#ml