Re: [Monotone-devel] sync defaults

[Timothy Brownawell]

On Sun, 2008-03-16 at 19:46 -0400, Stephen Leake wrote:
> I'm proposing the following patch to cmd_netsync.cc:
[...]
> The first change (passing 'true' to find_key_if_needed) requires a key
> (needed for writing to the server database); that means it will look
> in _MTN/options if present. That was the behavior in mtn 0.39, and is
> the same as 'mtn push'. 'mtn pull' sets this to false, since it can
> work anonymously.
> 
> The second change writes the workspace options if --set-default is
> specified and a workspace is present. That allows fixing an error
> reported by sync when there are multiple keys in the keydir.
> 
> Hmm. 'push' should also write the 
> 
> Most commands write the workspace options even without --set-default,
> but it seems appropriate to require that for sync.
> 
> The current help for '--set-default' says:
> 
>   --set-default        use the current arguments as the future default
> 
> I initially interpreted that to mean "arguments and options", and
> wondered why it wasn't writing 'key' to _MTN/options. This change
> makes it mean that.
> 
> Any objections?

I like it, but the first half will conflict with some changes I'm in the
middle of. Do we have a test that checks this behavior?