Page MenuHomeFreeBSD

loader: Fix boot menu on BIOS
ClosedPublic

Authored by imp on Nov 2 2024, 4:33 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 31, 4:18 AM
Unknown Object (File)
Sun, Dec 15, 9:02 AM
Unknown Object (File)
Dec 8 2024, 11:46 PM
Unknown Object (File)
Dec 4 2024, 3:10 AM
Unknown Object (File)
Dec 1 2024, 2:08 PM
Unknown Object (File)
Nov 28 2024, 12:08 PM
Unknown Object (File)
Nov 21 2024, 2:59 PM
Unknown Object (File)
Nov 21 2024, 1:33 PM

Details

Summary

Only the gfx-enabled boot loader supports unicode. Otherwise, we have to
use the old cons25 / ibmpc upper code page drawing characters. Check to
see if we have the gfx.term_drawbox function. If we do, we support the
unicode drawing characters. If we don't, then we have an older loader
that doesn't support it *OR* we have the reduced function, text-only
boot loader. Either way, we need to use the old graphics characters.

Sponsored by: Netflix

Diff Detail

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

Event Timeline

imp requested review of this revision.Nov 2 2024, 4:33 PM

Tested successfully on a laptop: first, running stable/14-n269448-4e8444d5750a, then running main-n273387-bb8b3b174118.

Update to move these knowledge into core.hasUnicode() in case
other issues like this crop up, or in case we export this knowledge
more directly.

better comment
also remove comment from where we create old bindings:
with this new setup they aren't relevant.

emaste added inline comments.
stand/lua/core.lua
548
This revision was not accepted when it landed; it landed in state Needs Review.Nov 2 2024, 10:03 PM
This revision was automatically updated to reflect the committed changes.

Thank you for coming up with the solution. I can confirm that it fixes the issue for sc(4). Please don't hesitate to push it, preferably shortly.
Edit:
Wow.... already committed. Impressive timings!