On Wed, Mar 16, 2011 at 12:15:05PM +0530, Arwin Arni wrote:
> On Tuesday 15 March 2011 08:34 PM, Julian Foad wrote:
> >I see that '--parents' and '--set-depth' are not allowed in dry-run
> >mode. What is the reason behind that? Is it because they seem to be
> >difficult to implement, or you think they are not needed, or you intend
> >to implement them in a follow-up patch, or something else?
> >
>
> Well, the reason is that both --parents and --set-depth make
> permanent changes to the WC which will not be reported in the output
> at all.. If the user is passing these parameters, he has a fair idea
> of what these "invisible" changes are, and so shouldn't be adding
> them to a dry-run command. (Maybe we could just make them FALSE
> during a dry-run update so it doesn't throw those errors that I
> coded. The user will be oblivious to this, as the output isn't going
> to change in any way)
Hmmm... shouldn't we print something for these changes?
Received on 2011-03-16 11:36:40 CET