Re: Parse --o in format-patch

On Fri, Jun 28, 2013 at 06:44:40PM +0200, Fredrik Gustafsson wrote:

On Fri, Jun 28, 2013 at 12:31:53PM -0400, Jeff King wrote:

quoted

It's possible to have an "optional" argument by using the
PARSE_OPT_OPTARG flag. However, it is not backwards compatible from the
user's perspective, as they must use the "sticked" form:

That would be a possibility but I don't like breaking backwards
compability.

Yes, I did not say it outright, but I meant "...and that is why we
cannot go that route." :)

my goal is to make:
       git format-patch [-k] [(-o|--output-directory) <dir> | --stdout] [ <since> | <revision range> ]
to be:
       git format-patch [-k] [(-o|--output-directory) [dir] | --stdout] [ <since> | <revision range> ]

that would do:
git format patch -> current dir
git format patch -o -> default dir (for example GIT_DIR/.outgoing/)
git format patch -o <dir> -> user defined <dir>

Ah, that makes much more sense to me.

But I guess I would need a new option instead. Something like
--default-output-dir.

It depends on how the default is specified. Is it hard-coded? Or do you
specify format.outputDirectory? If the latter, I would think you would
want it on all the time when "-o" is not given[1], and no new option is
required. Otherwise, yes, I'd think you would want a new option.

-Peff

[1] format-patch may be considered plumbing, in which case an output
    directory config option might cause problems with scripts that
    expect to run it and find the output in the current directory. I'm
    not sure how big a deal that is.