Opened 13 years ago

Closed 13 years ago

#259 closed defect (fixed)

Option --config-file fails

Reported by: Jari Häkkinen Owned by: Peter Johansson
Priority: major Milestone: 0.6.3
Component: configuration Version: 0.6.2
Keywords: Cc:

Description

Trying to set config file with the below command fails.

svndigest --copyright --config-file=path/to/config

svndigest: invalid option: --config-file=path/to/config
Type `svndigest --help' for usage.

According to the help the option should be used as

--config-file=ARG   configuration file [<ROOT>/.svndigest/config]       

The below command works

svndigest --copyright --config-file path/to/config

Change History (5)

comment:1 Changed 13 years ago by Peter Johansson

Ok, so the question is should we fix the --help output or the comandline parsing?

So the alternatives are:

  1. Allow --config-file=ARG
  2. Allow --config-file ARG and change --help output appropriately.

A third option is to allow both --config-file=ARG and --config-file ARG. What is standard?

comment:2 Changed 13 years ago by Peter Johansson

After looking into the code, it looks like this is an issue also for --root and --target.

comment:3 Changed 13 years ago by Peter Johansson

I tried cp --target-dir=tmp foo and cp --target-dir tmp foo and they resulted in same thing, which is that foo was copied to directory tmp. I guess this implies option 3 in discussion above.

I thiunk we should go for option 3. Options taking argument should work both with --option=arg as well as --option arg.

comment:4 Changed 13 years ago by Peter Johansson

Milestone: 0.6.3
Status: newassigned

comment:5 Changed 13 years ago by Peter Johansson

Resolution: fixed
Status: assignedclosed

In [469]

Note: See TracTickets for help on using tickets.