windows/curl, windows/wget: fix tldr client param inconsistencies

[reinhart1010]

• [x] The page(s) are in the correct platform directories: common, linux, osx, windows, sunos, android, etc.
• [x] The page(s) have at most 8 examples.
• [x] The page description(s) have links to documentation or a homepage.
• [x] The page(s) follow the content guidelines.
• [x] The PR title conforms to the recommended templates.
• Version of the command being documented (if known):

Due to tldr-pages/tldr-node-client#336 impacting some Windows users who are using the Node.js version of tldr client, I decided to add the non-standard --os version to the documentation as a workaround.

Hopefully tldr-pages/tldr-node-client#337 can be merged to fix this issue.

[reinhart1010]

Only the Node.js client uses --os - see https://github.com/tldr-pages/tldr-node-client/issues/336

@marchersimon Maybe I should stick back using -p and -o?

[marchersimon]

Yes, we should definitively use the short options, we already had to reject some PRs about exactly this in the past.

I'm not so sure regarding the -o flag. We don't even mention it in the main tldr page, so why should we add it here? (Also, I actually have a PR open to fix this issue, but it seems a little stuck → https://github.com/tldr-pages/tldr-node-client/pull/337)

[reinhart1010]

I'm not so sure regarding the -o flag. We don't even mention it in the main tldr page, so why should we add it here?

Our client spec doesn't require clients to output the version (i.e. Node.js, C++, Python instead of 1.0.1) this tldr client came from, on their -v and -h, making it difficult for existing Windows users to check whether they are using the "correct" (spec-compliant version) of tldr client.

Sure, Windows users can check the directory where tldr was run by using where tldr or Get-Command tldr (equivalent to which tldr in Windows Command Prompt and PowerShell).

https://github.com/tldr-pages/tldr-node-client/pull/336 already breaks some Windows users who installed tldr via Node.js as they were unable to view the original curl/wget documentation using tldr curl -p common. So I have to mention a workaround instead until https://github.com/tldr-pages/tldr-node-client/pull/337 is merged (and wait for months until everyone gets the latest Node.js version of tldr).

[marchersimon]

@CleanMachine1 @sbrl @navarroaxel any thoughts on this? If we add the -o option here, should we maybe also add it to the tldr page itself?

[navarroaxel]

I think we should stick to the client specification, otherwise someone can ask to add another flag used in another tldr client. Also, @marchersimon why you don't finish the node client PR? :smile:

[marchersimon]

Also, @marchersimon why you don't finish the node client PR? :smile:

That's because I hardly know any javascript. I just replaced the occurences of --os with --platform. This would remove the old --os` flag, which introduces a breaking change. And now nobody really wants to finish it. :smile:

[sbrl]

Ummm the client spec does require you to output the version of the spec you implement: https://github.com/tldr-pages/tldr/blob/main/CLIENT-SPECIFICATION.md?plain=1#L38

-v, --version | Yes | Shows the current version of the client, and the version of this specification that it implements.

....and then slightly earlier:

A number of command-line options MUST be supported (unless otherwise specified) if a CLI is implemented:

...indeed, the Node.js client using --os predates the client spec, and is a known issue nobody has gotten around to fixing yet.

Ideally, the Node.js client should be updated rather than this page being updated to workaround the actual issue.

[github-actions[bot]]

Hi all! This thread has not had any recent activity. Are there any updates? Thanks!

[reinhart1010]

Hello, is there any updates about this?