On Thu, Apr 15, 2010 at 12:09 AM, Adam Barth <aba...@webkit.org> wrote:
> On Wed, Apr 14, 2010 at 11:47 PM, Darin Fisher <da...@chromium.org> wrote: > > On Wed, Apr 14, 2010 at 5:06 PM, Adam Barth <aba...@webkit.org> wrote: > >> On Wed, Apr 14, 2010 at 4:52 PM, Geoffrey Garen <gga...@apple.com> > wrote: > >> >> I'd rather keep the current behavior and have it operate on the > entire > >> >> repository. I think it will be too easy to make mistakes and leave > out > >> >> large portions of a patch if it only operates upon the current > working > >> >> directory. > >> > > >> > That's a fixable mistake, though, if/when it happens. > >> > > >> > Taking huge amounts of time unnecessarily diff'ing the LayoutTests > >> > directory is a fixed cost that is unavoidable and unfixable. > >> > >> How long is a huge amount of time? Here are some stats from my > >> laptop. In the first run, I suspect webkit was not in my disk cache > >> (since I recently compiled another checkout): > >> > >> ~/svn/webkit3$ time svn diff > >> real 0m23.437s > >> user 0m1.936s > >> sys 0m7.531s > >> > >> In the second run, I suspect webkit was in my disk cache: > >> > >> ~/svn/webkit3$ time svn diff > >> real 0m8.929s > >> user 0m2.019s > >> sys 0m6.012s > >> > >> ~/svn/webkit3$ svn --version > >> svn, version 1.6.3 (r38063) > >> > >> If the diff is taking much longer than these times, you probably have > >> a borked SVN working copy. We've seen really slow working copies when > >> the working copy was created with SVN 1.5 and then upgraded to 1.6. > >> If your SVN is slow, you might consider blowing away your working copy > >> and making a new one with SVN 1.6. > > > > This may not seem like a huge difference, but 23 seconds is enough to > > encourage me to "cd WebCore" before running svn-create-patch or > > prepare-ChangeLog. > > Does that mean you're not writing tests for your changes? ;) > > I don't have a good sense for what SVN workflows are like. I've > switched over to SVN for a bit to try to understand how it works, but > I haven't learned all the tricks yet. > > Adam > It is common for me to re-create WebCore/ChangeLog as I modify my solution, and I've also worked on a fair number of patches that are just porting / getting existing tests to pass :-/ My workflow with svn is to work from a subdirectory if I can to minimize the cost of svn operations. If I need to work from the topmost directory, then I typically specify the directories I'm interested in on the svn command line (e.g., svn commit WebCore LayoutTests). That way the commands run faster. -Darin
_______________________________________________ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev