hwpmc(4): instead of manual edition, use sysrc(8)

[grahamperrin]

Exemplifying safe use of sysrc(8) has become commonplace in documentation such as the FreeBSD Handbook.

[igalic]

👍

[bsdimp]

Looks good to me...
But do you really need the "" in this case? they aren't passed through to the sysrc command and module names never have spaces or globbing characters

[igalic]

Looks good to me... But do you really need the "" in this case? they aren't passed through to the sysrc command and module names never have spaces or globbing characters

good point actually, sysrc correctly quotes by itself

[grahamperrin]

Thank you both.

#785 (comment)

… do you really need the "" in this case? …

Side notes, for a future pull request for the manual page for sysrc(8):

• content of the EXAMPLES section is not consistent with direction given (duplicated) in two preceding sections; and formatting of examples within each of those two is inconsistent both within the section, and when compared with the EXAMPLES section
• some examples are misplaced.

(Now, I realise why I nearly always needed to look at the page two or three times (or look away) to make sense of, and find, the required knowledge.)

Postscript

for a future pull request

Instead:

• 272747 – sysrc(8): order and consistency within the manual page

[grahamperrin]

% git -C /usr/src branch
  main
  pkg-stage
  prefer-ee
  releng/13.1
  releng/13.2
  stable/12
* stable/13
% git -C /usr/src cherry-pick -x 666c54d --edit
Auto-merging share/man/man4/hwpmc.4
CONFLICT (content): Merge conflict in share/man/man4/hwpmc.4
error: could not apply 666c54d12ec9... hwpmc(4): instead of manual edition, use sysrc(8)
hint: After resolving the conflicts, mark them with
hint: "git add/rm <pathspec>", then run
hint: "git cherry-pick --continue".
hint: You can instead skip this commit with "git cherry-pick --skip".
hint: To abort and get back to the state before "git cherry-pick",
hint: run "git cherry-pick --abort".
% 

I dealt with merge conflicts once or twice in the past, but (sorry) recently I just haven't got the head space for anything beyond Git basics. If someone else would like to MFC, please go ahead, thanks.

https://github.com/freebsd/freebsd-src/commits/stable/13/share/man/man4/hwpmc.4

---

Side note:

https://github.com/refined-github/refined-github/blob/main/source/features/list-prs-for-file.tsx

• list-prs-for-file

Alerts you if the current file is altered by an open PR.

[grahamperrin]

First: thanks to Mina.

---

Above, #785 (comment)

… Git basics …

Looking at (tcsh) history across multiple tabs, it's possible that I omitted a (basic) pull:

git -C /usr/src pull --ff-only freebsd stable/13

URLs

% git -C /usr/src config --list | grep url
remote.origin.url=https://github.com/grahamperrin/freebsd-src.git
remote.upstream.url=https://github.com/freebsd/freebsd-src.git
remote.freebsd.url=https://git.freebsd.org/src.git
remote.freebsd.pushurl=git@repo.freebsd.org:src.git
% 

---

I can't imagine why the .Dd date line was treated as conflict in #795, but not as a conflict here in #785 …

Most recent history

  …
  2036  8:50    git -C /usr/src reset --hard HEAD
  2037  8:50    git -C /usr/src pull --ff-only freebsd stable/13
  2038  8:51    git -C /usr/src cherry-pick -x 3541d90 --edit
  2039  8:51    nano /usr/src/lib/geom/part/gpart.8
  2040  8:52    git -C /usr/src status
  2041  8:53    git -C /usr/src add lib/geom/part/gpart.8 && git -C /usr/src commit
  2042  8:57    git -C /usr/src show
  2043  8:57    git -C /usr/src push freebsd stable/13
  2044  9:14    git -C /usr/src cherry-pick -x 666c54d --edit
  2045  9:17    git -C /usr/src show
  2046  9:17    git -C /usr/src push freebsd stable/13