Date: Tue, 3 Mar 2026 10:16:59 -0800 From: "Enji Cooper (yaneurabeya)" <yaneurabeya@gmail.com> To: Warner Losh <imp@bsdimp.com> Cc: Brooks Davis <brooks@freebsd.org>, Enji Cooper <ngie@freebsd.org>, 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: 912864912b71 - main - Run `make obj` before running `make test-includes` Message-ID: <E18DC595-DF37-4981-B73A-467EC9AEE136@gmail.com> In-Reply-To: <CANCZdfrfKgMm7N9nE-DNkWzr9=vqmaKzd%2BRrvYeg_7OQ=aSL4Q@mail.gmail.com> References: <69a678f3.3f811.148b0d7d@gitrepo.freebsd.org> <CANCZdfpdrkwj8RZtKubzg4feo5i61ivfwRSgwZM7U0k_9V%2B=pQ@mail.gmail.com> <aabRjaEUESOKOFZj@spindle.one-eyed-alien.net> <CANCZdfrfKgMm7N9nE-DNkWzr9=vqmaKzd%2BRrvYeg_7OQ=aSL4Q@mail.gmail.com>
index | next in thread | previous in thread | raw e-mail
[-- Attachment #1 --] > On Mar 3, 2026, at 4:22 AM, Warner Losh <imp@bsdimp.com> wrote: > > On Tue, Mar 3, 2026 at 5:18 AM Brooks Davis <brooks@freebsd.org> wrote: > On Tue, Mar 03, 2026 at 04:25:48AM -0700, Warner Losh wrote: > > On Mon, Mar 2, 2026, 11:00???PM Enji Cooper <ngie@freebsd.org> wrote: > > > > > The branch main has been updated by ngie: > > > > > > URL: > > > https://cgit.FreeBSD.org/src/commit/?id=912864912b71951f9a636190b1dba80528f588eb > > > > > > commit 912864912b71951f9a636190b1dba80528f588eb > > > Author: Enji Cooper <ngie@FreeBSD.org> > > > AuthorDate: 2026-03-03 05:59:50 +0000 > > > Commit: Enji Cooper <ngie@FreeBSD.org> > > > CommitDate: 2026-03-03 05:59:56 +0000 > > > > > > Run `make obj` before running `make test-includes` > > > > > > Before this change, `make test-includes` (run as part of buildworld) > > > would place test files in the current directory, which would clutter up > > > git clones. Run `make obj` beforehand to ensure that the files are put > > > in `${.OBJDIR}` instead of `${.CURDIR}`. This helps cut down on the > > > noise significantly when running commands like `git status`. > > > > > > > I've never seen this happen. .OBJDIR is created automatically for me always > > for the last 5 years... I run buildworld all the time. Can you explain when > > / how it happens more specifically or back out the change? > > The commit message is unintentionally misleading and incomplete. I hit > the problem a week or so ago when fixing gcc12 builds and lacked the > time to follow up. test-includes is fine as part of buildworld, but > when run directly from src does fill tools/build/test-includes with .c > files if you blow away the objdir (which seems to the the only reliably > way to cause test-includes to retest some more complex cases.) > > Ah! That should have been in the commit message as the edge case > that provoked this. That's a decent explanation, though there's other > bits of buildworld that will also cause problems if you do them in isolation > and not part of buildworld. This one, though, is important enough to make > an exception for, I think. It's one that's most likely to be used in isolation. Hi Warner, I consistently run into this behavior when doing mfsbsd builds on my 14.3-RELEASE host using this src.conf: https://reviews.freebsd.org/P702 . MK_AUTO_OBJ=yes as shown here: https://reviews.freebsd.org/P703 . I always run this target as part of buildworld on my 14.3-RELEASE host, so whatever is working on your end is likely an artifact of the build host environment, whereas what’s not working for me like you expect is an artifact of my build host environment. This: > Before this change, `make test-includes` (run as part of buildworld) > would place test files in the current directory, which would clutter up > git clones. Could more accurately be expressed like this: > Before this change, `make test-includes` (run as part of buildworld) > would place test files in the current directory on my 14.3-RELEASE > build host, which would clutter up git clones. The exact reason why there’s a difference eludes me right now. -Enji PS I was waiting for a response to the Differential for over a week; I landed the change because I hadn’t received feedback on the review in that period of time and because it gets really old quickly having to always do `git status -uno` in my clones to make sure I’m not accidentally committing things that shouldn’t be committed. This is especially annoying when doing rebases with my “Mac” dev branch because it has a lot of overlapping sections of code in asmc(4) and thunderbolt(4). [-- Attachment #2 --] -----BEGIN PGP SIGNATURE----- iQIzBAEBCAAdFiEEkHfexGRJ3gYRdA2gGpE5DjPsNJgFAmmnJZsACgkQGpE5DjPs NJirEQ//YJuRxzhrEiJmZE6Oa1OnC+z9d9MKSGL2wxUrHYyKVh6vMTstrHaEG+uS 1R9zrZPPnKK4GazKcKcfA+86iATUwdFl9elQ0TmUPCjvqqdtyN5jYpkwL0h84hJ1 slgo9di4vgYZC7qQdTSWNXZt+wwpYNM+QE9GjTBuyFfgEpFryKMfeNyDLUE7Amgr n9LRbRucZv/kDPA3WrHHpGLyl8oiwgLO7bMyw9ch8ZGJQCQn967iA4Vrj2hjST8+ H2USEt785lOK7I8lQcTCOjvv+oqNUaldf8mtP7lfdWzay21vNajYlrPmx2HFsRwt S3ShoPiOTJmmBxNfcZRVAGiYPfJIli044CM7k+75l/Va9eZBcHa4/wD3EnNHrIDg md0gLbkt/wavf8rNXYJX1cheELYlRODpvrHfpXY+o7B4suGIuAAkRTNKqTKIyAPw WhfjaEcQwmDTKHollNfSpQVf8aQ48b+DQSpnl/HRkAE/T5Koz0Qy0nwxsqEIyf4q +IkqiqrOFGsIvbnSCHdLF+yMudGLYuNtvJIjbR0ZcHR1XPAWprdP9LQQ8JTs/TYz sQvziVHjw0yPwZGbwZtdQaDj4fP5E/yK+q6KdxVD8yayy6Lzj+iL9Zh1EQM2+wXt foetu1s2wPkDWJoBhAUsG5tCSzwrXTr7QbFeGa929KKQAxUmLN0= =7t/q -----END PGP SIGNATURE-----home | help
Want to link to this message? Use this
URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?E18DC595-DF37-4981-B73A-467EC9AEE136>