ant-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Rory I. Sinclair" <r...@levelfour.com>
Subject Tiny patch to CvsTagDiff.java
Date Mon, 07 Jul 2003 11:03:44 GMT
Hi

	Forgive a wet-behind-the-ears newbie here, this is my first contrib.

	I have made a tiny patch to CvsTagDiff.java to support multiple packages.
CVS itself happily accepts a space-separated list of packages/files to
perform the rdiff command upon, but the Ant task (at least in 1.5.3) fails
with a StringIndexOutOfBounds exception.

	The patched file is attached, and I have separately emailed the original
authors.  The changes are minimal.  Basically, instead of using the string
length of the module (since there can now be many) to remove the 'File
ModuleName/' part at the start of each diff line (when parsing the rdiff
output), it just searches for the first index of '/', and substrings
everything from one character after that index.  I haven't tested this in a
Unix environment, so I don't know if the same slash is used, perhaps this
could be looked at by another?


		// get rid of 'File module/"
                line = line.substring(line.indexOf("/")+1);


	Cheers

		Rory Sinclair


This message is sent in confidence for the addressee only. It may 
contain legally privileged information. The contents are not to be 
disclosed shared or copied to anyone other than the addressee. 
Unauthorised recipients are requested to preserve this confidentiality 
and to advise the sender immediately of any error in transmission.
This mail has been scanned by DKAV anti-virus software.


Mime
View raw message