Subject: Re: CVS commit: [gdamore-uart] src/sys/arch/m68k/fpsp
To: None <gdamore@NetBSD.org>
From: Izumi Tsutsui <tsutsui@ceres.dti.ne.jp>
List: source-changes
Date: 06/18/2006 23:12:24
gdamore@NetBSD.org wrote:
> Module Name: src
> Committed By: gdamore
> Date: Sun Jun 18 05:17:54 UTC 2006
>
> Modified Files:
> src/sys/arch/m68k/fpsp [gdamore-uart]: Makefile asm2gas
>
> Log Message:
> Use TOOL_SED for asm2gas, which makes it possible to cross-compile this
> on Solaris 10. (Solaris 10's sed doesn't like the comment lines, and
> also complains about too many commands in the input.)
Maybe you forgot to commit changes to bsd.own.mk and bsd.sys.mk,
which should define TOOL_SED, didn't you?
(maybe you also have to note TOOL_SED in bsd.README, BTW)
Anyway, I don't think it's a good idea to change asm2gas args
because some other tools might also invoke it, and I think
it's better to specify such tool commands via environments
rather than args.
How about the attached one?
---
Izumi Tsutsui
Index: sys/arch/m68k/fpsp/Makefile
===================================================================
RCS file: /cvsroot/src/sys/arch/m68k/fpsp/Makefile,v
retrieving revision 1.13.4.1
diff -u -r1.13.4.1 Makefile
--- sys/arch/m68k/fpsp/Makefile 18 Jun 2006 05:17:54 -0000 1.13.4.1
+++ sys/arch/m68k/fpsp/Makefile 18 Jun 2006 13:53:41 -0000
@@ -56,7 +56,7 @@
# For the Library Version:
#
AR? = ar
-LIB_FILTER = sed 's/fpsp.defs/l_fpsp.defs/'
+LIB_FILTER = ${TOOL_SED} 's/fpsp.defs/l_fpsp.defs/'
LIB_TARGET = lib$(TARGET).a
#
# SYS selects the template set to use
@@ -77,9 +77,9 @@
PREFIX = L_
.sa.s:
- ${HOST_SH} ${FPSPDIR}/asm2gas ${TOOL_SED} ${.IMPSRC} >${.TARGET}
+ SED=${TOOL_SED:Q} ${HOST_SH} ${FPSPDIR}/asm2gas ${.IMPSRC} >${.TARGET}
.h.defs:
- ${HOST_SH} ${FPSPDIR}/asm2gas ${TOOL_SED} ${.IMPSRC} >${.TARGET}
+ SED=${TOOL_SED:Q} ${HOST_SH} ${FPSPDIR}/asm2gas ${.IMPSRC} >${.TARGET}
.s.o:
${CC} ${AFLAGS} ${CPPFLAGS} -c -o ${.TARGET} ${.IMPSRC}
Index: sys/arch/m68k/fpsp/asm2gas
===================================================================
RCS file: /cvsroot/src/sys/arch/m68k/fpsp/asm2gas,v
retrieving revision 1.6.66.1
diff -u -r1.6.66.1 asm2gas
--- sys/arch/m68k/fpsp/asm2gas 18 Jun 2006 05:17:54 -0000 1.6.66.1
+++ sys/arch/m68k/fpsp/asm2gas 18 Jun 2006 13:53:41 -0000
@@ -40,8 +40,10 @@
# This ugly script converts assembler code from Motorola's format to a
# form that gas (MIT syntax) can digest.
-SED=$1
-cat $2 | ${SED} -e '
+if [ -z "$SED" ]; then
+ SED=sed
+fi
+cat $1 | ${SED} -e '
# format canonicalization
/[ ]IDNT[ ]/{s/^/|/;p;d;}