Page MenuHomeFreeBSD
Differential D58932

nanobsd: Add comments to defaults.sh, legacy.sh
Needs ReviewPublic
Actions

Authored by senguptaangshuman17_gmail.com on Aug 18 2026, 10:38 PM.
Tags
None
Referenced Files
F174184426: D58932.diff
Thu, Oct 1, 5:22 AM
Unknown Object (File)
Wed, Sep 30, 5:08 AM
Unknown Object (File)
Mon, Sep 28, 7:58 PM
Unknown Object (File)
Mon, Sep 28, 5:35 PM
Unknown Object (File)
Sun, Sep 27, 4:42 PM
Unknown Object (File)
Sun, Sep 27, 4:09 PM
Unknown Object (File)
Sun, Sep 27, 3:58 AM
Unknown Object (File)
Sun, Sep 27, 3:57 AM
View All Files
Subscribers
imp

Details

Reviewers
imp
jrm
jlduran
ashish
Summary

Add comments to defaults.sh and legacy.sh function definitions.

Diff Detail

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

Event Timeline

senguptaangshuman17_gmail.com created this revision.Aug 18 2026, 10:38 PM
Herald added a subscriber: imp. · View Herald TranscriptAug 18 2026, 10:38 PM
senguptaangshuman17_gmail.com requested review of this revision.Aug 18 2026, 10:38 PM
Harbormaster completed remote builds in B75689: Diff 184285.Aug 18 2026, 10:38 PM
senguptaangshuman17_gmail.com added a parent revision: D58931: nanobsd: Normalize style in common functions.Aug 18 2026, 10:38 PM
senguptaangshuman17_gmail.com added a child revision: D58933: nanobsd: Add an err() convenience function.Aug 18 2026, 10:41 PM
imp accepted this revision.Aug 19 2026, 5:06 AM
This revision is now accepted and ready to land.Aug 19 2026, 5:06 AM
senguptaangshuman17_gmail.com added reviewers: jrm, jlduran, ashish.Aug 19 2026, 5:18 AM
senguptaangshuman17_gmail.com updated this revision to Diff 184447.Aug 20 2026, 10:09 AM
This revision now requires review to proceed.Aug 20 2026, 10:09 AM
Harbormaster completed remote builds in B75781: Diff 184447.Aug 20 2026, 10:09 AM
senguptaangshuman17_gmail.com added a comment.EditedAug 20 2026, 10:16 AM

There are no changes done in this commit, just because of the previous commit update, I thought all the commits needed to be updated as things have been rebased. So pushed an update to all thus removing this differential's accepted status.

So @imp please accept this differential again.

senguptaangshuman17_gmail.com updated this revision to Diff 184815.Aug 23 2026, 11:00 PM
Harbormaster completed remote builds in B76001: Diff 184815.Aug 23 2026, 11:00 PM
senguptaangshuman17_gmail.com added a comment.Aug 23 2026, 11:05 PM

No code changes in this commit, same as before. An earlier commit in the stack (D58931) needed a commit-message reword per review feedback, which required a bottom-up arc diff refresh of everything above it, and that's what reset the accepted status again.

@imp, could you re-accept when you get a chance? Sorry for the repeat ping.

senguptaangshuman17_gmail.com updated this revision to Diff 184985.Aug 25 2026, 6:13 AM
Harbormaster completed remote builds in B76108: Diff 184985.Aug 25 2026, 6:13 AM
senguptaangshuman17_gmail.com updated this revision to Diff 185242.Aug 27 2026, 9:31 PM
senguptaangshuman17_gmail.com edited the summary of this revision. (Show Details)

Fix typo in Avila reference and normalize doc comments

Harbormaster completed remote builds in B76232: Diff 185242.Aug 27 2026, 9:32 PM

Revision Contents
Changeset List

PathSize
tools/
tools/
nanobsd/
128 lines
17 lines

Diff 184447

View Options

tools/tools/nanobsd/defaults.sh

Loading...
View Options

tools/tools/nanobsd/legacy.sh

Loading...