Page MenuHomeFreeBSD

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

Authored by ziaee on Thu, Jun 5, 11:20 PM.
Tags
None
Referenced Files
F121964335: D50708.id157631.diff
Tue, Jul 1, 1:09 AM
F121941041: D50708.id156853.diff
Mon, Jun 30, 8:48 PM
Unknown Object (File)
Sun, Jun 29, 9:53 AM
Unknown Object (File)
Sun, Jun 29, 12:28 AM
Unknown Object (File)
Sun, Jun 29, 12:25 AM
Unknown Object (File)
Sat, Jun 28, 6:17 PM
Unknown Object (File)
Sat, Jun 28, 10:01 AM
Unknown Object (File)
Sat, Jun 28, 5:18 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 Not Applicable
Unit
Tests Not Applicable

Event Timeline

ziaee requested review of this revision.Thu, Jun 5, 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.Thu, Jun 12, 12:21 AM