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

pages*: use manned.org as much as possible (part 1) #13711

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

sebastiaanspeck
Copy link
Member

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

@sebastiaanspeck
Copy link
Member Author

#12999

@github-actions github-actions bot added page edit Changes to an existing page(s). translation Translate pages from one language to another. labels Sep 15, 2024
@sebastiaanspeck sebastiaanspeck added the mass changes Changes that affect multiple pages. label Sep 15, 2024
@tldr-bot
Copy link

Hello! I've noticed something unusual when checking this PR:

  • The page pages.de/linux/pacman.md is outdated, based on number of commands.
  • The page pages.fr/linux/pacman.md is outdated, based on number of commands.
  • The page pages.ml/linux/pacman.md is outdated, based on number of commands.
  • The page pages.pt_BR/linux/at.md is outdated, based on number of commands.
  • The page pages.pt_PT/linux/pacman.md is outdated, based on number of commands.
  • The page pages.tr/linux/pacman.md is outdated, based on number of commands.
  • The page pages.zh/linux/pacman.md is outdated, based on number of commands.

Is this intended? If so, just ignore this comment. Otherwise, please double-check the commits.

@@ -3,7 +3,7 @@
> Arch Linux package manager utility.
> See also: `pacman-database`, `pacman-deptest`, `pacman-files`, `pacman-key`, `pacman-mirrors`, `pacman-query`, `pacman-remove`, `pacman-sync`, `pacman-upgrade`.
> For equivalent commands in other package managers, see <https://wiki.archlinux.org/title/Pacman/Rosetta>.
> More information: <https://man.archlinux.org/man/pacman.8>.
> More information: <https://manned.org/pacman.8>.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

According to the current style guide, linking to the author's provided man page is preferred.

On the More information link line, we prefer linking to the author's provided documentation of the command-line reference or the man page.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Then we should be linking to https://pacman.archlinux.page/pacman.8.html

Copy link
Member

@kant kant left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

spanish chapter: LGTM

@Managor
Copy link
Collaborator

Managor commented Sep 17, 2024

I just saw a page you might have missed: adduser https://github.com/tldr-pages/tldr/blob/main/pages/linux/adduser.md

@sebastiaanspeck
Copy link
Member Author

sebastiaanspeck commented Sep 17, 2024

I just saw a page you might have missed: adduser https://github.com/tldr-pages/tldr/blob/main/pages/linux/adduser.md

I know there are still pages left. I used this list:

  • man7.org
  • linux.org
  • die.net
  • man.page
  • mankier.com
  • manpage.io
  • manpages.org
  • man.he.net

Since the page you point out, is manpages.debian.org, I didn't spot it. The list of changed files is already a lot to be reviewed. We definitely need a new PR to fix the remaining URLs.

@Managor
Copy link
Collaborator

Managor commented Sep 17, 2024

I did find . -type f -exec awk '!/manned/ && /man/ && /More information/ ' {} \; > ~/log on all the pages, sorted them and filtered by unique and there were a couple that caught my eye

  • manpages.ubuntu.com
  • manpages.debian.org
  • manpages.opensuse.org
  • manpagez.com
  • man.freebsd.org
  • man.openbsd.org
  • man.netbsd.org
  • man.voidlinux.org
  • man.cx

Full list of the resulting lines if you want to check it
log3.txt

@sebastiaanspeck sebastiaanspeck changed the title pages*: use manned.org as much as possible pages*: use manned.org as much as possible (part 1) Sep 17, 2024
@sebastiaanspeck
Copy link
Member Author

sebastiaanspeck commented Sep 17, 2024

I did find . -type f -exec awk '!/manned/ && /man/ && /More information/ ' {} \; > ~/log on all the pages, sorted them and filtered by unique and there were a couple that caught my eye

  • manpages.ubuntu.com
  • manpages.debian.org
  • manpages.opensuse.org
  • manpagez.com
  • man.freebsd.org
  • man.openbsd.org
  • man.netbsd.org
  • man.voidlinux.org
  • man.cx

Full list of the resulting lines if you want to check it log3.txt

Thank you for providing the list. I want to point out that I focussed on Linux.

Looking at man.freebsd.org e.g., I found out that other platforms used those site as well (2 pages in netbsd and 1 in osx). I renamed this PR to part 1. There is still some work to do to reach a uniform way.

@sebastiaanspeck
Copy link
Member Author

Since this PR is maybe a little too big, I created #13725 to replace manpages.debian.org with manned.org. That PR will contain some pages also included in this PR. Therefore I mark this PR as draft.

@sebastiaanspeck sebastiaanspeck marked this pull request as draft September 17, 2024 09:18
@Managor
Copy link
Collaborator

Managor commented Sep 17, 2024

Couple more

Although I'm not sure about some of these. Some of these are kinda first party sources but they contain manpages that can also be found in manned

Then there are www.unix.com links which return 404

@tldr-bot
Copy link

Hello! I've noticed something unusual when checking this PR:

  • The page pages.de/linux/pacman.md is outdated, based on number of commands.
  • The page pages.fr/linux/pacman.md is outdated, based on number of commands.
  • The page pages.ml/linux/pacman.md is outdated, based on number of commands.
  • The page pages.pt_BR/linux/at.md is outdated, based on number of commands.
  • The page pages.pt_PT/linux/pacman.md is outdated, based on number of commands.
  • The page pages.tr/linux/pacman.md is outdated, based on number of commands.
  • The page pages.zh/linux/pacman.md is outdated, based on number of commands.

Is this intended? If so, just ignore this comment. Otherwise, please double-check the commits.

@Managor Managor mentioned this pull request Sep 17, 2024
5 tasks
@Managor
Copy link
Collaborator

Managor commented Sep 17, 2024

I ran all "More information" links through lychee, pruned "Too Many Requests" and "Forbidden" responses, sorted and uniqued.
This is the leftover links that returned 404 or something similar
log.txt

@sebastiaanspeck
Copy link
Member Author

I ran all "More information" links through lychee, pruned "Too Many Requests" and "Forbidden" responses, sorted and uniqued.

This is the leftover links that returned 404 or something similar

log.txt

Thanks, the scope of this PR is to uniform the Linux URLs, so the non-Linux ones should be addressed in another PR. FYI, in #12506 there is a Python-script that I use to check all links

@Managor
Copy link
Collaborator

Managor commented Sep 18, 2024

Yeah, it wasn't intended for this PR. I just thought it would be nice if all these manpage TODOs were collected in the same place.

@kbdharun kbdharun self-assigned this Sep 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mass changes Changes that affect multiple pages. page edit Changes to an existing page(s). translation Translate pages from one language to another.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants