Merge lp:~barry/python-distutils-extra/670188-ftbfs into lp:python-distutils-extra

Proposed by Barry Warsaw
Status: Merged
Merged at revision: 223
Proposed branch: lp:~barry/python-distutils-extra/670188-ftbfs
Merge into: lp:python-distutils-extra
Diff against target: 64 lines (+15/-5)
3 files modified
debian/changelog (+8/-0)
setup.py (+1/-1)
test/auto.py (+6/-4)
To merge this branch: bzr merge lp:~barry/python-distutils-extra/670188-ftbfs
Reviewer Review Type Date Requested Status
Registry Administrators Pending
Review via email: mp+40136@code.launchpad.net

Description of the change

Let's try this again, this time against the upstream branch!

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/changelog'
--- debian/changelog 2010-11-04 02:37:33 +0000
+++ debian/changelog 2010-11-04 18:44:43 +0000
@@ -1,3 +1,11 @@
1python-distutils-extra (2.22-4) unstable; urgency=low
2
3 * When the environment has $PYTHONPATH in it, we still need to prepend
4 oldcwd in test/auto.py, so that the DistUtilsExtra package can be
5 found. (LP: #670188)
6
7 -- Barry Warsaw <barry@ubuntu.com> Thu, 04 Nov 2010 14:33:04 -0400
8
1python-distutils-extra (2.22-3) unstable; urgency=low9python-distutils-extra (2.22-3) unstable; urgency=low
210
3 [ Martin Pitt ]11 [ Martin Pitt ]
412
=== modified file 'setup.py'
--- setup.py 2010-09-16 12:06:26 +0000
+++ setup.py 2010-11-04 18:44:43 +0000
@@ -1,7 +1,7 @@
1#!/usr/bin/env python1#!/usr/bin/env python
22
3from setuptools import setup3from setuptools import setup
4import glob, os, sys4import sys
55
6sys.path.insert(0, '.')6sys.path.insert(0, '.')
7from DistUtilsExtra import __version__ as pkgversion7from DistUtilsExtra import __version__ as pkgversion
88
=== modified file 'test/auto.py'
--- test/auto.py 2010-10-01 16:03:27 +0000
+++ test/auto.py 2010-11-04 18:44:43 +0000
@@ -2,7 +2,7 @@
22
3# test DistUtilsExtra.auto3# test DistUtilsExtra.auto
44
5import sys, unittest, shutil, tempfile, os, os.path, subprocess, re5import unittest, shutil, tempfile, os, os.path, subprocess, re
66
7class T(unittest.TestCase):7class T(unittest.TestCase):
8 def setUp(self):8 def setUp(self):
@@ -32,7 +32,7 @@
32 try:32 try:
33 # check that setup.py clean removes everything33 # check that setup.py clean removes everything
34 (o, e, s) = self.setup_py(['clean', '-a'])34 (o, e, s) = self.setup_py(['clean', '-a'])
35 self.assertEqual(s, 0)35 self.assertEqual(s, 0, o+e)
36 cruft = self.diff_snapshot()36 cruft = self.diff_snapshot()
37 self.assertEqual(cruft, '', 'no cruft after cleaning:\n' + cruft)37 self.assertEqual(cruft, '', 'no cruft after cleaning:\n' + cruft)
38 finally:38 finally:
@@ -759,9 +759,11 @@
759 if not self.snapshot:759 if not self.snapshot:
760 self.do_snapshot()760 self.do_snapshot()
761761
762 env = os.environ762 env = os.environ.copy()
763 oldcwd = os.getcwd()763 oldcwd = os.getcwd()
764 if 'PYTHONPATH' not in env:764 if 'PYTHONPATH' in env:
765 env['PYTHONPATH'] = oldcwd + os.pathsep + env['PYTHONPATH']
766 else:
765 env['PYTHONPATH'] = oldcwd767 env['PYTHONPATH'] = oldcwd
766 os.chdir(self.src)768 os.chdir(self.src)
767 s = subprocess.Popen(['/proc/self/exe', 'setup.py'] + args, env=env,769 s = subprocess.Popen(['/proc/self/exe', 'setup.py'] + args, env=env,

Subscribers

People subscribed via source and target branches

to all changes:
to status/vote changes: