[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

RE: [PATCH] Add --dry-run flag to "svn update" client command

From: Kamesh Jayachandran <kamesh_at_collab.net>
Date: Mon, 14 Mar 2011 23:38:30 +0530

>> Kamesh Jayachandran wrote on Fri, Mar 11, 2011 at 18:12:21 +0530:
>> > Unless there are no objection I will commit this patch post I
>> > receive a r1075802 regression fix(and testcase) from either Daniel
>> > Becroft or Arwin.
>> Hold on, are you suggesting to commit the two thousand, six hundred and
>> sixty-three line diff that started this thread?

>So what if it's long? I'd be much more worried if this diff was shorter.
>It needs to make changes in lots of places.

>It does a lot of things, including running every update during tests
>with --dry-run, and checking whether the dry-run modified the working
>copy, and making the test fail if it did.
>And it checks if the dry-run output matches the output for normal update.
>If all the tests pass with this, that's fine, isn't it?
>Because how else would you do it? Not test it, just to make the diff shorter?

Thanks a lot stefan for taking closer look at this patch.

You voiced everything that I have to say about.

With regards
Kamesh Jayachandran
Received on 2011-03-14 19:10:42 CET

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.