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

RE: notification backward compatibilty of the testsuite

From: Bert Huijben <bert_at_qqmail.nl>
Date: Mon, 4 Apr 2011 17:44:46 +0200

> -----Original Message-----
> From: Philip Martin [mailto:philip.martin_at_wandisco.com]
> Sent: maandag 4 april 2011 17:37
> To: dev_at_subversion.apache.org
> Subject: notification backward compatibilty of the testsuite
>
> I've been working on a revert implementation that fixes some correctness
> problems. A side effect of this change is that the notifications
> change. In 1.6 revert only notifies for the root of an added tree, my
> new code notifies for all the added nodes. This causes 3 of the
> regression tests to fail. The 1.6 behaviour seems a bit arbitrary to
> me, and reproducing it with my new code would be difficult.
>
> Do we need any sort of backward compatibility in the testsuite? Is it
> acceptable to simply change the regression tests to expect the new
> notifications? Something like this:

I don't think we promise notifications to stay the same over versions and as
an application developer I prefer a bit more notifications. There are
several other places where we already changed notifications a few times
(merge, tree conflicts, etc. etc.)

+1 from me on just adding the extra notifications.

(I would have a problem if we removed notification calls for specific paths;
but not if they were just filtered in svn's output)

        Bert
Received on 2011-04-04 17:47:05 CEST

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.