Page MenuHomeFreeBSD

wpa_cli.8 manual page update revision
ClosedPublic

Authored by fnoyanisi_yahoo.com on Dec 2 2016, 7:50 AM.
Tags
None
Referenced Files
F105343646: D8691.id22659.diff
Sun, Dec 15, 3:50 AM
F105340940: D8691.id24105.diff
Sun, Dec 15, 3:05 AM
Unknown Object (File)
Tue, Dec 3, 1:04 PM
Unknown Object (File)
Nov 5 2024, 6:20 PM
Unknown Object (File)
Oct 28 2024, 3:22 AM
Unknown Object (File)
Sep 29 2024, 3:24 AM
Unknown Object (File)
Sep 29 2024, 3:24 AM
Unknown Object (File)
Sep 29 2024, 3:23 AM

Details

Summary

Diff file has been updated

wpa_cli.8.D8691_patched_edited:201
+.It Ic ifname
+Shows the current interface name.
+The default interface is the first interface found in the socket path.

passive -> active: s/Shows/Show/
Done!

wpa_cli.8.D8691_patched_edited:204
+.It Ic ping
+Pings the
+.Xr wpa_supplicant 8

passive -> active: s/Pings/Ping/
Done!

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

fnoyanisi_yahoo.com retitled this revision from to Updated diff file.
fnoyanisi_yahoo.com updated this object.
fnoyanisi_yahoo.com edited the test plan for this revision. (Show Details)
wblock added inline comments.
wpa_cli.8.d8593
155

s/The following/These/

158

"should" is best used for recommending something. Here, it kind of introduces doubt. If control sockets "should" be found there, does that mean sometimes they are not?
I suspect this really means

Control sockets path.
166

Passive to active:

Interface to be configured.
178

When possible, removing the verbs from the beginning of these can make them shorter and sometimes clearer.

PID file location.
188
See the
195

(I know, not yours in this diff.)
s/The following/These/
s/may be/can be/

202

s/Default/The default/

207

s/of/of the/

294

"should be invoked after" meaning "can be", or meaning "must be run after a BSS scan or bad things happen"?

297

s/network/the network/

308

s/given/the given/

fnoyanisi_yahoo.com marked 11 inline comments as done.EditedDec 21 2016, 10:01 AM

Diff file has been updated according to Warren's suggestions.
Thank you!

Hi there,

Some additional context on the motivation for this patch (from bug report 203406 [1])

Current manual page for wpi_cli(8) is out of date -dated 2005- and lacks descriptions of command line options and many commands.

With this patch;

  • SYNPOSIS section has been updated to reflect the command line options
  • OPTIONS section, which includes the description of each command line option (such as -v, -a, -G, -g, -B), has been added
  • Other supported commands have been added into the COMMANDS sections. The wpa_cli_commands[] structure contained in the /contrib/wpa/wpa_supplicant/wpa_cli.c file is used as the reference for adding new commands (commands that need a special compile time option have been excluded)

Patch was generated against the wpa_cli.8 man page included in the "head" branch.

wpa_cli.8.fni -> working copy
wpa_cli.8 -> original file from the "head" branch

Diff file was generated with the command below
diff -u wpa_cli.8 wpa_cli.8.fni > wpa_cli.8.diff

Sevan created the relevant Phabricator entry for the review of the patch in [1]. Diff 23173 [2] reflects corrections proposed by Benjamin and Warren.

[1] https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=203406
[2] https://reviews.freebsd.org/D8691?id=23173

wpa_cli.8.D8691_patched_edited
201 ↗(On Diff #23173)

passive -> active: s/Shows/Show/

204 ↗(On Diff #23173)

passive -> active: s/Pings/Ping/

fnoyanisi_yahoo.com retitled this revision from Updated diff file to wpa_cli.8 manual page update revision.
fnoyanisi_yahoo.com updated this object.
fnoyanisi_yahoo.com marked 2 inline comments as done.

Two updates have been incorporated into the patch;

1 ) passive -> active: s/Shows/Show/ in COMMANDS section of the manual page

wpa_cli.8.D8691_patched_edited:201
+.It Ic ifname
+Shows the current interface name.
+The default interface is the first interface found in the socket path.

2 ) passive -> active: s/Pings/Ping/ in COMMANDS section of the manual page

wpa_cli.8.D8691_patched_edited:204
+.It Ic ping
+Pings the
+.Xr wpa_supplicant 8

wblock added a reviewer: wblock.

Still no context (as in diff context, as in svn diff --diff-cmd=diff -x -U999999 or git diff -U999999 other-branch). But other than the whitespace issues noted, looks good. Please fix those, bump .Dd, and check the patched file with igor -R wpa_cli.8 and mandoc -Tlint wpa_cli.8 before commit.

wpa_cli.8
160 ↗(On Diff #24105)

igor says there is trailing whitespace on this line.

163 ↗(On Diff #24105)

igor says there is trailing whitespace on this line also, although it can't be seen in Phabricator.

This revision is now accepted and ready to land.Jan 20 2017, 3:40 PM
This revision was automatically updated to reflect the committed changes.