[daisy] [JIRA] Commented: (DSY-643) backup tool: NPE with missing parameters -l

[Karel Vervaeke (JIRA)]

    [ http://issues.cocoondev.org//browse/DSY-643?page=comments#action_13631 ] 

Karel Vervaeke commented on DSY-643:
------------------------------------

When running without arguments commons-cli still does not give a properly formatted error message:

"Missing required options: l[-R Rehash files from an existing backup, -r Restore an existing backup, -h Show this message, -b Create a new backup, -v Print version info]"

There are two missing options:  -l and [-R, -r, -b, -v, -h] (the latter being an option group)

The commons-cli error message lists both missing options (-l and the option group), but there are two things wrong with the error message:
  1. the -l option does not have a fulltext explanation
  2. The two missing options do not have a separator

Both problems should be fixed in commons-cli, but that project has not seen a lot of activity recently.  Will check with Bruno on how to solve this.

For now, I will apply DSY-643-2.patch as it already is an improvement

> backup tool: NPE with missing parameters -l
> -------------------------------------------
>
>          Key: DSY-643
>          URL: http://issues.cocoondev.org//browse/DSY-643
>      Project: Daisy
>         Type: Bug
>   Components: Backup
>     Versions: 2.3
>     Reporter: Andreas Deininger
>     Priority: Minor
>  Attachments: DSY-643-2.patch, DSY-643.diff
>
> How to reproduce: Invoke the backup tool without the -l parameter given:
> C:\>daisy-backup-tool.bat  -b -d e:\backuplocation
> java.lang.NullPointerException
>         at java.io.File.<init>(Unknown Source)
>         at org.outerj.daisy.backupTool.BackupTool.main(BackupTool.java:69)

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.cocoondev.org//secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira