Module Name:    src
Committed By:   uebayasi
Date:           Tue Aug 25 02:57:32 UTC 2015

Modified Files:
        src/sys/conf: Makefile.kern.inc

Log Message:
Redo Rev. 1.188:
o Explicitly add dependency of `<kernel>' on `<kernel>.ldscript', instead of
  relying on ${SYSTEM_DEP}, which is evaluated much earlier than
  `sys/conf/Makefile.kern.inc' is included, to avoid evaluation ordering
  problems.
o Print a message when creating a `<kernel>.ldscript'.
o Trim empty lines in `<kernel>.ldscript'.


To generate a diff of this commit:
cvs rdiff -u -r1.189 -r1.190 src/sys/conf/Makefile.kern.inc

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/sys/conf/Makefile.kern.inc
diff -u src/sys/conf/Makefile.kern.inc:1.189 src/sys/conf/Makefile.kern.inc:1.190
--- src/sys/conf/Makefile.kern.inc:1.189	Mon Aug 24 18:02:55 2015
+++ src/sys/conf/Makefile.kern.inc	Tue Aug 25 02:57:32 2015
@@ -1,4 +1,4 @@
-#	$NetBSD: Makefile.kern.inc,v 1.189 2015/08/24 18:02:55 christos Exp $
+#	$NetBSD: Makefile.kern.inc,v 1.190 2015/08/25 02:57:32 uebayasi Exp $
 #
 # This file contains common `MI' targets and definitions and it is included
 # at the bottom of each `MD' ${MACHINE}/conf/Makefile.${MACHINE}.
@@ -229,13 +229,6 @@ SYSTEM_OBJ?=	${MD_OBJS} ${MI_OBJS} ${OBJ
 SYSTEM_OBJ?=	${MD_OBJS} ${MI_OBJS} ${OBJS:O} ${SYSLIBCOMPAT} ${LIBKERN}
 .endif
 SYSTEM_DEP+=	Makefile ${SYSTEM_OBJ} .gdbinit
-.if defined(KERNLDSCRIPT)
-.if exists(${KERNLDSCRIPT})
-SYSTEM_DEP+=	-T ${KERNLDSCRIPT}
-.else
-SYSTEM_DEP+=	${.TARGET}.ldscript
-.endif
-.endif
 .if defined(CTFMERGE)
 SYSTEM_CTFMERGE= ${CTFMERGE} ${CTFMFLAGS} -o ${.TARGET} ${SYSTEM_OBJ} ${EXTRA_OBJ} vers.o
 .else
@@ -257,15 +250,13 @@ ${KERNLDSCRIPT}: ${SYSTEM_OBJ}
 .if defined(KERNLDSCRIPT)
 .for k in ${KERNELS}
 EXTRA_CLEAN+=	${k}.ldscript
+${k}: ${k}.ldscript
 ${k}.ldscript: ${KERNLDSCRIPT}
-	${CPP} ${KERNLDSCRIPT} | grep -v '^#' >$@
+	${_MKTARGET_CREATE}
+	${CPP} ${KERNLDSCRIPT} | grep -v '^#' | grep -v '^$$' >$@
 .endfor
-.if exists(${KERNLDSCRIPT})
-LINKSCRIPT=	-T ${KERNLDSCRIPT}
-.else
 LINKSCRIPT=	-T ${.TARGET}.ldscript
 .endif
-.endif
 
 TEXTADDR?=	${LOADADDRESS}			# backwards compatibility
 LINKTEXT?=	${TEXTADDR:C/.+/-Ttext &/}

Reply via email to