Page MenuHomeFreeBSD
Differential D20736

The porter's handbook Go chapter: document no_targets/run args and GO_PORT variable
ClosedPublic
Actions

Authored by dmgk on Jun 24 2019, 4:21 PM.
Tags
None
Referenced Files
F81638128: D20736.id.diff
Fri, Apr 19, 8:14 AM
Unknown Object (File)
Wed, Apr 17, 4:53 AM
Unknown Object (File)
Thu, Mar 28, 11:56 PM
Unknown Object (File)
Dec 25 2023, 4:24 AM
Unknown Object (File)
Nov 26 2023, 2:41 AM
Unknown Object (File)
Nov 24 2023, 12:15 AM
Unknown Object (File)
Nov 9 2023, 2:53 PM
Unknown Object (File)
Nov 8 2023, 10:30 AM
View All 24 Files
Subscribers
None

Details

Reviewers
mat
tobik
swills
jlaffaye
Commits
rD53205: Porter's Handbook: Document no_targets/run and GO_PORT after ports r505321

Diff Detail

Repository
rD FreeBSD doc repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

dmgk created this revision.Jun 24 2019, 4:21 PM
Herald added a reviewer: mat. · View Herald TranscriptJun 24 2019, 4:21 PM
Harbormaster completed remote builds in B24999: Diff 58948.Jun 24 2019, 4:21 PM
dmgk added reviewers: tobik, swills, jlaffaye.Jun 24 2019, 4:22 PM
tobik added inline comments.Jun 25 2019, 5:53 AM
en_US.ISO8859-1/books/porters-handbook/uses/chapter.xml
1151 ↗(On Diff #58948)

"A build dependency"

1152 ↗(On Diff #58948)

"the Go port selected via GO_PORT" or similar

1161 ↗(On Diff #58948)

s/run time dependency/runtime dependency/ or just "run dependency" on whatever is in GO_PORT.

1231–1232 ↗(On Diff #58948)

Should be in <warning> or <caution> block.

dmgk updated this revision to Diff 58993.Jun 25 2019, 12:49 PM

Address tobik@ feedback.

Harbormaster completed remote builds in B25024: Diff 58993.Jun 25 2019, 12:49 PM
dmgk marked 3 inline comments as done.Jun 25 2019, 12:51 PM
mat added inline comments.Jun 25 2019, 2:59 PM
en_US.ISO8859-1/books/porters-handbook/uses/chapter.xml
1231–1232 ↗(On Diff #58948)

and s/should not/must not/, or s/should not/cannot/.

dmgk updated this revision to Diff 59001.Jun 25 2019, 3:11 PM

should not -> must not

Harbormaster completed remote builds in B25032: Diff 59001.Jun 25 2019, 3:11 PM
dmgk marked an inline comment as done.Jun 25 2019, 3:11 PM
dmgk marked an inline comment as done.Jun 25 2019, 4:20 PM
mat added inline comments.Jun 26 2019, 11:46 AM
en_US.ISO8859-1/books/porters-handbook/uses/chapter.xml
1152 ↗(On Diff #59001)

maybe

on the Go compiler port
1158–1159 ↗(On Diff #59001)

use the <buildtarget> tag for build targets.

1226 ↗(On Diff #59001)

Same here

The Go compiler port to use.
1231–1232 ↗(On Diff #59001)

Bad formatting, there should be a blank space before the opening of <warning>, and then the <para>tage should be on its own lines.

dmgk updated this revision to Diff 59048.Jun 26 2019, 12:29 PM

Update based on mat@ comments.

Harbormaster completed remote builds in B25052: Diff 59048.Jun 26 2019, 12:29 PM
dmgk marked 4 inline comments as done.Jun 26 2019, 12:30 PM
dmgk retitled this revision from The porter's handbook Go chapter: document build/run args and GO_PORT variable to The porter's handbook Go chapter: document no_targets/run args and GO_PORT variable.
mat accepted this revision.Jun 26 2019, 4:24 PM
This revision is now accepted and ready to land.Jun 26 2019, 4:24 PM
Closed by commit rD53205: Porter's Handbook: Document no_targets/run and GO_PORT after ports r505321 (authored by tobik). · Explain WhyJun 29 2019, 3:57 AM
This revision was automatically updated to reflect the committed changes.
tobik added a commit: rD53205: Porter's Handbook: Document no_targets/run and GO_PORT after ports r505321.

Revision Contents
Changeset List

PathSize
head/
en_US.ISO8859-1/
books/
porters-handbook/
uses/
38 lines

Diff 59185

View Options

head/en_US.ISO8859-1/books/porters-handbook/uses/chapter.xml

Loading...