Details
- Reviewers
carlavilla - Group Reviewers
docs Linux Emulation - Commits
- R9:8d0c59026e4a: handbook: Get rid of "(R)" in "Linux(R)"
Diff Detail
- Repository
- R9 FreeBSD doc repository
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
Ah, found the Linux trademark guidelines: https://www.linuxfoundation.org/the-linux-mark/
- For each web page, advertisement, or publication, the first prominent appearance of LINUX should feature the “circle R” character adjacent to the X, as follows: Linux®
- At the end of your web page, advertisement, publication or media broadcast, include the following text in a legible font and size: Linux® is the registered trademark of Linus Torvalds in the U.S. and other countries.
The first is done in this review. I suspect we have a statement for 2 in a boilerplate footer on the doc site (we have https://www.freebsd.org/copyright/trademarks/) but we should add such a mention to the end of this page.
We do have the trademark information for the Handbook itself: https://docs.freebsd.org/en/books/handbook/, click "> trademarks" just below the Copyright, below "FreeBSD Handbook" title. I don't think we need to include the remark in every chapter; I don't see that done anywhere else in the handbook.
IANAL, so I assume that since there's one Linux® early one, removing the (R) everywhere else instead of replacing all with ® is enough of an acknowledgement. Other than that and the 3 nits I spotted, LGTM.
documentation/content/en/books/handbook/linuxemu/_index.adoc | ||
---|---|---|
58 | Since you're touching this line, maybe be more specific than "overly" (eg, how many are too many? 23? 42?) or get rid of it altogether. | |
148–149 | Maybe "a Linux system using the same CPU architecture", since the handbook instructs to copy the files a few paragraphs below? | |
299 | Since you're touching that area. |
Thanks! I'd prefer to make those fixes later on, in a large-ish manual patch; I've split out this thing here because it's a mechanical change and it will make it easier to review the rest.
Maybe it's worth mentioning that (R) gets rendered by AsciiDoctor as ® as is both outlined here and can be spotted on the rendered output of the existing page.
Yep I spotted that after the comment and left a note in the other review https://reviews.freebsd.org/D33738#763236
Does that also mean the existing ® (in "description:" near the top) should be changed to (R)?