Page MenuHomeFreeBSD
Differential D34592

loader.efi: Improve the manual page
ClosedPublic
Actions

Authored by 0mp on Mar 17 2022, 8:55 PM.
Tags
None
Referenced Files
F102809417: D34592.diff
Sun, Nov 17, 11:38 AM
Unknown Object (File)
Sun, Oct 20, 2:56 AM
Unknown Object (File)
Oct 17 2024, 11:51 AM
Unknown Object (File)
Oct 5 2024, 4:02 PM
Unknown Object (File)
Sep 24 2024, 4:26 AM
Unknown Object (File)
Sep 24 2024, 4:26 AM
Unknown Object (File)
Sep 22 2024, 11:41 PM
Unknown Object (File)
Sep 10 2024, 12:33 AM
View All 51 Files
Subscribers
dab
jlduran
pauamma_gundo.com

Details

Reviewers
imp
kevans
tsoome
manu
Group Reviewers
manpages
Commits
rGe92ad24da5a7: loader.efi: Improve the manual page
rG1df2b7c9549e: loader.efi: Improve the manual page
rG9ecf6e0f9f25: loader.efi: Improve the manual page
Summary
  • Add SPDX tag
  • Add a files section describing different locations related to the loader
  • Add an example explaining how to install a new loader.efi to ESP.
  • Reference uefi(8)

MFC after: 2 weeks

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 44812
Build 41700: arc lint + arc unit

Event Timeline

0mp created this revision.Mar 17 2022, 8:55 PM
Herald added a reviewer: manu. · View Herald TranscriptMar 17 2022, 8:55 PM
Herald added a reviewer: manpages. · View Herald Transcript
Herald added a subscriber: dab. · View Herald Transcript
0mp requested review of this revision.Mar 17 2022, 8:55 PM
Harbormaster completed remote builds in B44812: Diff 103961.Mar 17 2022, 8:55 PM
tsoome accepted this revision.Mar 17 2022, 9:21 PM

Seems good to me, thanks.

This revision is now accepted and ready to land.Mar 17 2022, 9:21 PM
pauamma_gundo.com added a subscriber: pauamma_gundo.com.Mar 18 2022, 12:09 AM

Two semi-taste nits.

stand/man/loader.efi.8
76
91
jlduran added a subscriber: jlduran.Mar 18 2022, 12:17 AM

This was much needed, thank you!

I don't know if its also worth mentioning a "scripted tool" for this procedure in src/tools/boot/install-boot.sh?

My only concern is when another operating systems has already installed its loader onto this default location.

Closed by commit rG9ecf6e0f9f25: loader.efi: Improve the manual page (authored by 0mp). · Explain WhyMar 18 2022, 8:56 AM
This revision was automatically updated to reflect the committed changes.
0mp marked 2 inline comments as done.
0mp added a commit: rG9ecf6e0f9f25: loader.efi: Improve the manual page.
0mp added a comment.Mar 18 2022, 11:13 AM
In D34592#783859, @jlduran_gmail.com wrote:

This was much needed, thank you!

I don't know if its also worth mentioning a "scripted tool" for this procedure in src/tools/boot/install-boot.sh?

In the source code it says that it's rather experimental. I'd not reference it.

Also, we would like to have a manual page for it like beinstall.sh(8) so that we can reference it.

My only concern is when another operating systems has already installed its loader onto this default location.

This is something to ask about on hackers@.

stand/man/loader.efi.8
91

Thanks!

0mp added a commit: rG1df2b7c9549e: loader.efi: Improve the manual page.Apr 4 2022, 5:51 PM
0mp added a commit: rGe92ad24da5a7: loader.efi: Improve the manual page.

Revision Contents
Changeset List

PathSize
stand/
man/
73 lines

Diff 103961

View Options

stand/man/loader.efi.8

Loading...