In D49757#1147858, @gordon wrote:

I’m not entirely sure what kind of approval from secteam is being sought. If someone in core would like to help me understand what kind of review is expected, I’d be happy to undertake it.

Until then, I don’t want to be the blocker for this otherwise reasonable sounding change.

handle case when var is *not* set and bmake complains about it

as mentioned on my email to re@ this would be handy for the multi-arch
container builds as well as possibly for general reproducible builds.

include bofh recommendations

waiting on doceng internal discussion.

thanks Doug, rebuilding with latest version atm.

LGTM, testing with stable/14 today. thanks Doug for tracking this down & explaining it.

LGTM, had to get past an unrelated panic to test this one :/

added clusteradm or secteam per review of 3rd party dependency.

per discussion in core@ this is OK, but should get an OK from either clusteradm or secteam, due to the 3rd party dependency. Other than that LGTM.

Some context:

{F114065412}

LGTM, maybe you were right and black will be a less jarring colour... needs doc approval for the website.

There are also PRs open for these, phab is simpler for review and a nicer patch mgmt workflow.

incorporate diizzy feedback where appropriate

fix the fixed plist oh the shame

woops plist

• add dbiao fix for daemonic success
• add diizzy polishing

rebase to handle buildstream deprecation

example phab review for mark

remove anubis

BTW I think daemon is broken, https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=285742 I tried on 14.2-RELEASE, 15.0-CURRENT, and a 14-STABLE from today, none of them will shut down as expected.

I needed this too, https://anubis.techaro.lol/

💥

thanks, can't reproduce this issue anymore after this patch

LGTM, just a few quibbles from portlint/portfmt/portclippy, address those and & you're good to go!

jhb already dealt with this

add README and pkg-message

fix CURRENT, thanks igoro