Page MenuHomeFreeBSD
Differential D54079

build.7: Improve make delete-old example
ClosedPublic
Actions

Authored by ziaee on Dec 4 2025, 9:11 PM.
Tags
None
Referenced Files
F167044582: D54079.id168329.diff
Tue, Aug 18, 6:43 PM
Unknown Object (File)
Mon, Aug 17, 9:23 AM
Unknown Object (File)
Thu, Aug 13, 12:31 AM
Unknown Object (File)
Wed, Aug 12, 7:01 PM
Unknown Object (File)
Wed, Aug 12, 6:19 PM
Unknown Object (File)
Mon, Aug 10, 11:57 AM
Unknown Object (File)
Mon, Aug 10, 8:28 AM
Unknown Object (File)
Mon, Aug 10, 7:50 AM
View All Files
Subscribers
emaste
fuz
imp

Details

Reviewers
jbo
jhb
Group Reviewers
manpages
Commits
rG73a02425e434: build.7: Improve make delete-old example
rG1d69c8e4ac7d: build.7: Improve make delete-old example
Summary

Be more explicit that deleting old files and libraries needs to be
really after you're rebooted a third time and you're really actually
sure your applications aren't using old libs, following irc discussion.

Also adjust an example slightly to fit on 80 character console.

MFC after: 3 days

Diff Detail

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

Event Timeline

ziaee created this revision.Dec 4 2025, 9:11 PM
Herald added a subscriber: imp. · View Herald TranscriptDec 4 2025, 9:11 PM
ziaee requested review of this revision.Dec 4 2025, 9:11 PM
Harbormaster completed remote builds in B69058: Diff 167560.Dec 4 2025, 9:11 PM
fuz added a subscriber: fuz.Dec 18 2025, 3:30 PM
fuz added inline comments.
share/man/man7/build.7
1127

Should be consistent, also check for additional occurrences.

ziaee updated this revision to Diff 168329.Dec 18 2025, 4:19 PM

switch examples consistently to /client, thanks @fuz!

Harbormaster completed remote builds in B69373: Diff 168329.Dec 18 2025, 4:19 PM
jhb added inline comments.Jan 28 2026, 9:51 PM
share/man/man7/build.7
1026–1027

I would be tempted to leave make delete-old here and only defer make delete-old-libs to the new paragraph. In practice I always run make delete-old right after etcupdate -B. It's only make delete-old-libs that I defer until after updating packages.

ziaee added a child revision: D56217: build.7: Explain packaging a custom kernel.Apr 1 2026, 4:00 PM
emaste added a subscriber: emaste.Apr 1 2026, 4:33 PM
emaste added inline comments.
share/man/man7/build.7
1026–1027

Agreed. They're split into two targets for a reason.

ziaee updated this revision to Diff 174976.Apr 6 2026, 4:10 PM

thanks for review guys!

Harbormaster completed remote builds in B71998: Diff 174976.Apr 6 2026, 4:10 PM
ziaee updated this revision to Diff 174977.Apr 6 2026, 4:25 PM

rebase due to addressing feedback in previous revision.

i didn't report this

i got this information from your mail on the pkgbase list, iiuc im supposed to then credit you with reported by.

i think we also need to document the behaviour of building multiple kernels in KERNCONF

agree. i think we should put that in the KERNCONF entry, im going to attempt it in a subsequent revision.

Harbormaster completed remote builds in B71999: Diff 174977.Apr 6 2026, 4:25 PM
ziaee updated this revision to Diff 174978.Apr 6 2026, 4:32 PM

fix rebase error

Harbormaster completed remote builds in B72000: Diff 174978.Apr 6 2026, 4:32 PM
ziaee updated this revision to Diff 174979.Apr 6 2026, 4:37 PM

sorry, last attempt at diff actually just made it worse, i think this is the correct diff

Harbormaster completed remote builds in B72001: Diff 174979.Apr 6 2026, 4:37 PM
ziaee added a comment.Apr 16 2026, 11:37 PM

ping

This revision was not accepted when it landed; it landed in state Needs Review.Apr 22 2026, 5:30 PM
Closed by commit rG1d69c8e4ac7d: build.7: Improve make delete-old example (authored by ziaee). · Explain Why
This revision was automatically updated to reflect the committed changes.
ziaee added a commit: rG1d69c8e4ac7d: build.7: Improve make delete-old example.
ziaee added a commit: rG73a02425e434: build.7: Improve make delete-old example.Apr 25 2026, 5:04 PM

Revision Contents
Changeset List

PathSize
share/
man/
man7/
34 lines

Diff 174978

View Options

share/man/man7/build.7

Loading...