[issue13170] distutils2 test failures
Roundup Robot devn...@psf.upfronthosting.co.za added the comment: New changeset c1b1b537196d by Éric Araujo in branch 'default': Remove usage of stdlib shutil. http://hg.python.org/distutils2/rev/c1b1b537196d -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue13170 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue13170] distutils2 test failures
Roundup Robot devn...@psf.upfronthosting.co.za added the comment: New changeset 5b096fc6e65d by Éric Araujo in branch 'default': Fix import in install_data (#13170). Thanks to David Barnett. http://hg.python.org/distutils2/rev/5b096fc6e65d New changeset 2d469ccfe30e by Éric Araujo in branch 'python3': Merge fixes for #13170 and #12386 and other misc. changes from default http://hg.python.org/distutils2/rev/2d469ccfe30e -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue13170 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue13170] distutils2 test failures
Éric Araujo mer...@netwok.org added the comment: All three issues fixed, thanks! I can’t push today but I will as soon as possible, probably Monday. I will open another report for the change I reverted in config; it was done on purpose by another developer to fix a bug with distutils2’s own setup script, so there is a real bug to fix here. -- resolution: - fixed stage: - committed/rejected status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue13170 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue13170] distutils2 test failures
Éric Araujo mer...@netwok.org added the comment: When I do import shutil inside _backport/shutil.py You shouldn’t do that! Our backported module is fully stand-alone. I have fixed the bug with this change in install_data.py: -from shutil import Error +from distutils2._backport.shutil import Error I think I’m going to update our whole codebase to only use the backported shutil, never the stdlib one, for clarity (i.e. to avoid similar bugs in the future). -- assignee: tarek - eric.araujo ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue13170 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue13170] distutils2 test failures
Éric Araujo mer...@netwok.org added the comment: The remaining test (test_command_install_data.InstallDataTestCase.test_simple_run) was broken in r1152. This looks like a local revision number, which has no meaning outside of one specific repository. What is the changeset identifier? (see http://hgbook.red-bean.com/read/a-tour-of-mercurial-the-basics.html#id345536) Thanks for the diagnosis! Code calling functions from the backported shutil should obviously catch exceptions from the backported shutil. I’ll review all modules using the backport. FYI, some distutils2 modules use shutil from the stdlib, for functions that are already here in 2.4; not sure if that is good or bad. Ideally, I think distutils2/_backport/shutil.py should do from shutil import Error instead of defining its own Error class Oh, no, our backported module is strictly independent from the stdlib one. importing shutil from the stdlib is broken in that file besides. I don’t understand this. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue13170 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue13170] distutils2 test failures
David Barnett davidbarne...@gmail.com added the comment: The remaining test (test_command_install_data.InstallDataTestCase.test_simple_run) was broken in r1152. This looks like a local revision number, which has no meaning outside of one specific repository. What is the changeset identifier? (see http://hgbook.red-bean.com/read/a-tour-of-mercurial-the-basics.html#id345536) Oh, sorry, it's d1d251292ee7. (I got the short rev from the hg.python.org interface, so I thought it would be authoritative enough, but I guess it is ambiguous) importing shutil from the stdlib is broken in that file besides. I don’t understand this. When I do import shutil inside _backport/shutil.py, it seems to be importing itself (??) if I don't explicitly enable absolute imports (in python 2.7). If I delete the class Error definition and instead add from shutil import Error, I get an ImportError. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue13170 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue13170] distutils2 test failures
David Barnett davidbarne...@gmail.com added the comment: The remaining test (test_command_install_data.InstallDataTestCase.test_simple_run) was broken in r1152. What's happening is that the type of exception being raised was changed and it's getting through the try/except block in install_data.run(). Instead of calling shutil.copyfile, the code in 1152 is calling distutils2._backport.shutil.copyfile, which instead of raising a shutil.Error instance raises a distutils2._backport.shutil.Error instance. Ideally, I think distutils2/_backport/shutil.py should do from shutil import Error instead of defining its own Error class, but I'm not sure if that's kosher for the way backports are supposed to work, and importing shutil from the stdlib is broken in that file besides. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue13170 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue13170] distutils2 test failures
Roundup Robot devn...@psf.upfronthosting.co.za added the comment: New changeset cda119958db3 by Éric Araujo in branch 'default': Kludge around shlex not supporting unicode in 2.x (#13170). http://hg.python.org/distutils2/rev/cda119958db3 -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue13170 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue13170] distutils2 test failures
Roundup Robot devn...@psf.upfronthosting.co.za added the comment: New changeset 5d858149df06 by Éric Araujo in branch 'default': Synchronize config with packaging (fixes #13170) http://hg.python.org/distutils2/rev/5d858149df06 -- nosy: +python-dev ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue13170 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue13170] distutils2 test failures
David Barnett davidbarne...@gmail.com added the comment: I looked into the ConfigTestCase.test_config failure, and it looks like it just needs the righthand edited to match the left. package_data lines are supposed to allow comma-separated lists of paths on the righthand. -- nosy: +mu_mind ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue13170 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue13170] distutils2 test failures
David Barnett davidbarne...@gmail.com added the comment: The ConfigTestCase.test_parse_extensions_in_config failure is a manifestation of http://bugs.python.org/issue6988. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue13170 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue13170] distutils2 test failures
New submission from Éric Araujo mer...@netwok.org: I get three errors/failures on linux3: ERROR: test_simple_run (distutils2.tests.test_command_install_data.InstallDataTestCase) -- Traceback (most recent call last): File distutils2/tests/test_command_install_data.py, line 76, in test_simple_run cmd.run() File distutils2/command/install_data.py, line 50, in run out = self.copy_file(_file[0], dir_dest)[0] File distutils2/command/cmd.py, line 378, in copy_file copyfile(infile, outfile) File distutils2/_backport/shutil.py, line 83, in copyfile raise Error(`%s` and `%s` are the same file % (src, dst)) Error: `/tmp/user/1013/tmpEl9F8m/tmp91wrrV/foo/inst/three` and `/tmp/user/1013/tmpEl9F8m/tmp91wrrV/foo/inst/three` are the same file == FAIL: test_config (distutils2.tests.test_config.ConfigTestCase) -- Traceback (most recent call last): File distutils2/tests/test_config.py, line 291, in test_config self.assertEqual(dist.package_data, {'cheese': 'data/templates/*'}) AssertionError: {u'cheese': [u'data/templates/*']} != {'cheese': 'data/templates/*'} - {u'cheese': [u'data/templates/*']} ? - -- - + {'cheese': 'data/templates/*'} == FAIL: test_parse_extensions_in_config (distutils2.tests.test_config.ConfigTestCase) -- Traceback (most recent call last): File distutils2/tests/test_config.py, line 341, in test_parse_extensions_in_config self.assertEqual(ext.sources, ['c_src/speed_coconuts.c']) AssertionError: Lists differ: ['c\x00\x00\x00_\x00\x00\x00s\... != ['c_src/speed_coconuts.c'] First differing element 0: c_src/speed_coconuts.c c_src/speed_coconuts.c - ['c\x00\x00\x00_\x00\x00\x00s\x00\x00\x00r\x00\x00\x00c\x00\x00\x00/\x00\x00\x00s\x00\x00\x00p\x00\x00\x00e\x00\x00\x00e\x00\x00\x00d\x00\x00\x00_\x00\x00\x00c\x00\x00\x00o\x00\x00\x00c\x00\x00\x00o\x00\x00\x00n\x00\x00\x00u\x00\x00\x00t\x00\x00\x00s\x00\x00\x00.\x00\x00\x00c\x00\x00\x00'] + ['c_src/speed_coconuts.c'] The last one looks like a codecs issue. -- assignee: tarek components: Distutils2 messages: 145470 nosy: alexis, eric.araujo, tarek priority: normal severity: normal status: open title: distutils2 test failures versions: 3rd party ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue13170 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com