Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 13 Oct 2011 22:14:43 +0900 (JST)
From:      TAKAHASHI Yoshihiro <nyan@FreeBSD.org>
To:        des@freebsd.org
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r226329 - head/usr.bin/kdump
Message-ID:  <20111013.221443.343708041391999047.nyan@FreeBSD.org>
In-Reply-To: <201110122145.p9CLjCcm085538@svn.freebsd.org>
References:  <201110122145.p9CLjCcm085538@svn.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
In article <201110122145.p9CLjCcm085538@svn.freebsd.org>
Dag-Erling Smorgrav <des@freebsd.org> writes:

> Author: des
> Date: Wed Oct 12 21:45:12 2011
> New Revision: 226329
> URL: http://svn.freebsd.org/changeset/base/226329
> 
> Log:
>   Make kdump compile cleanly at WARNS level 6, with one exception: the
>   ipfilter headers contain a duplicated function declaration.  Turn off
>   -Werror to allow kdump to compile in spite of this.
>   
>   It would be neat to be able to turn off -Werror on a file-by-file basis...

> Modified: head/usr.bin/kdump/mkioctls
> ==============================================================================
> --- head/usr.bin/kdump/mkioctls	Wed Oct 12 21:38:48 2011	(r226328)
> +++ head/usr.bin/kdump/mkioctls	Wed Oct 12 21:45:12 2011	(r226329)
> @@ -15,12 +15,21 @@ LC_ALL=C; export LC_ALL
>  # XXX should we use an ANSI cpp?
>  ioctl_includes=`
>  	cd $1
> -	find -H -s * -name '*.h' |
> +	find -H -s * -name '*.h' | grep -v '.*disk.*\.h' | \
>  		xargs egrep -l \
>  '^#[ 	]*define[ 	]+[A-Za-z_][A-Za-z0-9_]*[ 	]+_IO[^a-z0-9_]' |
>  		awk '{printf("#include <%s>\\\\n", $1)}'
>  `
>  
> +case "`uname -m`" in
> +*pc98*)
> +	ioctl_includes="$ioctl_includes#include <sys/diskpc98.h>\\n"
> +	;;
> +*)
> +	ioctl_includes="$ioctl_includes#include <sys/diskmbr.h>\\n"
> +	;;
> +esac
> +
>  awk -v x="$ioctl_includes" 'BEGIN {print x}' |
>  	gcc -E -I$1 -dM -DCOMPAT_43TTY - |
>  	awk -v ioctl_includes="$ioctl_includes" '
> 

This change breaks cross building.

---
TAKAHASHI Yoshihiro <nyan@FreeBSD.org>



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20111013.221443.343708041391999047.nyan>