Page MenuHomeFreeBSD
Differential D47476

nanobsd: Fix parallel make
ClosedPublic
Actions

Authored by jlduran on Nov 7 2024, 6:04 PM.
Tags
None
Referenced Files
F131564701: D47476.id146143.diff
Thu, Oct 9, 7:43 AM
F131564302: D47476.id146577.diff
Thu, Oct 9, 7:36 AM
F131555501: D47476.diff
Thu, Oct 9, 5:39 AM
Unknown Object (File)
Sun, Sep 28, 2:03 PM
Unknown Object (File)
Thu, Sep 25, 2:39 AM
Unknown Object (File)
Fri, Sep 19, 5:18 AM
Unknown Object (File)
Thu, Sep 18, 9:23 PM
Unknown Object (File)
Thu, Sep 18, 7:34 PM
View All 55 Files
Subscribers
None

Details

Reviewers
imp
emaste
Commits
rG16fc83629869: nanobsd: Fix parallel make
rG67dd41cdb10d: nanobsd: Fix parallel make
rG3c5d19a40de7: nanobsd: Fix parallel make
Summary

The NANO_NCPU (number of CPUs) value gets ignored by the build script
when setting the NANO_PMAKE (parallel make) value.

Fix it by setting the NANO_PMAKE later in the process.

PR: 277269
MFC after: 1 month

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 60437
Build 57321: arc lint + arc unit

Event Timeline

jlduran requested review of this revision.Nov 7 2024, 6:04 PM
jlduran created this revision.
Harbormaster completed remote builds in B60437: Diff 146143.Nov 7 2024, 6:04 PM
jlduran added a parent revision: D47475: nanobsd: Add missing options to usage().Nov 7 2024, 6:04 PM
jlduran added a comment.Nov 7 2024, 6:24 PM

I'd be tempted to remove NANO_PMAKE from embedded/common, rescue/common, and dhcpd/common.

imp accepted this revision.Nov 7 2024, 6:39 PM
This revision is now accepted and ready to land.Nov 7 2024, 6:39 PM
emaste accepted this revision.Nov 7 2024, 6:48 PM
jlduran updated this revision to Diff 146175.Nov 8 2024, 4:05 PM
  • Move the check for -DNO_CLEAN (-n), so this value is reflected when exporting the variable
This revision now requires review to proceed.Nov 8 2024, 4:05 PM
emaste accepted this revision.Nov 8 2024, 5:45 PM
This revision is now accepted and ready to land.Nov 8 2024, 5:45 PM
imp accepted this revision.Nov 8 2024, 8:21 PM
Closed by commit rG3c5d19a40de7: nanobsd: Fix parallel make (authored by jlduran). · Explain WhyNov 16 2024, 7:24 AM
This revision was automatically updated to reflect the committed changes.
jlduran added a commit: rG3c5d19a40de7: nanobsd: Fix parallel make.
jlduran added a commit: rG67dd41cdb10d: nanobsd: Fix parallel make.Dec 16 2024, 3:14 PM
jlduran added a commit: rG16fc83629869: nanobsd: Fix parallel make.

Revision Contents
Changeset List

PathSize
tools/
tools/
nanobsd/
2 lines

Diff 146143

View Options

tools/tools/nanobsd/defaults.sh

Loading...