Date: Wed, 31 Jan 2024 13:57:41 -0700 From: Warner Losh <imp@bsdimp.com> To: Ed Maste <emaste@freebsd.org> Cc: src-committers <src-committers@freebsd.org>, "<dev-commits-src-all@freebsd.org>" <dev-commits-src-all@freebsd.org>, "<dev-commits-src-main@freebsd.org>" <dev-commits-src-main@freebsd.org> Subject: Re: git: ed3563b0ac31 - main - csu: add crtbrand.o dependency on sys/param.h Message-ID: <CANCZdfqCWMd_tzRwHR4=kQHmuVZPyvdvEW6ZRWUVqbN4VC%2BY2Q@mail.gmail.com> In-Reply-To: <202401312041.40VKfgE0080600@gitrepo.freebsd.org> References: <202401312041.40VKfgE0080600@gitrepo.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--000000000000e0f95d0610442124 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Wed, Jan 31, 2024, 1:41=E2=80=AFPM Ed Maste <emaste@freebsd.org> wrote: > The branch main has been updated by emaste: > > URL: > https://cgit.FreeBSD.org/src/commit/?id=3Ded3563b0ac31d854bf907d4d847ac01= 95ec9637b > > commit ed3563b0ac31d854bf907d4d847ac0195ec9637b > Author: Ed Maste <emaste@FreeBSD.org> > AuthorDate: 2024-01-12 15:01:49 +0000 > Commit: Ed Maste <emaste@FreeBSD.org> > CommitDate: 2024-01-31 20:40:27 +0000 > > csu: add crtbrand.o dependency on sys/param.h > > __FreeBSD_version is recorded in *crt1.o and crti.o via crtbrand.o. > Add > an explicit dependency to pick up __FreeBSD_version bumps. > > Additional changes are required to fully plumb *crt1.o dependencies > through the build. > So a kernel bump now rebuilds (or at least relinks) everything? Or soon will? Reported by: bapt > Reviewed by: bapt > Sponsored by: The FreeBSD Foundation > Differential Revision: https://reviews.freebsd.org/D43417 > --- > lib/csu/Makefile.inc | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/lib/csu/Makefile.inc b/lib/csu/Makefile.inc > index 55aaf589f48f..1509a1ece2c3 100644 > --- a/lib/csu/Makefile.inc > +++ b/lib/csu/Makefile.inc > @@ -55,6 +55,9 @@ Scrt1_c.o: ${CRT1SRC} > Scrt1.o: Scrt1_c.o ${CRT1OBJS} ${CRT1OBJ} > ${LD} ${_LDFLAGS} -o ${.TARGET} -r ${.ALLSRC:M*.o} > > +# __FreeBSD_version is recorded in crt1.o et al via crtbrand. > +crtbrand.o: ${SRCTOP}/sys/sys/param.h > + > crtbegin.o: crtbegin.c > crtbeginS.o: crtbegin.c > crtbeginT.o: crtbegin.c > --000000000000e0f95d0610442124 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable <div dir=3D"auto"><div><br><br><div class=3D"gmail_quote"><div dir=3D"ltr" = class=3D"gmail_attr">On Wed, Jan 31, 2024, 1:41=E2=80=AFPM Ed Maste <<a = href=3D"mailto:emaste@freebsd.org">emaste@freebsd.org</a>> wrote:<br></d= iv><blockquote class=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;border-left= :1px #ccc solid;padding-left:1ex">The branch main has been updated by emast= e:<br> <br> URL: <a href=3D"https://cgit.FreeBSD.org/src/commit/?id=3Ded3563b0ac31d854b= f907d4d847ac0195ec9637b" rel=3D"noreferrer noreferrer" target=3D"_blank">ht= tps://cgit.FreeBSD.org/src/commit/?id=3Ded3563b0ac31d854bf907d4d847ac0195ec= 9637b</a><br> <br> commit ed3563b0ac31d854bf907d4d847ac0195ec9637b<br> Author:=C2=A0 =C2=A0 =C2=A0Ed Maste <emaste@FreeBSD.org><br> AuthorDate: 2024-01-12 15:01:49 +0000<br> Commit:=C2=A0 =C2=A0 =C2=A0Ed Maste <emaste@FreeBSD.org><br> CommitDate: 2024-01-31 20:40:27 +0000<br> <br> =C2=A0 =C2=A0 csu: add crtbrand.o dependency on sys/param.h<br> <br> =C2=A0 =C2=A0 __FreeBSD_version is recorded in *crt1.o and crti.o via crtbr= and.o.=C2=A0 Add<br> =C2=A0 =C2=A0 an explicit dependency to pick up __FreeBSD_version bumps.<br= > <br> =C2=A0 =C2=A0 Additional changes are required to fully plumb *crt1.o depend= encies<br> =C2=A0 =C2=A0 through the build.<br></blockquote></div></div><div dir=3D"au= to"><br></div><div dir=3D"auto">So a kernel bump now rebuilds (or at least = relinks) everything? Or soon will?=C2=A0</div><div dir=3D"auto"><br></div><= div dir=3D"auto"><div class=3D"gmail_quote"><blockquote class=3D"gmail_quot= e" style=3D"margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> =C2=A0 =C2=A0 Reported by:=C2=A0 =C2=A0 bapt<br> =C2=A0 =C2=A0 Reviewed by:=C2=A0 =C2=A0 bapt<br> =C2=A0 =C2=A0 Sponsored by:=C2=A0 =C2=A0The FreeBSD Foundation<br> =C2=A0 =C2=A0 Differential Revision: <a href=3D"https://reviews.freebsd.org= /D43417" rel=3D"noreferrer noreferrer" target=3D"_blank">https://reviews.fr= eebsd.org/D43417</a><br> ---<br> =C2=A0lib/csu/Makefile.inc | 3 +++<br> =C2=A01 file changed, 3 insertions(+)<br> <br> diff --git a/lib/csu/Makefile.inc b/lib/csu/Makefile.inc<br> index 55aaf589f48f..1509a1ece2c3 100644<br> --- a/lib/csu/Makefile.inc<br> +++ b/lib/csu/Makefile.inc<br> @@ -55,6 +55,9 @@ Scrt1_c.o: ${CRT1SRC}<br> =C2=A0Scrt1.o: Scrt1_c.o ${CRT1OBJS} ${CRT1OBJ}<br> =C2=A0 =C2=A0 =C2=A0 =C2=A0 ${LD} ${_LDFLAGS} -o ${.TARGET} -r ${.ALLSRC:M*= .o}<br> <br> +# __FreeBSD_version is recorded in crt1.o et al via crtbrand.<br> +crtbrand.o: ${SRCTOP}/sys/sys/param.h<br> +<br> =C2=A0crtbegin.o: crtbegin.c<br> =C2=A0crtbeginS.o: crtbegin.c<br> =C2=A0crtbeginT.o: crtbegin.c<br> </blockquote></div></div></div> --000000000000e0f95d0610442124--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfqCWMd_tzRwHR4=kQHmuVZPyvdvEW6ZRWUVqbN4VC%2BY2Q>