Page MenuHomeFreeBSD

Replace f10 with c6 in the Handbook
ClosedPublic

Authored by xmj on Nov 5 2014, 4:36 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mar 15 2024, 7:14 AM
Unknown Object (File)
Mar 15 2024, 7:08 AM
Unknown Object (File)
Dec 16 2023, 6:26 AM
Unknown Object (File)
Dec 16 2023, 6:26 AM
Unknown Object (File)
Dec 11 2023, 4:31 AM
Unknown Object (File)
Nov 26 2023, 8:00 AM
Unknown Object (File)
Nov 23 2023, 3:46 PM
Unknown Object (File)
Nov 22 2023, 10:09 PM
Subscribers

Details

Reviewers
swills
wblock
koobs
bcr
Group Reviewers
Doc Committers
Summary

In preparation for replacing Fedora 10 with CentOS 6.6 as default Linux
userland, amend the handbook by s/f10/c6/ (and correcting one typo).

Test Plan

make -> looks good
make validate -> no errors

Diff Detail

Repository
rD FreeBSD doc repository - subversion
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

xmj retitled this revision from to Replace f10 with c6 in the Handbook.
xmj updated this object.
xmj edited the test plan for this revision. (Show Details)
xmj added reviewers: Doc Committers, swills, koobs.
wblock added inline comments.
en_US.ISO8859-1/books/handbook/desktop/chapter.xml
224

Install <package role="port">www/nspluginwrapper</package> from the port.

225–226

Not your fault, but this sentence is weird. It's not really that large a port. Is it large enough to make a difference to the user? Probably not, so I suggest ending the sentence after the package name.

If "which is a large port" is kept, a comma is needed before it.

229–230

Install <package role="port">ww/linux-c6-flashplugin11</package> from the port.

229–230

I think the command needs to be updated because browser_plugins is not present on new systems. But I have not tested it.

Either way, we should try to avoid line continuations.

230–232

As above, I'm not sure this is the way to do it any more. If it is, this step should be mentioned before creating the link.

309–310

Just mentioning the port name is enough:

Then install <package role="port">www/opera-linuxplugins</package>.

en_US.ISO8859-1/books/handbook/linuxemu/chapter.xml
130–131

s/In order for/For/

130–131

s/the following/this/

en_US.ISO8859-1/books/handbook/desktop/chapter.xml
229–230

This is handled by USES= webplugin:linux, normally automatically, from within a port:

xmj@mx12:~/repos/linux-ports-cleantree/www/linux-c6-flashplugin11$ make -V WEBPLUGIN_DIR
/usr/local/lib/browser_plugins/linux-c6-flashplugin

So, that step is actually superfluous. I'll remove it.

230–232

Again, superfluous. Nice catch!

Update with wblock's suggestions

OK, by me, needs review by doc folks.

en_US.ISO8859-1/books/handbook/desktop/chapter.xml
223

Best to use <package role="port"> when explicitly referring to a port.

230–232

<package role="port">

309–310

<package role="port">

309–310

The "...port, as a package is not available" can be misread. Break into two sentences:

"... port. Licensing prevents making a package available."

xmj edited edge metadata.

Update with wblock's suggestions pt2

wblock added a reviewer: wblock.

Looks good to me. Thanks!

en_US.ISO8859-1/books/handbook/desktop/chapter.xml
311

Nice!

This revision is now accepted and ready to land.Nov 12 2014, 3:22 PM
xmj edited edge metadata.

Typos pointed out by @netchild

This revision now requires review to proceed.Dec 6 2014, 9:52 AM
bcr added a reviewer: bcr.
bcr added a subscriber: bcr.

Looks good to me.

This revision is now accepted and ready to land.Dec 7 2014, 12:28 PM
en_US.ISO8859-1/books/handbook/desktop/chapter.xml
8

This is a whitespace-only change and should be done separately.

en_US.ISO8859-1/books/handbook/desktop/chapter.xml
8

The first actual change is on line 223, so I'm not sure what you mean.

If it was different it would be marked red/green here in Phabric

en_US.ISO8859-1/books/handbook/desktop/chapter.xml
8

It was light green here, apparently a change in the underlying file. Never mind.

Committed revision 46078.