got some global version done, SVN 1.3.x or later now required

--HG--
extra : rebase_source : def9ab923ee6455791c92334ee79c09d9164c43e

setuptools/command/sdist.py

@@ -63,7 +63,7 @@ def _default_revctrl(dirname=''):
 
 def externals_finder(dirname, filename):
     """Find any 'svn:externals' directories"""
-    for name in SVNEnteries.load(dirname).get_external_dirs(filename):
+    for name in svn_utils.parse_externals(dirname):
         yield joinpath(dirname, name)
 
 

setuptools/svn_utils.py

@@ -94,7 +94,7 @@ def _get_entry_schedule(entry):
 #-R without --xml parses a bit funny
 def parse_externals(path):
     try:
-        _, lines = _run_command(['svn',
+        code, lines = _run_command(['svn',
                                  'propget', 'svn:externals', path])
 
         if code:

setuptools/tests/test_svn.py

@@ -91,11 +91,19 @@ class TestSvn_1_7(unittest.TestCase):
         expected = set([
             os.path.join('.', 'readme.txt'),
             os.path.join('.', 'other'),
-            os.path.join('.', 'other', 'test.py'),
             ])
         self.assertEqual(set(x for x in sdist.entries_finder('.', '')),
                          expected)
 
+    def test_external_iterator(self):
+        expected = set([
+            os.path.join('.', 'third_party'),
+            os.path.join('.', 'third_party2'),
+            os.path.join('.', 'third_party3'),
+            ])
+        self.assertEqual(set(x for x in sdist.externals_finder('.', '')),
+                         expected)
+
 def test_suite():
     return unittest.defaultTestLoader.loadTestsFromName(__name__)