Re: [PATCH 1/6] gitopt: a new command-line option parser for git

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Eric Wong <normalperson@xxxxxxxx> wrote:

> I'm striving for backwards compatibility with existing usage.  That
> means as a diff option, -C alone works, as does -C20.  I've made -C 20
> _not_ work because it breaks existing usage (where 20 could be a
> filename, or a tree-ish).  -C=20 would mean something
> else,

I think optional arguments are still confusing.  We could support both
-C (no args) and -C=20 syntax.

> since I wanted to make pickaxe work exactly as it did before:
> -S=var would search for '=var', not 'var'.

Some other options use -x=y syntax so this would be confusing. Pickaxe's
-Stext syntax is a bit strange.  I think -S text or -S=text would be
more logical.

-- 
http://onion.dynserv.net/~timo/
-
: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]