[Top][All Lists]

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

Re: [Monotone-devel] sync defaults

From: Stephen Leake
Subject: Re: [Monotone-devel] sync defaults
Date: Tue, 18 Mar 2008 05:30:58 -0400
User-agent: Gnus/5.1008 (Gnus v5.10.8) Emacs/22.1 (windows-nt)

Timothy Brownawell <address@hidden> writes:

> On Sun, 2008-03-16 at 19:46 -0400, Stephen Leake wrote:
>> I'm proposing the following patch to
> [...]
>> 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?

I've committed the above changes, and a test 'sync_use_ws_options'.

When do you need to pass 'false' to find_key_if_needed?

-- Stephe

reply via email to

[Prev in Thread] Current Thread [Next in Thread]