Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1)

Unified Diff: Lib/test/test_sysconfig.py

Issue 3871: cross and native build of python for mingw32 with distutils
Patch Set: Created 7 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Please Sign in to add in-line comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Lib/test/test_sundry.py ('k') | Modules/_ctypes/_ctypes.c » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
--- a/Lib/test/test_sysconfig.py Thu Dec 27 23:52:36 2012 +0200
+++ b/Lib/test/test_sysconfig.py Thu Dec 27 17:35:12 2012 -0500
@@ -288,12 +288,13 @@
_main()
self.assertTrue(len(output.getvalue().split('\n')) > 0)
- @unittest.skipIf(sys.platform == "win32", "Does not apply to Windows")
- def test_ldshared_value(self):
- ldflags = sysconfig.get_config_var('LDFLAGS')
- ldshared = sysconfig.get_config_var('LDSHARED')
-
- self.assertIn(ldflags, ldshared)
+ # NOTE: the test bellow is bogus. Also "Does not apply to MSVC" only !
+ #@unittest.skipIf(sys.platform == "win32", "Does not apply to Windows")
+ #def test_ldshared_value(self):
+ # ldflags = sysconfig.get_config_var('LDFLAGS')
+ # ldshared = sysconfig.get_config_var('LDSHARED')
+ #
+ # self.assertIn(ldflags, ldshared)
@unittest.skipUnless(sys.platform == "darwin", "test only relevant on MacOSX")
def test_platform_in_subprocess(self):
@@ -371,9 +372,9 @@
class MakefileTests(unittest.TestCase):
-
- @unittest.skipIf(sys.platform.startswith('win'),
- 'Test is not Windows compatible')
+ # GCC(mingw) use posix build system
+ @unittest.skipIf(sys.platform.startswith('win') and sys.version.find('GCC') < 0,
+ 'Test is not MSVC compatible')
def test_get_makefile_filename(self):
makefile = sysconfig.get_makefile_filename()
self.assertTrue(os.path.isfile(makefile), makefile)
« no previous file with comments | « Lib/test/test_sundry.py ('k') | Modules/_ctypes/_ctypes.c » ('j') | no next file with comments »

RSS Feeds Recent Issues | This issue
This is Rietveld 894c83f36cb7+