Some links in the FreeBSD handbook are HTTP links. I am changing them to HTTPS links.
Details
Details
- Reviewers
minsoochoo0122_proton.me - Group Reviewers
docs - Commits
- R9:8139a74a3147: Handbook - Upgrade some links in introduction chapter, also use https
cd documentation
make run
Diff Detail
Diff Detail
- Repository
- R9 FreeBSD doc repository
- Lint
Lint Skipped - Unit
Tests Skipped
Event Timeline
Comment Actions
If you checked that all the URLs you changed work with https, can you mention that in the test plan? If you haven't, you can use something like
git grep -oE 'https://[^[]+'
to extract URLs from the source, feeding into a sh/csh/awk/perl loop using something like
fetch -o /dev/null $url || echo $url doesn't work.
| documentation/content/en/books/handbook/introduction/_index.adoc | ||
|---|---|---|
| 123 | For future work: merge into TrueNAS below in opensource projects FreeBSD spawned? Or at least crossreference them? | |
| 126 | While there: US English collective singular (see above "The TrueNAS line of unified storage appliances is based on FreeBSD.") and missing hyphen. | |
| 229 | Future change or while here: s/is used/was used with SVN/ | |
Comment Actions
Maybe expand the title here, to include the phrase FreeBSD Handbook.
| documentation/content/en/books/handbook/introduction/_index.adoc | ||
|---|---|---|
| 121–122 | https://www.emc.com/isilon is 403 forbidden. Related Dell pages are somewhat outdated (with items not found, and so on) … amongst the essentials at https://www.dell.com/en-uk/dt/storage/isilon/onefs-operating-system.htm, alongside the technical overview (404 not found) I found this April 2022 PDF:
Not a typo. The Technical Overview that is found is linked from the deep dive, not from the Technical Overview. https://www.skendric.com/map/EMC-Isilon-OneFS-An-Ops-Managers-Introduction.pdf (2021) has two EMC links at top left, both are 403 forbidden. | |
| 122–136 | Lowercase URL, with a trailing / | |
| 123 | Also for the future, here and elsewhere, https://www.truenas.com/compare-editions/ FreeBSD is:
| |
| 126 | Whilst here: bhyve is no longer new. | |
| 138–140 | For the future: review this paragraph. Indirectly via After 23 years, Weather Channel’s iconic computerized channel is shutting down | Ars Technica:
| |
| documentation/content/en/books/handbook/introduction/_index.adoc | ||
|---|---|---|
| 121–122 | The EMC lineup has been moved to PowerScale lineup. I will edit the link to PowerScale link as well as the description. | |
| 138–140 | Thank you for the information! Intellistar will be no longer used as the article says. I will remove the Weather channel from the list | |
| documentation/content/en/books/handbook/introduction/_index.adoc | ||
|---|---|---|
| 123 | I'll make a small reference here in a different revision since it is an open-source software AND enterprise software. | |
| documentation/content/en/books/handbook/introduction/_index.adoc | ||
|---|---|---|
| 229 | ||
| documentation/content/en/books/handbook/introduction/_index.adoc | ||
|---|---|---|
| 123 | ||
| documentation/content/en/books/handbook/introduction/_index.adoc | ||
|---|---|---|
| 135 | It's not immediately obvious, but at https://www.sophos.com/en-us/support/downloads/sophos-email and elsewhere I see:
 | |
| documentation/content/en/books/handbook/introduction/_index.adoc | ||
|---|---|---|
| 135 | … I'll raise a separate bug report, or review, after there's a commit for this review. Thanks. | |
Comment Actions
I reflected all the suggestions and it seems that the reviewers don't have further opinion on this. I'll accept this revision just for now, but if this revision needs any modification, please I would like to hear your opinions.