[issue28664] test_bz2 fails with BrokenPipeError when bunzip2 is missing

2016-11-11 Thread Serhiy Storchaka
Changes by Serhiy Storchaka : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___ _

[issue28664] test_bz2 fails with BrokenPipeError when bunzip2 is missing

2016-11-11 Thread Roundup Robot
Roundup Robot added the comment: New changeset 648cd8450f4f by Serhiy Storchaka in branch '3.5': Issue #28664: test_bz2 now works on non-Windows platforms without bunzip2 https://hg.python.org/cpython/rev/648cd8450f4f New changeset 9184f7f11b30 by Serhiy Storchaka in branch '3.6': Issue #28664:

[issue28664] test_bz2 fails with BrokenPipeError when bunzip2 is missing

2016-11-11 Thread Xavier de Gaye
Xavier de Gaye added the comment: Much better indeed :) With this patch, the test runs fine on Android. I left some minor comments on rietveld. -- ___ Python tracker ___

[issue28664] test_bz2 fails with BrokenPipeError when bunzip2 is missing

2016-11-11 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Wouldn't be better to check once the existence of the bunzip2 executable? Proposed patch also simplifies invocation of the bunzip2 command. -- nosy: +serhiy.storchaka versions: +Python 3.5 Added file: http://bugs.python.org/file45441/test_bz2-cmdline-

[issue28664] test_bz2 fails with BrokenPipeError when bunzip2 is missing

2016-11-11 Thread Xavier de Gaye
New submission from Xavier de Gaye: bunzip2 is missing on Android and the following tests of test_bz2 fail randomly: test_implicit_binary_modes, test_binary_modes and testAppend, with the following backtrace: ERROR: testAppend (test.test_bz2.BZ2FileTest)