Re: bzero.S and assym.h (Re: CVS commit: syssrc/sys/lib/libkern/arch/powerpc)

[Izumi Tsutsui <tsutsui%ceres.dti.ne.jp@localhost>]

In article <newscache$mt5ong$mt2$1%maschndrohtzaun.emsi.priv.at@localhost>
mjl%emsi.priv.at@localhost wrote:

> |  # make depend
> |  depending the kern library objects
> |  mkdep -a -D_LOCORE -I../../../../../../lib/libkern/arch/powerpc \
> [..]
> |  ../../../../../../lib/libkern/arch/powerpc/bzero.S:33: \
> |  assym.h: No such file or directory
> 
>   You are right, there's some dependency missing somewhere.
> Do you know the right place to add this? I'm afraid the intricate
> maze of Makefiles all alike, will always remain a mystery to me...

My workaround is to add dependency of assym.h to Makefile.powerpc
(patch attached), but I think it is a bit ugly.

If powerpc/bzero.S is the only asm source which requires assym.h,
my patch would be OK, but I think this should be MI and
sys/lib/libkern/Makefile.inc should have this dependency.

BTW, current bzero.S requires assym.h if libkern is built
for kernel, but how about libkern for standalone programs?
If libkern for SA programs also requires CACHELINESIZE,
we have to prepare another method..
---
Izumi Tsutsui
tsutsui%ceres.dti.ne.jp@localhost

Index: Makefile.powerpc
===================================================================
RCS file: /cvsroot/syssrc/sys/arch/powerpc/conf/Makefile.powerpc,v
retrieving revision 1.19
diff -u -r1.19 Makefile.powerpc
--- Makefile.powerpc    2001/10/26 06:45:39     1.19
+++ Makefile.powerpc    2001/12/01 15:37:11
@@ -72,6 +72,7 @@
 
 ### find out what to use for libkern
 .include "$S/lib/libkern/Makefile.inc"
+dependkernlib: assym.h
 .ifndef PROF
 LIBKERN=       ${KERNLIB}
 .else