Page MenuHomeFreeBSD
Differential D39577

uefi(8) mentions loader(8) and loader.efi but not loader.efi(8)
ClosedPublic
Actions

Authored by grahamperrin on Apr 14 2023, 8:45 PM.
Tags
None
Referenced Files
Unknown Object (File)
Apr 27 2024, 1:47 AM
Unknown Object (File)
Apr 27 2024, 1:47 AM
Unknown Object (File)
Apr 27 2024, 1:47 AM
Unknown Object (File)
Apr 27 2024, 1:47 AM
Unknown Object (File)
Apr 27 2024, 12:11 AM
Unknown Object (File)
Apr 21 2024, 9:04 AM
Unknown Object (File)
Feb 24 2024, 6:10 AM
Unknown Object (File)
Feb 19 2024, 3:53 AM
View All 33 Files
Subscribers
fernape
imp
mhorne

Details

Reviewers
fernape
mhorne
Group Reviewers
docs
Src Committers
Commits
rG481d727f3ede: uefi(8): Refer to a related manual page
rG9c6710cd2453: uefi(8): Refer to a related manual page
rGd01bbdf4572b: uefi(8): Refer to a related manual page
rGf49d6f583e9d: uefi(8): Refer to a related manual page
Summary

Fix https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=265834.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

grahamperrin created this revision.Apr 14 2023, 8:45 PM
Herald added a subscriber: imp. · View Herald TranscriptApr 14 2023, 8:45 PM
grahamperrin requested review of this revision.Apr 14 2023, 8:45 PM
grahamperrin planned changes to this revision.Apr 14 2023, 8:48 PM

From https://man.freebsd.org/cgi/man.cgi?query=uefi&sektion=8&manpath=FreeBSD+14.0-CURRENT#DESCRIPTION:

  1. loader.efi loads and boots the kernel, as described in loader(8).

Should that be loader.efi(8) instead of loader(8)?

fernape requested changes to this revision.Apr 15 2023, 3:30 PM
fernape added a subscriber: fernape.

LGTM for the .Xr part.

For the other part, I think yes, it should be loader.efi(8) instead of loader(8)

share/man/man8/uefi.8
26

Why is the .Dd two months old?

grahamperrin marked an inline comment as done.Apr 15 2023, 5:37 PM
grahamperrin added inline comments.
share/man/man8/uefi.8
26

The patch was provided by tom@ in February; https://bugs.freebsd.org/265834#c3.

https://bugs.freebsd.org/265834#c6 … I'll make a separate commit. … (for Tom to be shown as the author).

grahamperrin updated this revision to Diff 120363.Apr 15 2023, 5:40 PM
grahamperrin marked an inline comment as done.

uefi(8): refer to loader.efi(8) instead of loader(8)

Also, update the date.

grahamperrin added 1 blocking reviewer(s): Src Committers.Apr 15 2023, 5:41 PM
grahamperrin updated this revision to Diff 123687.Jun 23 2023, 5:13 PM

The 2023-06-13 patch from Tom Hukins.

fernape accepted this revision.Jun 24 2023, 3:54 PM

LGTM, thanks!

mhorne accepted this revision.Sep 4 2023, 6:38 PM
mhorne added a subscriber: mhorne.

Bump .Dd and commit?

Someone opened a duplicate of this on GitHub: https://github.com/freebsd/freebsd-src/pull/832

This revision was not accepted when it landed; it landed in state Needs Review.Sep 4 2023, 7:27 PM
Closed by commit rGf49d6f583e9d: uefi(8): Refer to a related manual page (authored by Tom Hukins <tom@FreeBSD.org>, committed by mhorne). · Explain Why
This revision was automatically updated to reflect the committed changes.
mhorne added a commit: rGf49d6f583e9d: uefi(8): Refer to a related manual page.
mhorne added a commit: rGd01bbdf4572b: uefi(8): Refer to a related manual page.Sep 11 2023, 9:43 PM
mhorne added a commit: rG9c6710cd2453: uefi(8): Refer to a related manual page.
mhorne added a commit: rG481d727f3ede: uefi(8): Refer to a related manual page.Sep 11 2023, 10:37 PM

Revision Contents
Changeset List

PathSize
share/
man/
man8/
5 lines

Diff 126859

View Options

share/man/man8/uefi.8

Loading...