Page MenuHomeFreeBSD

[upgrading] igor -Ry and some other rewording and fixes.
ClosedPublic

Authored by mat on Aug 20 2014, 3:09 PM.

Diff Detail

Repository
rD FreeBSD doc repository - subversion
Lint
Lint Skipped
Unit
Unit Tests Skipped

Event Timeline

mat retitled this revision from to [upgrading] igor -Ry and some other rewording and fixes..
mat updated this object.
mat edited the test plan for this revision. (Show Details)
wblock added inline comments.
en_US.ISO8859-1/books/porters-handbook/upgrading/chapter.xml
17 ↗(On Diff #1252)

<acronym> tags on FTP.

31 ↗(On Diff #1252)

It would be better not to mention the particular bug tracking software. Also probably should mention it by both names: the PR or bug database.

108 ↗(On Diff #1252)

s/us//

109 ↗(On Diff #1252)

send-pr is gone.

169 ↗(On Diff #1252)

Break sentence on semicolon.

185 ↗(On Diff #1252)

s/ports/port/

202 ↗(On Diff #1252)

s/try/attempt/

203 ↗(On Diff #1252)

Break sentence on semicolon.

223 ↗(On Diff #1252)

The paren must not be followed by a space.

249 ↗(On Diff #1252)

s/how a port must be,/the structure of a port,/

267 ↗(On Diff #1252)

s/Patch/The patch/

275 ↗(On Diff #1252)

Could probably remove "The Files" here.

283 ↗(On Diff #1252)

s/documented this in this/documented in this/

302 ↗(On Diff #1252)

"As shown, the command will only work with ..."

303 ↗(On Diff #1252)

s/This will work with both/Instead, use the form shown below, which will work with both/

337 ↗(On Diff #1252)

s/New entries must be added to/New entries are added to/

338 ↗(On Diff #1252)

s/order/order,/

339 ↗(On Diff #1252)

with the latest entries first.</para>

Update with wblock's comments

en_US.ISO8859-1/books/porters-handbook/upgrading/chapter.xml
109 ↗(On Diff #1252)

Oh, I did not know that, still use it myself 0:-)

123 ↗(On Diff #1252)

So I guess the problem-reports article should be updated too, to trim all the references to using send-pr.

203 ↗(On Diff #1252)

Maybe igor should try to detect that, and tell people they should make separate sentences ?

en_US.ISO8859-1/books/porters-handbook/upgrading/chapter.xml
16 ↗(On Diff #1328)

This is hard to understand. I think it is trying to say "make sure that the ports tree on the local machine is current, because the port might have already been upgraded. The most recent version of ports can be found..."

But that part about where they can be found is kind of obsolete. People now should just look in the repo.

"When a port is not the most recent version available from the authors, update the local working copy of <filename>/usr/ports</filename>. The port might have already been updated to the new version."

37 ↗(On Diff #1328)

s/the searching/searching/

184 ↗(On Diff #1328)

Serial comma: s/moving/moving,/

en_US.ISO8859-1/books/porters-handbook/upgrading/chapter.xml
17 ↗(On Diff #1437)

The addition helps, but I also meant to remove the rest of the paragraph.

wblock added a reviewer: wblock.

Please build-test and commit.

This revision is now accepted and ready to land.Sep 15 2014, 10:06 PM
mat updated this revision to Diff 1639.

Closed by commit rD45617 (authored by @mat).