I'd be happy to make the change to support the root issue, although I
don't like the idea of a bogus name. I can make it an argument to the
sensor plus have a benign default like "SVNRoot" or something similar.

Todd
 

-----Original Message-----
From: Discussion list for hackystat developers
[mailto:[EMAIL PROTECTED] On Behalf Of (Cedric) Qin ZHANG
Sent: Thursday, August 11, 2005 7:33 PM
To: [email protected]
Subject: Re: [HACKYSTAT-DEV-L] still workspace root problem

The SVN sensor has been working on Ikayzo for sometime. Should we
include it as an official Hackystat sensor?

Both CVS and SVN sensor does the same thing:
   (1) extracting different revisions
   (2) comparing two revisions

The code for (1) is different, but for (2) is the same. If we decide to
include it in Hackystat release, is it a good idea to put the SVN code
in hackyVCS module?

Thanks.

Cedric

Philip Johnson wrote:
> --On Wednesday, August 10, 2005 6:55 PM -1000 "(Cedric) Qin ZHANG" 
> <[EMAIL PROTECTED]> wrote:
> 
>> Should I change the SVN sensor to prepend some bogus names before the

>> file name, so that it looks like: fileName = "SVNBogusName/A.java"?
> 
> 
> Files without any directory path have always been a problem for the 
> Workspace mechanism.
> 
> Rather than 'SVNBogusName', how about something a little more elegant,

> such as 'ikayzo/svn'.
> 
> Also, rather than create a 'hacked' version of the SVN sensor, can we 
> have an argument to the sensor to prepend such prefixes?
> 
> Cheers,
> Philip

Reply via email to