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

Re: [PATCH] Handle special characters in Junit report

From: Bhuvaneswaran A <bhuvan_at_collab.net>
Date: Fri, 04 Dec 2009 19:53:09 +0530

On Fri, 2009-12-04 at 13:41 +0000, Julian Foad wrote:
> Bhuvaneswaran A wrote:
> > Branko, Julian: Thank you for the review comments.
> >
> > Incorporated the above suggestion and committed in r887178. Now that the
> > script is stable, as discussed in this thread [1], shall I nominate this
> > script to 1.6.x branch?
> > [1]
> > http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2410330
>
> Do we need an up-to-date copy of it in branches/1.6.x - in other words,
> do we want the Hudson system to use the script that it finds in the
> "dev/tools/" directory of the branch it is building? I would have
> thought it should just use the copy that's in trunk, even when it is
> building and testing a branch, because the schedule and purpose of
> maintaining subversion-1.6.x is separate from the schedule and purpose
> of maintaining the testing system.

That's right. Right now we use the script in trunk to generate report
for 1.6.x build. The build script is ugly to execute the script from
relative path. This is not an issue right now, as both builds are tied
to same box. But in case, we tie trunk and 1.6.x build to separate box
we might have to co this script from trunk while we build 1.6.x branch,
solely due to the fact that it does not exist in 1.6.x branch.

That's the reason I prefer to backport this script to 1.6.x branch.

-- 
Bhuvaneswaran A    
CollabNet Software P Ltd.  |  www.collab.net

Received on 2009-12-04 15:24:00 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.