Use svn info --xml instead of svn info to get svn revisions (#11466)

- `svn info` prints different results depending on the system locale
  - in particular, Japanese output doesn't contain "Revision:"

- Change Spack code to use XML output instead of using the human output

lib/spack/spack/fetch_strategy.py

@@ -26,6 +26,7 @@
 import re
 import shutil
 import copy
+import xml.etree.ElementTree
 from functools import wraps
 from six import string_types, with_metaclass
 
@@ -771,13 +772,9 @@ def source_id(self):
         return self.revision
 
     def get_source_id(self):
-        output = self.svn('info', self.url, output=str)
+        output = self.svn('info', '--xml', self.url, output=str)
-        if not output:
+        info = xml.etree.ElementTree.fromstring(output)
-            return None
+        return info.find('entry/commit').get('revision')
-        lines = output.split('\n')
-        for line in lines:
-            if line.startswith('Revision:'):
-                return line.split()[-1]
 
     @_needs_stage
     def fetch(self):

lib/spack/spack/test/conftest.py

@@ -9,7 +9,7 @@
 import os
 import os.path
 import shutil
-import re
+import xml.etree.ElementTree
 
 import ordereddict_backport
 import py
@@ -725,12 +725,9 @@ def mock_svn_repository(tmpdir_factory):
     }
 
     def get_rev():
-        output = svn('info', output=str)
+        output = svn('info', '--xml', output=str)
-        assert "Revision" in output
+        info = xml.etree.ElementTree.fromstring(output)
-        for line in output.split('\n'):
+        return info.find('entry/commit').get('revision')
-            match = re.match(r'Revision: (\d+)', line)
-            if match:
-                return match.group(1)
 
     t = Bunch(checks=checks, url=url, hash=get_rev, path=str(repodir))
     yield t