Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 10 Oct 2019 23:07:52 -0700
From:      Conrad Meyer <cem@freebsd.org>
To:        Brooks Davis <brooks@freebsd.org>
Cc:        src-committers@freebsd.org, svn-src-all@freebsd.org,  svn-src-head@freebsd.org
Subject:   Re: svn commit: r353408 - head
Message-ID:  <CAG6CVpUdL297XdaW96mC7Jcgo=wk2JpZB9k=9-hFDQkEMR8T2A@mail.gmail.com>
In-Reply-To: <201910101629.x9AGTDkJ024957@repo.freebsd.org>
References:  <201910101629.x9AGTDkJ024957@repo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
Fwiw, this doesn=E2=80=99t seem to fix the NO_CLEAN build for amd64-xtoolch=
ain-gcc.

On Thu, Oct 10, 2019 at 09:29 Brooks Davis <brooks@freebsd.org> wrote:

> Author: brooks
> Date: Thu Oct 10 16:29:13 2019
> New Revision: 353408
> URL: https://svnweb.freebsd.org/changeset/base/353408
>
> Log:
>   Fix -DNO_CLEAN build across r353340 and r353381
>
>   opensolaris_atomic.S is now only used on i386 with opensolaris_atomic.c
>   used on other platforms.  After r353381 it doesn't exist on those
>   platforms so the stale dependency would result in a build error.
>
> Modified:
>   head/Makefile.inc1
>
> Modified: head/Makefile.inc1
>
> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D
> --- head/Makefile.inc1  Thu Oct 10 16:04:43 2019        (r353407)
> +++ head/Makefile.inc1  Thu Oct 10 16:29:13 2019        (r353408)
> @@ -949,7 +949,18 @@ _sanity_check: .PHONY .MAKE
>  # replacing generated files.  Handle these cases here in an ad-hoc
> fashion.
>  _cleanobj_fast_depend_hack: .PHONY
>  # Syscall stubs rewritten in C and obsolete MD assembly implementations
> -# Date      SVN Rev  Syscalls
> +# Date      SVN Rev  Syscalls/Changes
> +# 20191009  r353340  removal of opensolaris_atomic.S (also r353381)
> +.if ${MACHINE} !=3D i386
> +.for f in opensolaris_atomic
> +       @if [ -e "${OBJTOP}/cddl/lib/libzpool/.depend.${f}.o" ] && \
> +           egrep -qw 'opensolaris_atomic\.S'
> ${OBJTOP}/cddl/lib/libzpool/.depend.${f}.o; then \
> +               echo "Removing stale dependencies for opensolaris_atomic"=
;
> \
> +               rm -f ${OBJTOP}/cddl/lib/libzpool/.depend.${f}.* \
> +
> ${LIBCOMPAT:D${LIBCOMPAT_OBJTOP}/cddl/lib/libzpool/.depend.${f}.*}; \
> +       fi
> +.endfor
> +.endif
>  # 20190925  r352689  removal of obsolete i386 memchr.S
>  .for f in memchr
>         @if [ -e "${OBJTOP}/lib/libc/.depend.${f}.o" ] && \
>



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAG6CVpUdL297XdaW96mC7Jcgo=wk2JpZB9k=9-hFDQkEMR8T2A>