Re: [PATCH] automatic properties

Branko ÄŒibej <brane@xbc.nu> writes:

>>>@@ -151,7 +155,8 @@
>>> "Put files and directories under revision control, scheduling\n"
>>> "them for addition to repository. They will be added in next commit.\n"
>>> "usage: add PATH...\n",
>>>- {svn_cl__targets_opt, 'N', 'q', svn_cl__config_dir_opt} },
>>>+ {svn_cl__targets_opt, 'N', 'q', svn_cl__config_dir_opt,
>>>+ svn_cl__enable_autoprops_opt, svn_cl__disable_autoprops_opt} },
>>
>>The command takes both options? Isn't one the default? We don't
>>generally have both enable and disable options on the same command.
>>
> In this case you sort of need them, because you set the default in the
> config file. You want to be able to override the default.

Really? As far as I can tell

  $ svn add foo.c

is equivalent to one of

  $ svn add --auto-props foo.c
  $ svn add --no-auto-props foo.c

and so either --auto-props or --no-auto-props is redundant. It's just
like -R/-N, since add is recursive by default it does not accept -R.

-- 
Philip Martin
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org