On 2022/06/11 11:30, Klemens Nanni wrote:
> On Sat, Jun 11, 2022 at 10:54:14AM +0100, Stuart Henderson wrote:
> > This should be handled by db/config.no-coreutils, but you are using
> > CONFIGURE_STYLE=simple for a GNU autoconf-based build, is there a reason
> > for that?
> 
> No, I just started with "simple" which worked and I forgot about that.
> Theo mentioned the failure and I knew how to fix it but did not check
> whether "autoconf" was set.

Normally it would be "gnu" unless you are patching files (or upstream
doesn't ship a generated file, but since "simple" worked then that's not the
case here)

> This diff builds and 'mkdir -p' is picked up even when /usr/local/bin/gmake
> is present.
> 
> That's probably better.
> I bump REVISION because its free, just in case anything changed.
> Feedback? OK?
> 
> 
> Index: Makefile
> ===================================================================
> RCS file: /cvs/ports/security/xml-security-c/Makefile,v
> retrieving revision 1.2
> diff -u -p -r1.2 Makefile
> --- Makefile  10 Jun 2022 19:12:49 -0000      1.2
> +++ Makefile  11 Jun 2022 11:22:58 -0000
> @@ -1,6 +1,7 @@
>  COMMENT =            C++ implementation of W3C security standards for XML
>  
>  V =                  2.0.4
> +REVISION =           0
>  DISTNAME =           xml-security-c-${V}
>  CATEGORIES =         security textproc
>  
> @@ -25,7 +26,9 @@ LIB_DEPENDS =               textproc/icu4c \
>                       textproc/xalan-c \
>                       textproc/xerces-c>=3.2
>  
> -CONFIGURE_STYLE =    simple
> +CONFIGURE_STYLE =    autoconf
> +AUTOCONF_VERSION =   2.69
> +AUTOMAKE_VERSION =   2.71
>  
>  # ChangeLog lists --with-xerces and --with-xalan, but
>  # a) configure does not support --with-xerces
> @@ -36,9 +39,6 @@ CONFIGURE_ARGS +=   --with-xalan=/usr/loca
>  # configure: WARNING: NSS is no longer officially supported
>  CONFIGURE_ARGS +=    --without-nss \
>                       --with-openssl
> -
> -# Do not pick up coreutils gmakedir(1) as "race-free mkdir -p"
> -CONFIGURE_ARGS +=    ac_cv_path_mkdir=/bin/mkdir
>  
>  do-test:
>       ${WRKBUILD}/xsec/xsec-xtest
> 

Reply via email to