Correct -P description in boot.config(5)
AcceptedPublic
Actions

Authored by brd on Jul 13 2017, 10:10 PM.

Details

Reviewers

allanjude
bcr
imp

Group Reviewers

manpages

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

No Lint Coverage

Unit

No Test Coverage

Build Status

Buildable 10505
Build 10915: arc lint + arc unit

Event Timeline

brd created this revision.Jul 13 2017, 10:10 PM

Herald added a reviewer: manpages. · View Herald TranscriptJul 13 2017, 10:10 PM

Herald added a subscriber: imp. · View Herald Transcript

Harbormaster completed remote builds in B10482: Diff 30751.Jul 13 2017, 10:10 PM

allanjude accepted this revision.Jul 13 2017, 11:15 PM

This revision is now accepted and ready to land.Jul 13 2017, 11:15 PM

wblock added a subscriber: wblock.Jul 14 2017, 4:01 PM

wblock added inline comments.

head/share/man/man5/boot.config.5
78–80	Passive -> active: s/will activate/activates/

Address feedback

This revision now requires review to proceed.Jul 14 2017, 9:01 PM

Harbormaster completed remote builds in B10505: Diff 30800.Jul 14 2017, 9:01 PM

brd marked an inline comment as done.Jul 14 2017, 9:01 PM

OK from manpages after Warren's feedback was addressed.

This revision is now accepted and ready to land.Aug 5 2017, 9:18 PM

imp added inline comments.Aug 6 2017, 9:51 PM

head/share/man/man5/boot.config.5
80	Also, -P doesn't work in UEFI.
80	Also also, not all systems with BMCs provide keyboards. OCAs (at least the X9 Supermicro ones) this works.

brd added inline comments.Aug 15 2017, 6:21 PM

head/share/man/man5/boot.config.5
80	Why doesn't it work with UEFI?
80	What do they provide instead? Serial? Nothing?

ping

This is fine. -P on UEFI is hit or miss, and I'm not sure why.

Revision Contents
Changeset List

Path

Size

head/

share/

man/

man5/

boot.config.5

7 lines

Diff 30800

View Options

head/share/man/man5/boot.config.5

Correct -P description in boot.config(5)AcceptedPublicActions