Re: CVS commit: src/sys/conf

[Christoph Egger <Christoph_Egger%gmx.de@localhost>]

On 07/04/11 15:00, matthew green wrote:
> Module Name:    src
> Committed By:   mrg
> Date:           Mon Jul  4 13:00:19 UTC 2011
>
> Modified Files:
>         src/sys/conf: Makefile.kern.inc
>
> Log Message:
> apply some -Wno-foo to 3 files that are non-trivial to solve, for GCC 4.5.
>
>
> To generate a diff of this commit:
> cvs rdiff -u -r1.140 -r1.141 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.140 
> src/sys/conf/Makefile.kern.inc:1.141
> --- src/sys/conf/Makefile.kern.inc:1.140        Mon Jun 20 06:52:38 2011
> +++ src/sys/conf/Makefile.kern.inc      Mon Jul  4 13:00:19 2011
> @@ -1,4 +1,4 @@
> -#      $NetBSD: Makefile.kern.inc,v 1.140 2011/06/20 06:52:38 mrg Exp $
> +#      $NetBSD: Makefile.kern.inc,v 1.141 2011/07/04 13:00:19 mrg Exp $
>   #
>   # This file contains common `MI' targets and definitions and it is included
>   # at the bottom of each `MD' ${MACHINE}/conf/Makefile.${MACHINE}.
> @@ -510,12 +510,19 @@
>       uvm/uvm_pager.c dev/ic/aic7xxx.c dev/ic/aic79xx.c arch/xen/i386/gdt.c \
>       dev/ofw/ofw_subr.c
>
> -. if defined(HAVE_GCC) || defined(HAVE_PCC)
> +.if defined(HAVE_GCC) || defined(HAVE_PCC)

Please revert this part.

Christoph

>   .for __varstack in ${VARSTACK}
>   COPTS.${__varstack:T} += -Wno-stack-protector
>   .endfor
>   .endif
>
> +# XXX ugly ugly to fix
> +.if defined(HAVE_GCC)&&  ${HAVE_GCC} == 45
> +CWARNFLAGS.tulip.c+=   -Wno-array-bounds
> +CWARNFLAGS.radeonfb.c+=        -Wno-cast-qual
> +CWARNFLAGS.dbcool.c+=  -Wno-cast-qual
> +.endif
> +
>   AFLAGS+=      ${AOPTS.${.IMPSRC:T}}
>   CFLAGS+=      ${COPTS.${.IMPSRC:T}} ${CPUFLAGS.${.IMPSRC:T}}
>   CPPFLAGS+=    ${CPPFLAGS.${.IMPSRC:T}}