Page MenuHomeFreeBSD

MK_FORTH: Allow removing loader.4th w/ delete-old
ClosedPublic

Authored by ziaee on Jun 5 2025, 11:20 PM.
Tags
None
Referenced Files
F137933997: D50708.id156853.diff
Thu, Nov 27, 10:10 AM
Unknown Object (File)
Sun, Nov 23, 11:13 PM
Unknown Object (File)
Fri, Nov 14, 3:21 AM
Unknown Object (File)
Fri, Nov 7, 1:36 PM
Unknown Object (File)
Mon, Nov 3, 4:41 PM
Unknown Object (File)
Mon, Nov 3, 2:27 PM
Unknown Object (File)
Sun, Nov 2, 9:31 AM
Unknown Object (File)
Sun, Nov 2, 1:57 AM
Subscribers

Details

Summary

The lua loader is quite mature, and /boot is getting a bit cluttered.
Allow removing the forth loader with make delete-old if WITHOUT_FORTH
is set, instead of the previous behavior of only removing the doc.

As usual, please be suspicious; I have no idea what I'm doing. I have
been deleting these files for a while, and have tested this as well.
It makes /boot a lot more managable by being viewable with ls in one
window of 80x25. Particularly, I suspect .so is a library so I put that
under make delete-old-libs.

MFC: no, i dont think so since this is a breaking change of behavior

Diff Detail

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

Event Timeline

ziaee requested review of this revision.Jun 5 2025, 11:20 PM
ziaee created this revision.
tools/build/mk/OptionalObsoleteFiles.inc
2005

This one feels wrong

tools/build/mk/OptionalObsoleteFiles.inc
2005

Thanks for looking this over so soon! Why? I figured, if the administrator wants to removing the 4th loader, we should remove the all of it's stuff.

The administrator will still have loader_simp.old and loader_efi.old.

tsoome added inline comments.
tools/build/mk/OptionalObsoleteFiles.inc
2005

might there appear loader_4th.efi.old ?

tools/build/mk/OptionalObsoleteFiles.inc
2005

I do not know. Back before the Lua loader I did not take any interest in the internals.

UPDATING
31–32

Typo repeated in the commit message as well.

tools/build/mk/OptionalObsoleteFiles.inc
2005

We chatted a bit and it's probably fine to remove; it's created as a backup, might as well clean it up if we're nuking the rest of the scripts.

Fix typo, add loader_4th.efi.old. Thanks @kevans and @tsoome!

This revision is now accepted and ready to land.Jun 12 2025, 12:21 AM