Re: [macppc / lld] Unbreak games/corewars

2021-09-04 Thread George Koehler
On Sat,  4 Sep 2021 14:51:21 +
Charlene Wendling  wrote:

> Hi,
> 
> Corewars' build failed in the current macppc bulk:
> 
> > https://bin.charlenew.xyz/corewars.fail.log
> 
> Once again the `-lm' linker flag is needed on macppc. The below diff
> allows me to build and run corewars on macppc [0], with no regression on
> amd64.
> 
> While here, i've added a missing tab for CONFIGURE_STYLE.
> 
> OK?

ok gkoehler@

> Charlène.
> 
> 
> [0] https://bin.charlenew.xyz/corewars.fixed.log
> 
> 
> Index: Makefile
> ===
> RCS file: /cvs/ports/games/corewars/Makefile,v
> retrieving revision 1.29
> diff -u -p -u -p -r1.29 Makefile
> --- Makefile  28 Feb 2021 06:13:34 -  1.29
> +++ Makefile  4 Sep 2021 14:48:39 -
> @@ -3,7 +3,7 @@
>  COMMENT= computer simulation game
>  
>  DISTNAME=corewars-0.9.13
> -REVISION=10
> +REVISION=11
>  
>  CATEGORIES=  games x11
>  
> @@ -23,7 +23,10 @@ LIB_DEPENDS=   x11/gtk+2
>  MASTER_SITES=${MASTER_SITE_SOURCEFORGE:=corewars/}
>  
>  AUTOCONF_VERSION=2.59
> -CONFIGURE_STYLE=autoconf
> +CONFIGURE_STYLE= autoconf
> +
> +# Needed for architectures not inlining sqrt(3), like powerpc
> +CONFIGURE_ENV=   LDFLAGS="${LDFLAGS} -lm"
>  
>  post-install:
>   ${INSTALL_DATA_DIR} ${PREFIX}/share/doc/corewars
> 


-- 
George Koehler 



[macppc / lld] Unbreak games/corewars

2021-09-04 Thread Charlene Wendling
Hi,

Corewars' build failed in the current macppc bulk:

> https://bin.charlenew.xyz/corewars.fail.log

Once again the `-lm' linker flag is needed on macppc. The below diff
allows me to build and run corewars on macppc [0], with no regression on
amd64.

While here, i've added a missing tab for CONFIGURE_STYLE.

OK?

Charlène.


[0] https://bin.charlenew.xyz/corewars.fixed.log


Index: Makefile
===
RCS file: /cvs/ports/games/corewars/Makefile,v
retrieving revision 1.29
diff -u -p -u -p -r1.29 Makefile
--- Makefile28 Feb 2021 06:13:34 -  1.29
+++ Makefile4 Sep 2021 14:48:39 -
@@ -3,7 +3,7 @@
 COMMENT=   computer simulation game
 
 DISTNAME=  corewars-0.9.13
-REVISION=  10
+REVISION=  11
 
 CATEGORIES=games x11
 
@@ -23,7 +23,10 @@ LIB_DEPENDS= x11/gtk+2
 MASTER_SITES=  ${MASTER_SITE_SOURCEFORGE:=corewars/}
 
 AUTOCONF_VERSION=  2.59
-CONFIGURE_STYLE=autoconf
+CONFIGURE_STYLE=   autoconf
+
+# Needed for architectures not inlining sqrt(3), like powerpc
+CONFIGURE_ENV= LDFLAGS="${LDFLAGS} -lm"
 
 post-install:
${INSTALL_DATA_DIR} ${PREFIX}/share/doc/corewars