On Mon, Nov 20, 2017 at 10:30 PM, Mark Millard <mar...@dsl-only.net> wrote:
> It looks to me like head/stand/powerpc/boot1.chrp/Makefile might
> have += vs. = issues compared to the other examples:
>
> Warner Losh imp at FreeBSD.org wrote on
> Mon Nov 20 22:42:21 UTC 2017 :
>
> Modified: head/stand/powerpc/boot1.chrp/Makefile
> ============================================================
> ==================
> --- head/stand/powerpc/boot1.chrp/Makefile Mon Nov 20 22:41:22 2017
> (r326037)
> +++ head/stand/powerpc/boot1.chrp/Makefile Mon Nov 20 22:42:17 2017
> (r326038)
> @@ -13,9 +13,7 @@ SRCS= boot1.c ashldi3.c syncicache.c
>
> MAN=
>
> -CFLAGS= -ffreestanding -msoft-float \
> - -I${LDRSRC} -I${SYSDIR} -I${SASRC} \
> - -D_STANDALONE
> +CFLAGS= -I${LDRSRC} -I${SYSDIR} -I${SASRC} -D_STANDALONE
> LDFLAGS=-nostdlib -static -Wl,-N
>
> .PATH: ${SYSDIR}/libkern ${SRCTOP}/lib/libc/powerpc/gen ${.CURDIR}
>
> but the other files tend to use += for at least CFLAGS. In
> one case you converted an = into only using += locally:
>
> -CFLAGS= -ffreestanding \
> - -I${.CURDIR} \
> +CFLAGS+= -I${.CURDIR} \
>
> (Generally the mix seems to be as it was before
> the changes but it likely looked odd before the
> changes when the files were view together.)
>
> There are also examples like:
>
> LDFLAGS= -nostdlib -static -T ${.CURDIR}/ldscript.${MACHINE_
> CPUARCH}
> LDFLAGS+= -Wl,-znotext
>
> and others with just = . But some files seem to always
> use += for LDFLAGS.
>
> May be a round of = vs. += checking is appropriate
> for the involved files?
>
Yea. I've done it once or twice already. It's quite likely another pass may
be necessary, so it doesn't surprise me you've found at least one places
that needs a good close look... There's a few places LDFLAGS are used, and
it seems everywhere it has -nostdlib, and maybe a few others. And there's
likely a few places where some of the defines need to be moved around to.
Warner
_______________________________________________
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"
