Page MenuHomeFreeBSD

bsd.port.mk: run pkg-create with MAKE_JOB_NUMBER threads
ClosedPublic

Authored by siva on Feb 10 2026, 8:49 PM.
Tags
None
Referenced Files
F149451069: D55232.id173113.diff
Tue, Mar 24, 1:34 PM
Unknown Object (File)
Tue, Mar 17, 1:00 PM
Unknown Object (File)
Tue, Mar 17, 12:58 PM
Unknown Object (File)
Mon, Mar 16, 12:58 PM
Unknown Object (File)
Fri, Mar 13, 11:30 PM
Unknown Object (File)
Thu, Mar 12, 7:56 AM
Unknown Object (File)
Wed, Mar 11, 3:25 PM
Unknown Object (File)
Wed, Mar 11, 5:04 AM
Subscribers

Details

Summary

Following the same as https://reviews.freebsd.org/D53053
in the src tree.

Diff Detail

Repository
R11 FreeBSD ports repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 70593
Build 67476: arc lint + arc unit

Event Timeline

siva requested review of this revision.Feb 10 2026, 8:49 PM
siva created this revision.
This revision is now accepted and ready to land.Wed, Mar 4, 8:49 AM

(Is commenting on a closed review the Right Thing[tm] to do? Or should I be creating a separate one so as not to confuse things?)

I have a concern here, beyond the reproducibility/ratio/etc. comments that are being touched on in replies to the commit message.

Tying this explicitly to MAKE_JOBS_NUMBER means that when MAKE_JOBS_UNSAFE is set, packaging only gets one thread. Usually, one sets that latter variable when the Makefile has parallelism/ordering issues, and so having that also affect how packaging works is probably going to be unexpected behaviour.

Could we maybe introduce a variable for this purpose? I'm (attempting to) attach a patch to this comment (so as not to affect the main diff here) that adds in a PKG_CREATE_THREADS_NUMBER variable and sets it using some of the same logic for MAKE_JOBS_NUMBER.

Plus, given that there are some concerns being brought up on the list, this would be a way for people to continue having build parallelism, but avoid any unexpected side effects of multi-threaded packaging.

edit: Fixing comment that was in the completely wrong place.
{F146697295}