Page MenuHomeFreeBSD

De-advertize send-pr(1)
ClosedPublic

Authored by rene on Apr 27 2015, 7:09 PM.

Details

Summary

The send-pr(1) has been removed in FreeBSD 10, so remove it from most of the documentation (books, articles, websites) unless there is an explicit / "obvious" reason.

Do not touch old news articles, release notes, or the CGI scripts.

Test Plan
  • edit and review
  • test build

Diff Detail

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

Event Timeline

rene updated this revision to Diff 5031.Apr 27 2015, 7:09 PM
rene retitled this revision from to De-advertize send-pr(1).
rene updated this object.
rene edited the test plan for this revision. (Show Details)
rene added reviewers: Unknown Object (Project), kwm.
rene set the repository for this revision to rD FreeBSD doc repository.
rene added projects: Doc Committers, gnome.
mat added a subscriber: mat.Apr 27 2015, 11:04 PM

Good, looks like I did a good job at cleaning it from the Porter's Handbook 0:-)

eadler removed a project: Doc Committers.
bjk added a subscriber: bjk.May 1 2015, 6:35 PM

Thank you for doing this; a few minor issues inline.

en_US.ISO8859-1/articles/problem-reports/article.xml
1011 ↗(On Diff #5031)

This could all fit on one line, I think.

en_US.ISO8859-1/books/dev-model/book.xml
2079 ↗(On Diff #5031)

bugzilla reports do not go over email from the web interface

en_US.ISO8859-1/books/handbook/eresources/chapter.xml
1116 ↗(On Diff #5031)

WEB in all caps looks super-clunky this decade, but it need not change in this commit unless you want to.

en_US.ISO8859-1/htdocs/gnome/docs/bugging.xml
66 ↗(On Diff #5031)

should be "or", not "and" (probably no comma, either)

rene added inline comments.May 1 2015, 6:39 PM
en_US.ISO8859-1/articles/problem-reports/article.xml
1011 ↗(On Diff #5031)

True, but that would violate our rule of not mixing content and whitespace changes?

en_US.ISO8859-1/books/dev-model/book.xml
2079 ↗(On Diff #5031)

Oops, you're right.

en_US.ISO8859-1/htdocs/gnome/docs/bugging.xml
66 ↗(On Diff #5031)

We are using the serial comma, right?

bjk added inline comments.May 1 2015, 6:42 PM
en_US.ISO8859-1/articles/problem-reports/article.xml
1011 ↗(On Diff #5031)

I don't think so; the content on the second line is new content, so there is nothing being preserved other than the first half of the line being changed.

en_US.ISO8859-1/htdocs/gnome/docs/bugging.xml
66 ↗(On Diff #5031)

Our style mandates the serial comma, but there are only two options here, so no comma is needed.

rene updated this revision to Diff 5145.May 1 2015, 6:45 PM
rene removed rD FreeBSD doc repository as the repository for this revision.

Process comments from bjk

rene updated this revision to Diff 5146.May 1 2015, 6:53 PM

no serial comma needed...

bjk added inline comments.May 1 2015, 7:02 PM
en_US.ISO8859-1/books/handbook/ports/chapter.xml
1862 ↗(On Diff #5146)

"to submit"

rene updated this revision to Diff 5147.May 1 2015, 7:14 PM

add a missing "to"

bjk added inline comments.May 1 2015, 8:07 PM
en_US.ISO8859-1/htdocs/advocacy/myths.xml
110 ↗(On Diff #5147)

by using by using?

rene updated this revision to Diff 5152.May 1 2015, 8:22 PM

remove duplicated "by using"

bjk accepted this revision.May 1 2015, 9:18 PM
bjk added a reviewer: bjk.
This revision is now accepted and ready to land.May 1 2015, 9:18 PM
This revision was automatically updated to reflect the committed changes.