Opened 11 years ago

Closed 11 years ago

#448 closed task (fixed)

Add a svndigest --ignore-config option

Reported by: Jari Häkkinen Owned by: Jari Häkkinen
Priority: major Milestone: svndigest 0.8
Component: configuration Version: trunk
Keywords: Cc:

Description (last modified by Peter Johansson)

Running svndigest -g in a directory where .svndigest/config exists will output information read from the config file. That is well but if one wants to see the default configuration one either has to remove the config file or cd to another directory before running svndigest -g. Adding the possibility to ignore the configuration file would be nice.

Relates to ticket:261 and ticket:454

Change History (8)

comment:1 Changed 11 years ago by Peter Johansson

First, only for completeness, there is a third way:

touch empty_dummy.config
svndigest -g --config-file=empty_dummy.config

If possible I'd prefer to avoid adding an extra cmd switch for this corner case. I think it should be possible to embed this feature in the --config-file switch. We could, e.g., have a special interpretation of `--config-file=no'. As long as it is documented I think that is clear enough, and the potential collision that users actually have a config file named 'no' seems very unlikely.

comment:2 Changed 11 years ago by Peter Johansson

Description: modified (diff)

ticket:454 was marked as related

comment:3 Changed 11 years ago by Peter Johansson

Owner: changed from Peter Johansson to Jari Häkkinen

Ticket #454 was fixed, i.e., it now works with --config-file=/dev/null

Is that sufficient, so we can close this ticket, or?

comment:4 Changed 11 years ago by Jari Häkkinen

Should we document the use case that --config-file=/dev/null can be used to ignore reading files? It is not obvious.

comment:5 Changed 11 years ago by Peter Johansson

Yeah it's worthwhile a sentence or two.

Also, I think we should add a notice in the verbose output that the config file is loaded. Not that it matters for the generate-config option as it rarely used with --verbose, but it wouldn't hurt.

comment:6 Changed 11 years ago by Peter Johansson

(In [1102]) refs #448. adding notice in verbose mode that config file is read.

comment:7 Changed 11 years ago by Peter Johansson

(In [1103]) mention use of --config-file=/dev/null in manual. refs #448

comment:8 Changed 11 years ago by Peter Johansson

Milestone: svndigest 0.x+svndigest 0.8
Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.