[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] automatic properties

From: Branko Čibej <brane_at_xbc.nu>
Date: 2003-09-29 01:27:33 CEST

Philip Martin wrote:

>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.
>
>
No it's not. This patch adds two new config file options to set the
default behaviour of "svn add" and "svn import" WRT auto-props. Since
the user can set the config to be on or off, there has to be a way to
override either value on the command line.

-- 
Brane Čibej   <brane_at_xbc.nu>   http://www.xbc.nu/brane/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Sep 29 01:28:05 2003

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.