Page MenuHomeFreeBSD

Delete lines documenting removed network configuration variables.
AcceptedPublic

Authored by felix.the.red_gmail.com on Feb 14 2025, 5:46 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Mar 25, 11:36 PM
Unknown Object (File)
Mon, Mar 24, 3:52 AM
Unknown Object (File)
Sun, Mar 23, 2:05 PM
Unknown Object (File)
Wed, Mar 19, 12:03 AM
Unknown Object (File)
Tue, Mar 18, 1:50 AM
Unknown Object (File)
Mon, Mar 17, 2:08 PM
Unknown Object (File)
Sat, Mar 8, 2:37 PM
Unknown Object (File)
Feb 19 2025, 1:03 PM

Details

Reviewers
ziaee
carlavilla
mhorne
Group Reviewers
manpages
Summary

Remove documentation for variables nai, server_nai, and eappsk.
Support for these variables was removed upstream several years ago.

Test Plan

PR 284126

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 62451
Build 59335: arc lint + arc unit

Event Timeline

ziaee added reviewers: carlavilla, mhorne.
ziaee added subscribers: carlavilla, mhorne.

This is great!
The summary field should contain the commit log message, like a report on what's going on for people unfamiliar with the subject:

wpa_supplicant.conf.5: Remove removed variables

2005-09-25 - wpa_supplicant v0.4.5 removed "server_nai"
2008-02-22 - wpa_supplicant v0.6.3 removed "eappsk" and "nai"

PR: 284126
MFC after: 3 days
Reported by: J.R. Oldroyd <fbsd@opal.com>
Differential Revision: https://reviews.freebsd.org/D49010

The first line should be the file that's changing, followed by a brief summary for people searching though the commit logs.
The PR trailer updates the bug thread and provides the cross-reference.
Since this change is non breaking and applies to past versions of FreeBSD, we should use an MFC trailer to declare intent to backport this patch to stable.
The original reporter was J.R. Oldroyd, so we should give them credit.
Finally, the differential revision trailer provides a link to this discussion.

@carlavilla and/or @mhorne , may I commit this (with a bumped date)?

This revision is now accepted and ready to land.Feb 24 2025, 11:41 AM