- Add @{pre,post}{,un}exec and deprecate @{,un}exec.
- Update the way @sample works.
- Move all the deprecated entries at the end.
- Add the arguments section.
- Switch the big "New Keywords" varlistentry into sect3s.
- Update the @sample example to catch up with reality.
Details
Details
- Reviewers
brd wblock bcr - Commits
- rD47465: The @sample keyword can now take two arguments.
rD47466: Restructurate the custom keywords section in subsections.
rD47467: Update the @sample example to catch up with reality.
rD47463: Move all the deprecated plist keywords entries at the end of the section. No…
rD47464: Add "arguments" to the list of available directives for keywords files,
rD47462: Add @{pre,post}{,un}exec and deprecate @{,un}exec.
Diff Detail
Diff Detail
- Repository
- rD FreeBSD doc repository - subversion
- Lint
No Lint Coverage - Unit
No Test Coverage - Build Status
Buildable 647 Build 647: arc lint + arc unit
Event Timeline
| en_US.ISO8859-1/books/porters-handbook/plist/chapter.xml | ||
|---|---|---|
| 403–404 | It's not clear what <quote>real</quote> means here. Maybe it should just be existing without the quote? | |
| 404 | s/file/filename/ | |
| 406 | s/real/existing/ | |
| 465–514 | s|installation/deinstallation|installation or deinstallation| | |
| 636 | s|installation/deinstallation|installation or deinstallation| | |
| 671 | <listitem> should be on its own line: <listitem> <para><literal>attributes</literal></para> </listitem> (Also applies to following lines.) | |
| 820 | This sentence is a little confusing. Maybe "If set to true, changes how arguments are handled, splitting %1 and %2 into multiple arguments." | |
| 823 | Break sentence: s/works, as/works. When/ | |
| 824 | Break sentence: s/specified, for/specified. For/ | |
| 860 | I think this can be "Custom Keyword Examples", with the final plural meaning multiple examples and multiple keywords. | |
| 882 | Split sentence: s/things, it/things. It/ | |
| en_US.ISO8859-1/books/porters-handbook/plist/chapter.xml | ||
|---|---|---|
| 404 | This first usage of "actual" should explain what it means. The <quote>actual</quote>, non-sample file... | |
| 819 | Better, but still combines a lot of things into one sentence. Maybe: Set to <literal>true</literal> to enable argument handling. The whole <literal>%@</literal> line is split into numbered arguments, <literal>%1</literal>, <literal>%2</literal>, and so on. For example... | |
| 821 | "if enabled" is not needed, that is already the context for this paragraph. | |
| 822 | <quote> is not needed. here or in the rest of the paragraph. The <literal> shows the content. If the concern is that punctuation might be confused, maybe put the output into <programlisting> elements. | |
| 829 | I don't think the ", respectively" part is needed. | |