Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

[Bug] dita --install syntax is incorrect #129

Closed
shaneataylor opened this issue Feb 13, 2017 · 1 comment
Closed

[Bug] dita --install syntax is incorrect #129

shaneataylor opened this issue Feb 13, 2017 · 1 comment
Assignees
@infotexture
Labels
bug Something isn't working
Milestone
2.4.4

Comments

@shaneataylor
Copy link
Contributor

shaneataylor commented Feb 13, 2017 •
edited by infotexture
Loading

The 2.4 docs now give the following syntax:

dita-ot-dir/bin/dita --install=plug-in-zip

But this does not work (at least in 2.4.2). You must omit the = sign:

dita-ot-dir/bin/dita --install plug-in-zip
@infotexture
Copy link
Member

Looks like you're right, good catch.

The command-line help that appears when you run dita --help also shows the equals sign for both the --install and --uninstall options, so it looks like the Main.java source code needs to be changed on lines 1268, 1276 & 1278.

@infotexture infotexture added the bug Something isn't working label Feb 13, 2017
@infotexture infotexture added this to the 2.4.4 milestone Feb 13, 2017
infotexture added a commit to infotexture/dita-ot that referenced this issue Feb 18, 2017
@infotexture
Fix dita --install syntax per dita-ot/docs#129
0f8a4d2 
Signed-off-by: Roger Sheen <roger@infotexture.net>
@infotexture infotexture mentioned this issue Feb 18, 2017
Merged
infotexture added a commit that referenced this issue Feb 19, 2017
@infotexture
Remove equals sign from --un/install topics (#129)
cb258f3 
Amends f8913d2.

Signed-off-by: Roger Sheen <roger@infotexture.net>
infotexture added a commit that referenced this issue Feb 20, 2017
@infotexture
Merge branch 'develop' into hotfix/2.4.4
4f6dde1 
* develop:
  Fix path to `ant_sample/` directory
  Build XHTML + CSS along with other Ant samples
  Fix parameters in HTML5 Ant sample
  Fix path to `samples/` in distribution packages
  Remove obsolete Ant samples
  Remove empty relationship table rows
  Re-use note on plug-in subdirectory removal
  Remove equals sign from --un/install topics (#129)
  Clarify that `dita --uninstall` removes subdirectory
  Clarify that `dita --install` can also remove plugins
  Remove the equals sign from `dita --install` syntax
  Use new (GNU) double-hyphen syntax for CLI options
  Fix broken reltable link & remove redundant row
  Clarify `args.logdir` description
  Use latest DITA-OT version (2.4.3) for CI builds
  Mark `args.logdir` as obsolete (Fixes #127)
@infotexture infotexture self-assigned this Feb 24, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@infotexture infotexture

Labels
bug Something isn't working
Projects
None yet
Milestone
2.4.4
Development

No branches or pull requests
2 participants
@infotexture @shaneataylor