SvnLastRevisionTask fails if Subversion is localized (Spanish)

Sorry, the line from svn status that we want is the last changed revisión in the current dir, so respectively:

Last Changed Rev: 10

Revisión del último cambio: 10

This wasn't as easy as I thought because of the unicode characters (ó) in the output. RegExps seems to not work fine with Unicode characters, after testing with:

/Rev[.*]?/u'

Revisión doesn't match.

After searching a lot, the easiest way is removing that characters before the regexp. If anyone has not a better solution, I'll do this way. Anyway, this should be problematic with more languages.

I don't think we really want to adapt Phing to the large number of languages SVN supports, closing for now.

Hi guys,

I have a fix for the localization problem, since we came across the same problem in Xinc:

SvnLastRevisionTask.php

....

function main()
	{
		$this->setup('info');
		
		/**
		 * run in xml mode, allows us to retrieve workingcopy info in 
		 * a unified xml format, so we dont have to fight with internationalized
		 * versions of svn info output
		 */
		$output = $this->run(array('--xml'));
		try 
		{
			$xml = new SimpleXMLElement($output);
			/**
			 * walk the xml towards the last commit element
			 */
			$commits = $xml->xpath('/info/entry/commit');
			if (count($commits)>0) {
				$commit = $commits[0];
				/**
				 * get the attributes of the commit element
				 */
				$attributes = $commit->attributes();
				$this->project->setProperty($this->getPropertyName(), $attributes->revision);
			} else {
				throw new BuildException("Failed to parse the output of 'svn info'.");
			}
			
		}
		catch (Exception $e)
		{
			throw new BuildException("Failed to parse the output of 'svn info'.");
		}
	}
....

If you agree I can commit the change into the trunk. Dont know if its interesting to apply the patch to older versions?

Arno

(In [335]) Refs #195: switching to parse xml output from svn info (svn info --xml). more reliable than any localized string output.

(In [335]) Refs #195: switching to parse xml output from svn info (svn info --xml). more reliable than any localized string output.