| msg110886 - (view) |
Author: Hirokazu Yamamoto (ocean-city) *  |
Date: 2010-07-20 11:07 |
| I got an error while testing py3k on VC6. Here is a patch. ====================================================================== ERROR: test_remove_visual_c_ref (distutils.tests.test_msvc9compiler.msvc9compile rTestCase) ---------------------------------------------------------------------- Traceback (most recent call last): File "e:\python-dev\py3k\lib\distutils\tests\test_msvc9compiler.py", line 112, in test_remove_visual_c_ref from distutils.msvc9compiler import MSVCCompiler File "e:\python-dev\py3k\lib\distutils\msvc9compiler.py", line 291, in <module > raise DistutilsPlatformError("VC %0.1f is not supported by this module" % VE RSION) distutils.errors.DistutilsPlatformError: VC 6.0 is not supported by this module ---------------------------------------------------------------------- Ran 170 tests in 49.070s FAILED (errors=1, skipped=4) Traceback (most recent call last): File "test_distutils.py", line 18, in test_main() File "test_distutils.py", line 13, in test_main test.support.run_unittest(distutils.tests.test_suite()) File "e:\python-dev\py3k\lib\test\support.py", line 1054, in run_unittest _run_suite(suite) File "e:\python-dev\py3k\lib\test\support.py", line 1037, in _run_suite raise TestFailed(err) test.support.TestFailed: Traceback (most recent call last): File "e:\python-dev\py3k\lib\distutils\tests\test_msvc9compiler.py", line 112, in test_remove_visual_c_ref from distutils.msvc9compiler import MSVCCompiler File "e:\python-dev\py3k\lib\distutils\msvc9compiler.py", line 291, in <module > raise DistutilsPlatformError("VC %0.1f is not supported by this module" % VE RSION) distutils.errors.DistutilsPlatformError: VC 6.0 is not supported by this module |
|
|
| msg112244 - (view) |
Author: Éric Araujo (eric.araujo) * |
Date: 2010-08-01 01:05 |
| I don’t know the details of MSVC versions, but the error message seems to mean that this version is not supported, so there is nothing to fix. |
|
|
| msg112265 - (view) |
Author: Hirokazu Yamamoto (ocean-city) * |
Date: 2010-08-01 05:01 |
| Other test functions already have such skips. So I think another skip is acceptable here. But maybe it's better to skip entire msvc9compilerTestCase class (by class decorator). |
|
|
| msg116242 - (view) |
Author: Hirokazu Yamamoto (ocean-city) * |
Date: 2010-09-12 22:57 |
| Fixed in r84753(py3k). |
|
|
| msg116249 - (view) |
Author: Éric Araujo (eric.araujo) * |
Date: 2010-09-12 23:21 |
| Wouldn’t it be more accurate to skip the test instead of returning None (falsely indicating success)? |
|
|
| msg116250 - (view) |
Author: Éric Araujo (eric.araujo) * |
Date: 2010-09-12 23:21 |
| (Adding distutils2 component so that forward-porting the patch is not forgotten.) |
|
|
| msg116254 - (view) |
Author: Hirokazu Yamamoto (ocean-city) * |
Date: 2010-09-12 23:54 |
| How about attached patch? I confirmed this both on MSVC8 and MSVC6. |
|
|
| msg116259 - (view) |
Author: Éric Araujo (eric.araujo) * |
Date: 2010-09-13 00:26 |
| Looks excellent, just import the function at the module scope instead of function scope (I’ve heard about import deadlock issues, and performance is not an issue in tests, so there is no reason to import in the function). Please backport to 3.1 and 2.7 if the bug applies to them. |
|
|
| msg116277 - (view) |
Author: Hirokazu Yamamoto (ocean-city) * |
Date: 2010-09-13 05:43 |
| > I’ve heard about import deadlock issues, and performance is not an > issue in tests, so there is no reason to import in the function distutils.msvc9compiler imports winreg internally, and it only exists on windows, so importing distutils.msvc9..... compiler may raise ImportError on Linux. So I imported only when sys.platform == "win32". Committed in r84760(py3k). And will be merged into other branches. |
|
|
| msg120991 - (view) |
Author: Éric Araujo (eric.araujo) * |
Date: 2010-11-12 00:18 |
| Re-opening to remind myself about the forward-port. |
|
|
| msg129901 - (view) |
Author: Martin v. Löwis (loewis) * |
Date: 2011-03-02 15:09 |
| Éric, what forward porting exactly needs to be done? |
|
|
| msg131383 - (view) |
Author: Éric Araujo (eric.araujo) * |
Date: 2011-03-19 02:43 |
| Martin: Just applying the same changes to distutils2. |
|
|
| msg227766 - (view) |
Author: Francis MB (francismb) * |
Date: 2014-09-28 16:50 |
| Hi Éric, are the changes to distutils2 applied? could the issue be closed (has resolution:fixed) or is something to be done? |
|
|
| msg227767 - (view) |
Author: Mark Lawrence (BreamoreBoy) * |
Date: 2014-09-28 16:54 |
| Distutils2 is dead. |
|
|
| msg227770 - (view) |
Author: Francis MB (francismb) * |
Date: 2014-09-28 17:19 |
| > Distutils2 is dead. I wasn't aware of that and I'm sorry for that. In that case that issue can IMHO be closed. |
|
|