tldr: fix typo

[prometoys]

Fixed typo. It seems -o is --os selection.

Maybe --update instead of -u is more convenient, because old tldr-hs client on Ubuntu/Debian doesn't understand -u.

[navarroaxel]

Hi @prometoys, welcome and thanks for your submission. I think you should check the client-specification.

[mfrw]

Good Catch!
Thank you @prometoys for your contribution.

[mfrw]

Good Catch! Thank you @prometoys for your contribution.

Apologies, as per the client-specification this is not correct.
I am using tealdear link, which probably is not conformant to the spec.

[navarroaxel]

Apologies, as per the client-specification this is not correct. I am using tealdear link, which probably is not conformant to the spec.

😆 then I will request changes to avoid merge it by mistake

[prometoys]

Hi @navarroaxel,

thanks for the link. I checked different tldr clients:

tldr-hs 0.9.1
tldr from nmp 3.3.7
tealdeer v1.4.1 (Linux)
tldr-py 0.7.0

None of them understands the option -p. Which client accept -p ?

[CleanMachine1]

Our official clients - Python

Uses -p

[MasterOdin]

tldr-hs accepts the -p flag per their README.

[prometoys]

tldr-hs accepts the -p flag per their README.

Ok, I must apologize. The tldr-hs 0.9 does understand -p, the 0.4.0 in Ubuntu neither -o or -p.

I downloaded tldr-hs 0.9.1 yesterday, but probably after tests on different machines and different clients cand their versions onfused -p and -o myself.

Our official clients - Python
Uses -p

This is correct, can confirm it with tldr-py 1.2.0 on fedora. I guess with tldr-py (0.7) on Ubuntu is the same problem like tldr-hs, too old versions.

The node-client is recommended on tldr.org, but expects -o (Version 3.3.7).

https://github.com/tldr-pages/tldr-node-client/blob/1641dea94df3aabcd36f381dcac61a50905a36ea/bin/tldr#L25

From a user perspective running tldr tldr this situation is confusing. Maybe there should be a hint, like with update (-u)?

Thanks for you patience

(btw, I love the tldr approach)

[sbrl]

For tldr-node-client, see tldr-pages/tldr-node-client#337, which needs some attention

[github-actions]

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

[CleanMachine1]

Can we close this PR since it has been discussed and is technically incorrect

[sbrl]

Yeah, I suggest we close this PR. If there's more to discuss about this, please open a new issue.