Page MenuHomeFreeBSD

Replace f10 with c6 in the Handbook
ClosedPublic

Authored by xmj on Nov 5 2014, 4:36 PM.
Tags
None
Referenced Files
F87103455: D1117.diff
Sat, Jun 29, 10:02 AM
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
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
222

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

224

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.

230

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

236

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.

241

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.

321

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
131

s/In order for/For/

132

s/the following/this/

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

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.

241

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
221

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

241

<package role="port">

319

<package role="port">

320–321

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
323

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
6

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

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

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
6

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

Committed revision 46078.