Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

freebsd-version.1: corrections and suggested improvements #1319

Closed

Conversation

grahamperrin
Copy link
Contributor

For four, the word 'neither' is inappropriate.

Make clearer that with multiple options, user-specified order is disregarded.

Various other suggestions.

With four options (not two), the word 'neither' is inappropriate.

Move the hardcoding note to IMPLEMENTATION NOTES.
Make clearer that for multiple options, user-specified order is disregarded.
Consistency: describe userland as installed (not currently running).

Omit the path to the freebsd-version binary.

Omit option -u, to help demonstrate that the utility behaves as if this option is specified when no option is specified.
Improve the paragraph about discrepancies with freebsd-update.
Consistency: utility, not command.

Use the word 'utility' less frequently.
Correct my earlier omission.
Where version but not patch level is mentioned, rewrite this short sentence.
.Pp after .Ss (subsection) seems to have no effect.

Also, attempt to fix the option formatting mistake in my previous commit.
Again, attempt to fix my option formatting mistake.
@grahamperrin

This comment was marked as outdated.

@grahamperrin

This comment was marked as outdated.

@concussious
Copy link
Contributor

concussious commented Jul 8, 2024

With four options (not two), the word 'neither' is inappropriate.

No.

"1. Not either; generally used to introduce the first of two or more coördinate clauses of which those that follow begin with nor."
~ The GNU version of the Collaborative International Dictionary of English

Make clearer that with multiple options, user-specified order is disregarded

No. The order is quite clear and written with very strong language. The proposed change is insulting the intended audience. This is not a tutorial or handbook. Further, it reads like our beloved security officer wrote it and you wanna change that, while there's nothing wrong with it... is... shame on you!

a simple, non-contentious pr

".Ss Options" has been the subject of flame wars. Where do you see us doing that? Why would you want to change it from most other manual says? "The following options are available" is FreeBSD manual culture. $otherOS all have a different style. This so unnecessary, it's a stylistic regression, it's triggering, and to call it "simple (and) non-contentious" when you didn't have to say anything, seems like (as a former comitter) on top of everything your underlying intention is... doesn't have enough respect. I am extremely suspicious of anyone wants to change anything on planet earth without clear evidence that they appreciate and respect it first.

Please extend to me the grace you extend to random people who publicly disparage our OS on Reddit for the best effort nature of this response. I have explained why this doesn't work and is offensive, to the best of my emotional capacity, for your actionable benefit, at peril to my reputation, in the hopes that we will all be improved (including maybe someone will come and coach me how to do this better) so that we can all play better in the future. Thanks.

Edit: and there's a lot of actual regressions here where you've removed something that removes part of the meaning.

@dag-erling
Copy link
Member

You are both extremely tedious.

@dag-erling dag-erling closed this Jul 8, 2024
@grahamperrin grahamperrin deleted the grahamperrin-freebsd-version branch July 9, 2024 00:19
@grahamperrin

This comment was marked as resolved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants