Details
Details
- Reviewers
- None
- Group Reviewers
portmgr - Commits
- rD53173: Make it clear that using sed(1) to patch files is only to be used when
Diff Detail
Diff Detail
- Lint
No Lint Coverage - Unit
No Test Coverage - Build Status
Buildable 24958 Build 23679: arc lint + arc unit
Event Timeline
en_US.ISO8859-1/books/porters-handbook/slow-porting/chapter.xml | ||
---|---|---|
422 | This seems off:
For me, semicolon is used to separate two non related clauses, which could be separate sentences. Here, the two clauses are related and the second emphasizes the first. |
en_US.ISO8859-1/books/porters-handbook/slow-porting/chapter.xml | ||
---|---|---|
422 | The problem is that with a comma, one of the clauses must be dependent (i.e. a non-complete sentence). Here, both clauses are complete sentences, making it what's called a "comma splice." I fully agree that the semicolon is weird there. Perhaps "You must use patch files instead of sed to replace static content." Or something like that. |