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

Re: [PATCH] issue #4527: notify start of exporting external

From: Philip Martin <philip.martin_at_wandisco.com>
Date: Thu, 13 Nov 2014 16:57:07 +0000

Julian Foad <julianfoad_at_btopenworld.com> writes:

> Other than the indentation, Michael's patch looks like the right fix
> in the right place, wouldn't you agree?

Yes, I think it is correct. Extending the patch to fix the in_external
nesting problem would be ideal but we could commit this version.

> Michael, have you tested it by running the full test suite? If there
> is a test that covers this then I would expect that its expected
> output will need to be adjusted, unless it ignores this notification;
> and if there isn't any test that attempts an export with externals,
> then could you add one?

The normal testsuite mechanisms ignore all the output lines affected by
the patch. I suppose one could write a special test that checks the
lines ignored by the usual test mechanism but any such test would have
to work with all the output orderings that export could produce. That
might be non-trivial.

-- 
Philip Martin | Subversion Committer
WANdisco // *Non-Stop Data*
Received on 2014-11-13 17:58:26 CET

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