[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Monotone-devel] automate inventory
From: |
Thomas Keller |
Subject: |
Re: [Monotone-devel] automate inventory |
Date: |
Thu, 13 Sep 2007 12:07:07 +0200 |
User-agent: |
Thunderbird 2.0.0.6 (X11/20070728) |
Stephen Leake schrieb:
> Derek Scherger <address@hidden> writes:
>
>> I vaguely recall some mention of a restricted inventory case where only
>> one side of a rename was listed in the inventory. Is there a test for
>> this?
>
> Yes; see tests/automate_inventory_restricted/__driver__.lua, search
> for the comment:
>
> -- Rename a file from dir_a to dir_b, bookkeep-only; inventory dir_a
>
>
>> Is it still a problem?
>
> It reports the correct results.
Well, now that the full paths are outputted and the node states are
correct, its not that big of a problem anymore. IIRC the problem was
that we depended on the node_ids previously to distinguish between "is
an add", "is a drop", and "is a rename". We still do not output the
complete node stanza for the other (renamed) node (should we?), but I
guess this is more reasonable anyways for implementations than
spuriously putting out nodes which reside outside the original
restriction (which would be hard to parse).
>> BTW, has anyone looked at this branch? I think it's an improvement
>> and fixes a couple of invariant failures. Please see my earlier
>> emails for details.
>
> I just looked thru the log and the code. I gather the main point is
> the function make_restricted_roster. There are no comments saying what
> it does or why it is there (vis-a-vis Richard's recent complaint :).
I personally find it a bit hard to understand whats going on there, but
this may perfectly be my stupidity. I'll have a closer look tonight.
Thomas.
--
only dead fish swim with the stream: http://thomaskeller.biz/blog
Am Anfang war das Wort: http://www.schäuble-muss-weg.de
- Re: [Monotone-devel] automate inventory, (continued)
- Re: [Monotone-devel] automate inventory, Derek Scherger, 2007/09/05
- Re: [Monotone-devel] automate inventory, Stephen Leake, 2007/09/11
- Re: [Monotone-devel] automate inventory, Derek Scherger, 2007/09/12
- Re: [Monotone-devel] automate inventory, Stephen Leake, 2007/09/12
- Re: [Monotone-devel] automate inventory, Thomas Keller, 2007/09/12
- Re: [Monotone-devel] automate inventory, Derek Scherger, 2007/09/12
- Re: [Monotone-devel] automate inventory, Stephen Leake, 2007/09/13
- Re: [Monotone-devel] automate inventory, Thomas Keller, 2007/09/06
- Re: [Monotone-devel] automate inventory, Derek Scherger, 2007/09/12
- Re: [Monotone-devel] automate inventory, Stephen Leake, 2007/09/13
- Re: [Monotone-devel] automate inventory,
Thomas Keller <=
- [Monotone-devel] restriction fixes, Derek Scherger, 2007/09/15
- Re: [Monotone-devel] restriction fixes, Stephen Leake, 2007/09/16
- Re: [Monotone-devel] restriction fixes, Derek Scherger, 2007/09/28
- Re: [Monotone-devel] automate inventory, Derek Scherger, 2007/09/13
- Re: [Monotone-devel] automate inventory, Stephen Leake, 2007/09/13
- Re: [Monotone-devel] automate inventory, Derek Scherger, 2007/09/15
Re: [Monotone-devel] New commands (for mtn, in lua), Nathaniel Smith, 2007/09/04