Re: git: 66f36c368676 - main - packages: Add meta-package sets [ a pkgbase poudriere jail creation got a populated /usr/local/poudriere/jails/main-amd64/usr/src/ ], not just sys/

On Sep 17, 2025, at 09:14, Mark Millard <marklmi@yahoo.com> wrote:

> On Sep 17, 2025, at 08:04, Mark Millard <marklmi@yahoo.com> wrote:
> 
>> On Sep 14, 2025, at 09:04, Lexi Winter <ivy@freebsd.org> wrote:
>> 
>>> Mark Millard:
>>>> Lexi Winter <ivy_at_FreeBSD.org> wrote on
>>>>> packages: Add meta-package sets
>>>> [poudriere installs more packages than before]
>>> 
>>> sorry.  however, this is something poudriere should be able to fix quite
>>> easily by adjusting which packages they install.  for example, they
>>> could install the 'FreeBSD-set-base' set to get the entire base system,
>>> then add FreeBSD-src-* if required (i believe some ports need this).
>>> 
>>> please file a PR against poudriere for that.
>> 
>> Status update:
>> 
>> The split out of FreeBSD-set-src and FreeBSD-set-tests partially fixed
>> the overall issue without changes to poudriere(-devel).
> 
> My note about FreeBSD-set-tests was incomplete relative to tests
> overall, it appears:
> 
> # pkg-static -c /usr/local/poudriere/jails/main-aarch64 info | grep tests-
> pkg-static: Warning: Major OS version upgrade detected.  Running "pkg bootstrap -f" recommended
> FreeBSD-tests-16.snap20250917014314 Test Suite
> FreeBSD-tests-dev-16.snap20250916002513 Test Suite (development files)

Turns out that the split out of FreeBSD-set-tests is not
completely the case (but FreeBSD-set-src was completely
the case): FreeBSD-set-devel requires FreeBSD-tests-dev ,
which, in turn requires FreeBSD-tests . . .

# pkg-static -c /usr/local/poudriere/jails/main-armv7 info -r -g 'FreeBSD-tests*'
pkg-static: Warning: Major OS version upgrade detected.  Running "pkg bootstrap -f" recommended
FreeBSD-tests-16.snap20250917013424:
	FreeBSD-tests-dev-16.snap20250916000217
	FreeBSD-tests-dev-16.snap20250916000217 (libprivateatf-c.so.1)
	FreeBSD-tests-dev-16.snap20250916000217 (libval2.so.0)
FreeBSD-tests-dev-16.snap20250916000217:
	FreeBSD-set-devel-16.snap20250912211618

As stands, poudriere(-devel) is including FreeBSD-set-devel .

Again, this may be judged to be fine overall, even though
there is a FreeBSD-set-tests as well. I'm just noting the
change in poudreire(-devel) jail creation results so that
(more) accidental additions are avoided and that (most)
additions have been explicitly chosen.

I have been adding notes to:

https://github.com/freebsd/poudriere/issues/1247

as well.

>> One thing that is now worse for poudriere(devel) jails is that the
>> bundling of man pages with -dev means that such man pages are installed
>> in all poudriere(-devel) jails during creation. poudreire(-devel)
>> definitely does need -dev but does not need the related man pages.
> 
> I'll note that I'm trying to report differences in what is
> installed in poudiere(-devel) jails that used to not be.
> I'm just trying to avoid unnoticed additions from being
> implicitly accepted.
> 
> I'm not objecting if the overall choice ends up being that
> some of the additions are okay to leave in place. (I
> might be surprised in some cases, but that is not the same
> as trying to require a lack of surprise.)
> 
> As far as the choices go, it appears to me that such is
> between Byran D. Baptiste D., you, and possibly others.
> 
>> poudriere(-devel) does not not yet filter out FreeBSD-set-kernels-* 
>> as the plural form is not matched by poudriere(-devel)'s filtering:
>> 
>> grep -vE -- '-man|-dbg|-kernel-|-tests|-src-'
>> 
>> So, as stands, the individual kernel packages are filtered out but
>> the FreeBSD-set-kernels-* is not filtered out, causing the kernels
>> to end up being installed into the jail.
>> 
>> Note: poudriere(-devel) handles installing FreeBSD-src-sys
>>     separately.
>> 
>>>> NOTE: FreeBSD-kernel-16.snap20250911200332 turns out to be the man pages
>>>>    under: /usr/share/man/man[49]/
>>> 
>>> this is a regression caused by a8a18eeb5e17 which disabled MANSPLITPKG
>>> by default.  this package is meant to be called "FreeBSD-kernel-man" and
>>> there should not be any "FreeBSD-kernel" package at all.  i'll look at a
>>> fix for this.  the contents of the package should be correct, though.
>> 
>> Confirmed for my (non-poudriere jail) main 16 updates: FreeBSD-kernel-man
>> is now in place instead of FreeBSD-kernel .
> 



===
Mark Millard
marklmi at yahoo.com