Page MenuHomeFreeBSD

Add requested commercial vendors
ClosedPublic

Authored by sd_beastie.io on Apr 30 2016, 4:57 PM.

Details

Summary

PRs: 207615, 208989, 208919, 208460

Description text are mostly copied verbatim except in one case where we removed a personal reference back to an individual. Wanted to keep it organization oriented.

Ran igor to cleanup newly added section. This file has a ton of igor warnings/errors - filed a new bug to cleanup vendor list along with igor errors (209169).

Test Plan

igor and webbuild

Diff Detail

Repository
rD FreeBSD doc repository - subversion
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

sd_beastie.io retitled this revision from to Add requested commercial vendors.
sd_beastie.io updated this object.
sd_beastie.io edited the test plan for this revision. (Show Details)
sd_beastie.io added a reviewer: wblock.
sd_beastie.io set the repository for this revision to rD FreeBSD doc repository - subversion.

changed Arp Networks to ARP Networks

Remove .com from Delimeter title.

share/xml/commercial.isp.xml
381 ↗(On Diff #15973)

This is a whitespace-only change, and should be not be included with this content change.

1356 ↗(On Diff #15973)

Plural:
s/newspaper/newspapers/

1358 ↗(On Diff #15973)

Use the serial comma:
s/services/services,/

1368 ↗(On Diff #15973)

Use the serial comma:
s/colocation/colocation,/

1380 ↗(On Diff #15973)

This sentence has several problems. It appears to be at least two sentences smushed together.

1385 ↗(On Diff #15973)

Use the serial comma:
s/BSD/BSD,/

1394 ↗(On Diff #15973)

Use the serial comma:
s/NetBSD/NetBSD,/

1395 ↗(On Diff #15973)

Why is "server management" capitalized?

1396 ↗(On Diff #15973)

"&" is for company names and handwritten notes, don't use it here.

1397 ↗(On Diff #15973)

Use the serial comma:
s/PHP/PHP,/

Also needs a space before the opening paren.

1399 ↗(On Diff #15973)

Needs a comma after Columbia.

Lesson learned - these texts dont have to be verbatim copies of what the customer provider. Will add new patch soon.

share/xml/commercial.isp.xml
1380 ↗(On Diff #15973)

I was under the impression, perhaps incorrectly, that we were supposed to copy/paste the text verbatim from client. I am re-doing the paragraph to simplify the text.

1395 ↗(On Diff #15973)

As above - I thought this was supposed to copied verbatim from the client :-/

sd_beastie.io marked 2 inline comments as done.

Updated diff based on review comments

share/xml/commercial.isp.xml
1380 ↗(On Diff #16747)

"tool" needs to be "tools".

1381 ↗(On Diff #16747)

Please remove the comma after "installs".

sd_beastie.io added inline comments.
share/xml/commercial.isp.xml
1380 ↗(On Diff #16747)

My bad - VI with special characters display enabled show $ for line ending right after tool: "tool$". Missed that.

1381 ↗(On Diff #16747)

changed to installations instead of installs as well "automated OS installations and dedicated..."

sd_beastie.io marked 2 inline comments as done.

Updated based on review and fix a few more spelling mistakes in the file.

share/xml/commercial.isp.xml
655 ↗(On Diff #16755)

Got one, but missed one: "possibility"

wblock edited edge metadata.

Okay, good.

This revision is now accepted and ready to land.May 24 2016, 4:46 PM
This revision was automatically updated to reflect the committed changes.