cat: split POSIX, BSD and GNU and add 2 examples #9623
Conversation
github-actions
bot
added
the
new command
|
|
||
| `cat -n {{path/to/file}}` | ||
| `cat -u {{/dev/tty12}} > {{/dev/tty13}}` |
There was a problem hiding this comment.
Warning: -u is ignored.
Note: current style guide fallbacks to short options in this case but doesn't specify whether to keep not portable option or remove. My proposal is to discard examples with ignored options as they serve no purpose except that such options exist but do nothing. On the other hand it useful to know as it's a different behavior than interpreting option in the "right" way. Both points of view are reasonable. We can keep mentioning such ignored options in at the end of pages before help/version examples for informational purposes.
There was a problem hiding this comment.
I'm sorry I have some serious trouble parsing half of your comment.
I don't get what you mean by "not portable option[s]". Are you referring to -u? It is portable and this whole PR's reason to exist is that it is.
Some implementations might ignore it (every day I'm learning to hate GNU a bit more), but that doesn't matter as many do not (or at least they say they don't).
There was a problem hiding this comment.
There was a problem hiding this comment.
I'm sorry I have some serious trouble parsing half of your comment. I don't get what you mean by "not portable option[s]". Are you referring to
-u? It is portable and this whole PR's reason to exist is that it is. Some implementations might ignore it (every day I'm learning to hate GNU a bit more), but that doesn't matter as many do not (or at least they say they don't).
Agreed, also it seems to be included in the recent POSIX specification Ref.
Co-authored-by: Emily Grace Seville <EmilySeville7cfg@gmail.com>
Co-authored-by: Emily Grace Seville <EmilySeville7cfg@gmail.com>
No description provided.