handbook/mirrors: Refresh the whole section

[dbaio]

Summary

The main change here is that now the official mirrors are the official mirrors managed by cluster adm team.

The other mirrors are now named community mirrors.

-->

Remove all mirrors sections [1] (much outdated information) in favor of a small and unified one [2].

I checked every single mirror (for each protocol) and removed all that is not responding or that has duplicate IPs with another mirror [3].

This was discussed with clusteradm, but the text needs review and possible improvements.

1 - handbook/mirrors-http, handbook/mirrors-ftp, handbook/mirrors-rsync, handbook/eresources-web

2 - new section preview

3 - mirrors removed from the list

Test Plan

new section preview

[pauamma_gundo.com]

Likewise for all instances of icon:envelope.

documentation/content/en/books/handbook/mirrors/_index.adoc

54

Simplify

56

Maybe expand the state codes?

69

Maybe https instead for consistency with eg, the committer's guide? Or would (the lack of) certificates stand in the way?

73

https?

85

91

alt text? How would a screenreader render the icon?

551

Moved, see https://www.freebsdmall.com/cgi-bin/fm/company.html?id=VJpUtorT&mv_arg=mv_arg&mv_pc=2

561–564

No longer providing a postal address. I'd remove this or at least check with them. (Let me know, for the latter, if you want me to supply the French.)

565

Either remove completely or use contact form URL (https://www.getlinux.fr/index.php?route=information/contact) instead.

566

569–573

See end of sidebar on left for current contact info.

574

[trasz]

Nice! This cleanup was long overdue. A few nits though.

documentation/content/en/books/handbook/eresources/_index.adoc

1182

Can we just get rid of this section? First time I found it in Handbook, I was like "where are there two Mirrors sections", and now with this patch it seems entirely redundant.

documentation/content/en/books/handbook/mirrors/_index.adoc

55

It's perhaps just me, but I think it would look better without this (and several other) empty lines, ie. with longer paragraphs.

69

Also, maybe capitalise the protocol names, eg s/http/HTTP/g.

[pauamma_gundo.com]

I've suggested changing both "FreeBSD FTP server" and "FreeBSD server" (linking to download.freebsd.org) to "FreeBSD download server" for consistency, but if you'd rather s/FreeBSD FTP server/FreeBSD server/g only, that's fine with me as well.

documentation/content/en/books/arch-handbook/_index.adoc

53

documentation/content/en/books/arch-handbook/book.adoc

52

documentation/content/en/books/developers-handbook/_index.adoc

57

documentation/content/en/books/developers-handbook/book.adoc

56

documentation/content/en/books/handbook/_index.adoc

58

documentation/content/en/books/handbook/book.adoc

58

[dbaio]

• Address points from trasz and pauamma

[dbaio]

Hi, thanks for reviewing @trasz and @pauamma_gundo.com

documentation/content/en/books/arch-handbook/_index.adoc

53

I liked of FreeBSD download server, thanks

documentation/content/en/books/handbook/eresources/_index.adoc

1182

I agree, removed

documentation/content/en/books/handbook/mirrors/_index.adoc

69

There are too many protocols here on this page. I thought it would be easy to read this way, but I can change. What do you guys prefer? Capitalize all of them?

91

I couldn't make it work with icon-macro.

[pauamma_gundo.com]

Still need to update the postal addresses and contact info for retailer sites and make some accessibility changes (I think I got all missing alt= for envelope icons, but please check).

documentation/content/en/books/handbook/mirrors/_index.adoc

99–101

103–105

111–113

115–117

119–121

123–125

135–137

155–157

163–165

183–185

191–193

203–205

207–209

211–213

215–217

223–225

227–229

239–241

243–245

251–253

255–257

263–265

551

Still need to update that.

561–564

And that.

569–573

And that.

[dbaio]

documentation/content/en/books/handbook/mirrors/_index.adoc

99–101

oops, it seems I let my tests here.
I'll update this review tonight. tks

[dbaio]

• Address points from pauamma

[dbaio]

documentation/content/en/books/handbook/mirrors/_index.adoc

574

I'll let it without https; it's returning a certificate error.

[pauamma_gundo.com]

Having dug around in the AsciiDoctor syntax and looked at the example in https://docs.asciidoctor.org/asciidoc/latest/macros/icon-macro/#icon-macro-attributes-image-mode-only and not just the text above it, I figure the correct way to specify the alt= attribute in the resulting HTML img tag is United Kingdom icon:envelope[mirror contact,link=mailto:{mirrors-uk-email}] and likewise for other countries. Apologies for misreading that doc.

And once these are taken care of, the change LGTM.

[dbaio]

In D34156#774584, @pauamma_gundo.com wrote:

Having dug around in the AsciiDoctor syntax and looked at the example in https://docs.asciidoctor.org/asciidoc/latest/macros/icon-macro/#icon-macro-attributes-image-mode-only and not just the text above it, I figure the correct way to specify the alt= attribute in the resulting HTML img tag is United Kingdom icon:envelope[mirror contact,link=mailto:{mirrors-uk-email}] and likewise for other countries. Apologies for misreading that doc.

And once these are taken care of, the change LGTM.

I tried that, and it didn't change html.

This way works for image icons, and we are using here font icons; that's why I used the title option.

[philip]

I'm not sure if all those hostmaster@ aliases work but otherwise this diff looks good to me. Many thanks for doing this work!

[pauamma_gundo.com]

In D34156#774607, @dbaio wrote:

This way works for image icons, and we are using here font icons; that's why I used the title option.

Oh, OK. I wasn't aware of that. LGTM too, then.