Using our own utils.system() for the setup.py cal

CHANGES.rst

@@ -4,7 +4,10 @@ Changelog for zest.releaser
 3.52 (unreleased)
 -----------------
 
-- Nothing changed yet.
+- Fixed "longtest" command when run with a python without setuptools
+  installed. Similar fix to the one in 3.51.
+  See https://github.com/zestsoftware/zest.releaser/issues/57
+  [reinout]
 
 
 3.51 (2014-07-17)

zest/releaser/longtest.py

@@ -1,12 +1,13 @@
 """Do the checks and tasks that have to happen before doing a release.
 """
 import logging
+import os
 import sys
-import webbrowser
 import tempfile
-import os
+import webbrowser
 
 from zest.releaser import utils
+from zest.releaser.utils import system
 
 logger = logging.getLogger(__name__)
 
@@ -15,11 +16,12 @@ def show_longdesc():
     filename1 = tempfile.mktemp()
     filename2 = tempfile.mktemp()
     filename2 = filename2 + '.html'
-    error = os.system(utils.setup_py('--long-description > %s' %
-                                     filename1))
-    if error:
-        logging.error('Error generating long description.')
-        sys.exit()
+    # Note: for the setup.py call we use system() from our utils module. This
+    # makes sure the python path is set up right.
+    # For the other calls we use os.system(), because that returns an error
+    # code which we need.
+    system(utils.setup_py('--long-description > %s' %
+                          filename1))
     error = os.system('rst2html.py %s > %s' % (filename1, filename2))
     if error:
         # On Linux it needs to be 'rst2html', without the '.py'