Date: Tue, 03 Jul 2018 13:08:53 -0700 From: Cy Schubert <Cy.Schubert@cschubert.com> To: Bryan Drewery <bdrewery@FreeBSD.org> Cc: Cy Schubert <Cy.Schubert@cschubert.com>, src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r335906 - head/sys/conf Message-ID: <201807032008.w63K8rMr069334@slippy.cwsent.com> In-Reply-To: Message from Bryan Drewery <bdrewery@FreeBSD.org> of "Tue, 03 Jul 2018 13:07:04 -0700." <6d02404f-a7bb-01c7-3775-7a0349e306be@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
In message <6d02404f-a7bb-01c7-3775-7a0349e306be@FreeBSD.org>, Bryan Drewery wr ites: > This is an OpenPGP/MIME signed message (RFC 4880 and 3156) > --nrEq6SIfsbZGYTgfSZ5R3mSN64FvoLCIQ > Content-Type: multipart/mixed; boundary="N66uQifSC1aSf1iMifRJdrqNP6RkoOJg1"; > protected-headers="v1" > From: Bryan Drewery <bdrewery@FreeBSD.org> > To: Cy Schubert <Cy.Schubert@cschubert.com> > Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, > svn-src-head@freebsd.org > Message-ID: <6d02404f-a7bb-01c7-3775-7a0349e306be@FreeBSD.org> > Subject: Re: svn commit: r335906 - head/sys/conf > References: <201807031959.w63JxPA2069044@slippy.cwsent.com> > In-Reply-To: <201807031959.w63JxPA2069044@slippy.cwsent.com> > > --N66uQifSC1aSf1iMifRJdrqNP6RkoOJg1 > Content-Type: text/plain; charset=utf-8 > Content-Language: en-US > Content-Transfer-Encoding: quoted-printable > > On 7/3/2018 12:59 PM, Cy Schubert wrote: > > In message <201807031924.w63JOi5W051614@repo.freebsd.org>, Bryan=20 > > Drewery writes > > : > >> Author: bdrewery > >> Date: Tue Jul 3 19:24:44 2018 > >> New Revision: 335906 > >> URL: https://svnweb.freebsd.org/changeset/base/335906 > >> > >> Log: > >> No need to make a copy to build genoffset_test.o > >> > >> Modified: > >> head/sys/conf/kern.post.mk > >> > >> Modified: head/sys/conf/kern.post.mk > >> =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/sys/conf/kern.post.mk Tue Jul 3 19:09:46 2018 (r33590 > >> 5) > >> +++ head/sys/conf/kern.post.mk Tue Jul 3 19:24:44 2018 (r33590 > >> 6) > >> @@ -191,14 +191,12 @@ offset.inc: $S/kern/genoffset.sh genoffset.o > >> genoffset.o: $S/kern/genoffset.c > >> ${CC} -c ${CFLAGS:N-flto:N-fno-common} $S/kern/genoffset.c > >> =20 > >> -genoffset_test.c: $S/kern/genoffset.c > >> - cp $S/kern/genoffset.c genoffset_test.c > >> - > >> # genoffset_test.o is not actually used for anything - the point of c= > ompilin > >> g it > >> # is to exercise the CTASSERT that checks that the offsets in the off= > set.inc > >> # _lite struct(s) match those in the original(s).=20 > >> -genoffset_test.o: genoffset_test.c offset.inc > >> - ${CC} -c ${CFLAGS:N-flto:N-fno-common} -DOFFSET_TEST genoffset_test.= > c > >> +genoffset_test.o: $S/kern/genoffset.c offset.inc > >> + ${CC} -c ${CFLAGS:N-flto:N-fno-common} -DOFFSET_TEST ${.ALLSRC:M*.c}= > \ > >> + -o ${.TARGET} > >=20 > > -o with -c in the previous line will cause an error. > > -o and -c are fine together. The problem is .ALLSRC duplicating sources > from the dependencies read from .depend.genoffset_test.o file. > Workaround in r335908. gotcha. Sorry for the second spam. -- Cheers, Cy Schubert <Cy.Schubert@cschubert.com> FreeBSD UNIX: <cy@FreeBSD.org> Web: http://www.FreeBSD.org The need of the many outweighs the greed of the few.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201807032008.w63K8rMr069334>