[PATCH 1 of 1] test-subrepo-svn.t: adapt for Windows

Mads Kiilerich mads at kiilerich.com
Thu Jun 21 08:12:37 CDT 2012


On 21/06/12 12:05, Eduard-Cristian Stefan wrote:
> On Thu, Jun 21, 2012 at 11:33 AM, Mads Kiilerich <mads at kiilerich.com> wrote:
>> It worked fine with 1.7 on linux ... but fails with this patch. The 
>> reason must be something else. 
> I had to update the grep expression to catch the svn 1.7.5 extra line
> on windows.

Please make sure the patch description in reasonable details summarize 
the changes and explain why they are done. That makes it much easier to 
review and understand when someone in the future goes back in history 
and try to understand the change.

It is however not so relevant to list in the _commit_ description what 
changed between the two last drafts.

The path globs and handling of platform specific error codes might be so 
obvious that they don't need mentioning. The change of the name of the 
'forget' file should however be mentioned and explained briefly.

It seems very strange that the python expression for escapedwd is needed 
on some platforms but not on windows. I would expect the commit 
description to explain that. (And escapedwd should probably just be 
created 'correctly' instead of being used in different ways.)

We do for some reason use 'egrep' instead of 'grep -E' in the rest of 
the tests. -E is probably fine on all platforms too, but I would just 
use a simple
grep -v 's[/\]externals[/\]other'

/Mads



More information about the Mercurial-devel mailing list