Skip to content

sha*sum: add more information link #5669

New issue

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

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

Already on GitHub? # to your account

Merged
merged 1 commit into from
Apr 2, 2021

Conversation

marchersimon
Copy link
Collaborator

For #5510

@marchersimon marchersimon added page edit Changes to an existing page(s). translation Issues requesting translating pages from English to other languages. labels Apr 2, 2021
bl-ue
bl-ue previously approved these changes Apr 2, 2021
@bl-ue bl-ue dismissed their stale review April 2, 2021 17:30

wait.......

@bl-ue
Copy link
Contributor

bl-ue commented Apr 2, 2021

@marchersimon please see @vladimyr's PR #5658 where he shorted the coreutils links.

@marchersimon
Copy link
Collaborator Author

I tried that, but that doesn't work here, since the url is sligthly different from the other pages.

@bl-ue
Copy link
Contributor

bl-ue commented Apr 2, 2021

Yeah, I just discovered that myself. Well, never mind 😛

@bl-ue
Copy link
Contributor

bl-ue commented Apr 2, 2021

Though the linked page is tiny (this is all of the text content, Ctrl+A - Ctrl+C):

Previous: sha1sum invocation, Up: Summarizing files [Contents][Index]

6.7 sha2 utilities: Print or check SHA-2 digests

The commands sha224sum, sha256sum, sha384sum and sha512sum compute checksums of various lengths (respectively 224, 256, 384 and 512 bits), collectively known as the SHA-2 hashes. The usage and options of these commands are precisely the same as for md5sum and sha1sum. See md5sum invocation.

...there's that link at the end which is pretty obvious IMO. But if the others think, maybe we should change the links to all point to that md5sum link. 🤔

@marchersimon
Copy link
Collaborator Author

The problem I have with directly linking to the md5sum page, is that nowhere in the page is mentioned that those arguments also work for the sha*sum pages, which will probably confuse many and make it seem the link is wrong.

@bl-ue
Copy link
Contributor

bl-ue commented Apr 2, 2021

@navarroaxel, @glenacota, @YUTYO Someone, can we get this merged? I'd like to incorporate it into #5664 before that gets merged.

@navarroaxel navarroaxel merged commit 44b8762 into tldr-pages:master Apr 2, 2021
@marchersimon marchersimon deleted the shasum-link branch April 2, 2021 19:05
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
page edit Changes to an existing page(s). translation Issues requesting translating pages from English to other languages.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants