Re: CVS commit: src/sys/arch/zaurus/conf

[Nick Hudson <nick.hudson%gmx.co.uk@localhost>]

On 03/01/2020 18:42, Izumi Tsutsui wrote:
> I wrote:
> > > The missing COPTS was an accident or fallout from some other changes?
> >
> > Isn't it specified in -current?
>
> The problem is caused by sys/arch/arm/conf/Makefile.arm.
> It defines "COPTS+= -mapcs-frame" in recent rev 1.52
>   http://cvsweb.netbsd.org/bsdweb.cgi/src/sys/arch/arm/conf/Makefile.arm#rev1.52
> but MI sys/conf/Makefile.kern.inc defines COPTS+=-O2
> only if COPTS is empty.

I don't see how as sys/arch/zaurus/conf/INSTALL doesn't define DDB.

> This affects not only zaurus but all arm ports?

Maybe this patch is better?

Nick

Index: sys/arch/arm/conf/Makefile.arm
===================================================================
RCS file: /cvsroot/src/sys/arch/arm/conf/Makefile.arm,v
retrieving revision 1.52
diff -u -p -r1.52 Makefile.arm
--- sys/arch/arm/conf/Makefile.arm	2 Jan 2020 14:33:55 -0000	1.52
+++ sys/arch/arm/conf/Makefile.arm	4 Jan 2020 15:02:40 -0000
@@ -78,7 +78,7 @@ CFLAGS+=	-mno-unaligned-access
 
 OPT_DDB=	%DDB%
 .if !empty(OPT_DDB) && ${HAVE_GCC:U0} > 0
-COPTS+=		-mapcs-frame
+DEFCOPTS=		-O2 -mapcs-frame
 .endif
 
 ##