Re: git: 541e6e2d516b - main - bsd.dirs.mk: Fix MK_INSTALL_AS_USER for directories

On Sat, Jul 11, 2026 at 12:54 PM Warner Losh <imp@freebsd.org> wrote:
>
> The branch main has been updated by imp:
>
> URL: https://cgit.FreeBSD.org/src/commit/?id=541e6e2d516b6c9d3681b24464e9ef53c1f2579a
>
> commit 541e6e2d516b6c9d3681b24464e9ef53c1f2579a
> Author:     Warner Losh <imp@FreeBSD.org>
> AuthorDate: 2026-07-11 19:44:49 +0000
> Commit:     Warner Losh <imp@FreeBSD.org>
> CommitDate: 2026-07-11 19:52:17 +0000
>
>     bsd.dirs.mk: Fix MK_INSTALL_AS_USER for directories
>
>     Sponsored by:           Netflix
> ---
>  share/mk/bsd.dirs.mk | 5 +++++
>  1 file changed, 5 insertions(+)
>
> diff --git a/share/mk/bsd.dirs.mk b/share/mk/bsd.dirs.mk
> index 1f81bda80eab..e708ebc9d916 100644
> --- a/share/mk/bsd.dirs.mk
> +++ b/share/mk/bsd.dirs.mk
> @@ -12,8 +12,13 @@ DIRS?=
>  .    if defined(${dir}) && !empty(${dir})
>  # Set default permissions for a directory
>  ${dir}_MODE?=  0755
> +.if ${MK_INSTALL_AS_USER} == "yes"
> +${dir}_OWN?=   ${_uid}
> +${dir}_GRP?=   ${_gid}
> +.else
>  ${dir}_OWN?=   root
>  ${dir}_GRP?=   wheel
> +.endif
>  .      if defined(${dir}_FLAGS) && !empty(${dir}_FLAGS)
>  ${dir}_FLAG=   -f ${${dir}_FLAGS}
>  .      endif
>

I think this might be related to:
https://pkg-status.freebsd.org/beefy24/data/main-amd64-default/p0c6548094723_saf069400a8/logs/errors/unix-selfauth-helper-1.1_1.log

Could it be something wrong with the builder or does the _gid need to
be defined when run as root?