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?
