- 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
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
| en_US.ISO8859-1/books/porters-handbook/plist/chapter.xml | ||
|---|---|---|
| 403 ↗ | (On Diff #8999) | It's not clear what <quote>real</quote> means here. Maybe it should just be existing without the quote? |
| 404 ↗ | (On Diff #8999) | s/file/filename/ |
| 406 ↗ | (On Diff #8999) | s/real/existing/ |
| 464 ↗ | (On Diff #8999) | s|installation/deinstallation|installation or deinstallation| |
| 635 ↗ | (On Diff #8999) | s|installation/deinstallation|installation or deinstallation| |
| 670 ↗ | (On Diff #8999) | <listitem> should be on its own line: <listitem> <para><literal>attributes</literal></para> </listitem> (Also applies to following lines.) |
| 803 ↗ | (On Diff #8999) | This sentence is a little confusing. Maybe "If set to true, changes how arguments are handled, splitting %1 and %2 into multiple arguments." |
| 806 ↗ | (On Diff #8999) | Break sentence: s/works, as/works. When/ |
| 807 ↗ | (On Diff #8999) | Break sentence: s/specified, for/specified. For/ |
| 832 ↗ | (On Diff #8999) | I think this can be "Custom Keyword Examples", with the final plural meaning multiple examples and multiple keywords. |
| 854 ↗ | (On Diff #8999) | Split sentence: s/things, it/things. It/ |
| en_US.ISO8859-1/books/porters-handbook/plist/chapter.xml | ||
|---|---|---|
| 404 ↗ | (On Diff #9003) | This first usage of "actual" should explain what it means. The <quote>actual</quote>, non-sample file... |
| 819 ↗ | (On Diff #9003) | 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 ↗ | (On Diff #9003) | "if enabled" is not needed, that is already the context for this paragraph. |
| 822 ↗ | (On Diff #9003) | <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 ↗ | (On Diff #9003) | I don't think the ", respectively" part is needed. |