Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update dita command example #158

Closed
stefan-jung opened this issue Oct 11, 2017 · 1 comment
Closed

Update dita command example #158

stefan-jung opened this issue Oct 11, 2017 · 1 comment
Assignees
Labels
bug Something isn't working
Milestone

Comments

@stefan-jung
Copy link
Contributor

This snippet from /docsrc/samples/ant_sample/dita-cmd.xml should use the new propertyfile notation as explained on Building output using the dita command.

<macrodef name="dita-cmd">
  <attribute name="input"/>
  <attribute name="format"/>
  <attribute name="propertyfile"/>
  <sequential>
    <exec executable="${dita.dir}/bin/dita">
      <arg line="-i @{input} -f @{format} -propertyfile @{propertyfile}"/>
    </exec>
  </sequential>
</macrodef>

should be changed to

<macrodef name="dita-cmd">
  <attribute name="input"/>
  <attribute name="format"/>
  <attribute name="propertyfile"/>
  <sequential>
    <exec executable="${dita.dir}/bin/dita">
      <arg line="-i @{input} -f @{format} --propertyfile=@{propertyfile}"/>
    </exec>
  </sequential>
</macrodef>
@infotexture
Copy link
Member

@xephon2 Thanks, good catch. 👍

We try to use the GNU double-hyphen syntax for CLI options throughout the docs, so I'll also switch to --input and --format here.

@infotexture infotexture self-assigned this Oct 11, 2017
@infotexture infotexture added the bug Something isn't working label Oct 11, 2017
@infotexture infotexture added this to the 3.0 milestone Oct 11, 2017
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants