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
Unknown Object (File)
Wed, Sep 17, 5:16 AM
Unknown Object (File)
Sat, Sep 13, 8:06 AM
Unknown Object (File)
Sep 9 2025, 8:45 AM
Unknown Object (File)
Sep 5 2025, 5:44 AM
Unknown Object (File)
Sep 2 2025, 5:18 PM
Unknown Object (File)
Aug 26 2025, 4:46 PM
Unknown Object (File)
Aug 25 2025, 9:29 AM
Unknown Object (File)
Aug 21 2025, 9:23 PM
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 Not Applicable
Unit
Tests Not Applicable

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