[issue26927] test_mmap does not handle ValueError when no large file support

Serhiy Storchaka added the comment: I'm wondering in what cases other exceptions (OSError, OverflowError) can be raised? Initial test was added in issue4681. -- nosy: +pitrou, rosslagerwall ___ Python tracker

[issue27048] distutils._msvccompiler._get_vc_env() fails with UnicodeDecodeError if an env var is not encodable

STINNER Victor added the comment: 2016-05-22 11:18 GMT+02:00 Eryk Sun : > subprocess could use "/u /c" instead of "/c". This would only affect the > output of internal shell commands such as "dir" and "set". Oh, I didn't know the interesting "cmd /u" command. It looks

[issue26857] gethostbyname_r() is broken on android

Xavier de Gaye added the comment: > I've moved the include into pyport.h and also fixed the error in my patch, thanks :) With issue26857.diff, importing the socket module does not fail both with an API 21 emulator and an API 23 emulator. -- ___

[issue26647] ceval: use Wordcode, 16-bit bytecode

Changes by Serhiy Storchaka : Added file: http://bugs.python.org/file42947/wpyA.patch ___ Python tracker ___

[issue27067] Improve curses tests

Serhiy Storchaka added the comment: Updated patch makes test_curses working with -jN. But it spoils a screen when run with -jN on a terminal (shouldn't be a problem for buildbots). -- Added file: http://bugs.python.org/file42945/test_curses_no_atty2.patch

[issue26927] test_mmap does not handle ValueError when no large file support

Serhiy Storchaka added the comment: When OverflowError is raised? Shouldn't it be replaced with ValueError? -- ___ Python tracker ___

[issue26647] ceval: use Wordcode, 16-bit bytecode

Demur Rumed added the comment: I've tracked down the lnotab issue after modifying master/wpy to not vacuum NOPs. Old code puts LOAD_CONST at after NOPs, on the same line as return op, whereas new code is putting LOAD_CONST before NOPs, on it's own line I've attached a fix along with

[issue27084] Add dir_fd and follow_symlinks kwargs to os.listdir and os.scandir

Evgeny Kapun added the comment: Mostly for consistency with other functions. Also, this provides an easy way to walk a directory tree recursively: just call listdir on every member, and if it doesn't raise OSError, that member must be a directory. With follow_symlinks=False, this method

[issue26632] __all__ decorator

Martin Panter added the comment: Here are two examples where publicly-documented module attributes are intentionally omitted from __all__: * Issue 26234: typing.re and typing.io * Issue 23439: HTTP status codes like http.client.NOT_FOUND Despite these, I think @public is a reasonable name.

[issue27067] Improve curses tests

Martin Panter added the comment: I’m not 100% following this, but it is sounding like a terminal may be required for proper testing. Perhaps I could try and help by creating a pseudo-terminal in the Python test code, as I hinted at . --

[issue26632] __all__ decorator

Barry A. Warsaw added the comment: On May 20, 2016, at 07:02 AM, Berker Peksag wrote: >+1 for the idea. I saw a lot of different 'all' or 'public' decorators in the >wild. It would be nice to have a complete solution in Python. It would be >good to add a note to Doc/whatsnew/3.6.rst. Is that a

[issue26647] ceval: use Wordcode, 16-bit bytecode

Demur Rumed added the comment: Removes 0 <= unsigned assertion & fixes j < 0 check to avoid overflow bug -- Added file: http://bugs.python.org/file42950/wpyD.patch ___ Python tracker

[issue24225] Idlelib: changing file names

Terry J. Reedy added the comment: Attached are my current renames, with just a few changes from before Most are lower case versions of the current names, perhaps shortened. I expect names with '_' to disappear in future file merges. I may change 'outwin' before 3.6 is released. The

[issue27088] doc: select: epoll.poll: incorrect timeout units, missing maxevents

New submission from Steve Ford: The short description of epoll.poll is correct (gives timeout unit as seconds [float]; includes maxevents). However, the long description lists the timeout units as milliseconds ("If timeout is given, it specifies the length of time in milliseconds which the

[issue26632] __all__ decorator

Barry A. Warsaw added the comment: On May 22, 2016, at 11:30 PM, Martin Panter wrote: >Here are two examples where publicly-documented module attributes are >intentionally omitted from __all__: > >* Issue 26234: typing.re and typing.io >* Issue 23439: HTTP status codes like

[issue27084] Add dir_fd and follow_symlinks kwargs to os.listdir and os.scandir

Serhiy Storchaka added the comment: But why you want to pass O_NOFOLLOW to the underlying open(2) syscall? -- ___ Python tracker ___

[issue26980] The path argument of asyncio.BaseEventLoop.create_unix_connection is not documented

Guido van Rossum added the comment: It's the name of the UNIX domain socket. -- ___ Python tracker ___ ___

[issue26647] ceval: use Wordcode, 16-bit bytecode

STINNER Victor added the comment: wpyD.patch LGTM, go ahead! We can still polish it later and discuss how to implement the 16-bit fetch ;-) It would be nice to add a short comment in Python/wordcode_helpers.h explaining that it contains code shared by the compiler and the peephole optimizer.

[issue27071] unittest.TestCase.assertCountEqual is a very misleading name

Robert Collins added the comment: I don't particularly like either name FWIW :) - but sure, we can add a different name and deprecate assertCountEqual - but before we do anything lets look up the previous issue where the rename was done. -- ___

[issue26632] __all__ decorator

Barry A. Warsaw added the comment: On May 20, 2016, at 07:21 AM, Franklin? Lee wrote: >I am iffy about using ``public`` to define other values. That part might be >considered unpythonic. It's a bit of a stretch. I like it for the convenience, and the implementation is simple, but if e.g.

[issue26647] ceval: use Wordcode, 16-bit bytecode

Serhiy Storchaka added the comment: LGTM. If no one has more comments, I'm going to commit the patch. -- assignee: -> serhiy.storchaka stage: patch review -> commit review ___ Python tracker

[issue27085] Make it possible to select return type for os.listdir

New submission from Evgeny Kapun: Currently, os.listdir returns a list of strings, unless called with a bytes argument, in which case a list of byte strings is returned. I think that there should be a keyword argument to override this selection. -- components: Library (Lib) messages:

[issue27086] Add closefd argument to os.listdir

New submission from Evgeny Kapun: Currently, when given a file descriptor, os.listdir will duplicate it so that the original file descriptor is not closed. In many cases, a file descriptor is not needed anymore after directory is listed, so this is not necessary. I propose adding a keyword

[issue26647] ceval: use Wordcode, 16-bit bytecode

Demur Rumed added the comment: I have verified that wpyC does not produce signed/unsigned warnings with make DEBUG=1 -- Added file: http://bugs.python.org/file42949/wpyC.patch ___ Python tracker

[issue27067] Improve curses tests

STINNER Victor added the comment: Zachary Ware added the comment: >> Maybe we should have at least one buildbot testing test_curses with a TTY? > My non-debug and installed Gentoo bot does run the tests with a PTY, ... This one?

[issue27086] Add closefd argument to os.listdir

Serhiy Storchaka added the comment: What is the problem with closing a file descriptor after calling os.listdir()? -- nosy: +serhiy.storchaka ___ Python tracker

[issue27085] Make it possible to select return type for os.listdir

Serhiy Storchaka added the comment: You can just convert os.listdir() argument or result. os.listdir(os.fsencode(strpath)) os.listdir(os.fsdecode(bytespath)) list(map(os.fsencode, os.listdir(strpath))) list(map(os.fsdecode, os.listdir(bytespath))) -- nosy: +serhiy.storchaka

[issue27084] Add dir_fd and follow_symlinks kwargs to os.listdir and os.scandir

Serhiy Storchaka added the comment: os.listdir() with dir_fd: fd = os.open(path, os.O_RDONLY, dir_fd=dir_fd) try: result = os.listdir(fd) finally: os.close(fd) What meaning of follow_symlinks for os.listdir()? -- nosy: +serhiy.storchaka

[issue27087] unable to use socket send and sendall due to type error

New submission from Nick McElwaine: import socket # Import socket module s = socket.socket(socket.AF_INET,socket.SOCK_STREAM) s.connect(hostname,port) # this works message = 'text' s.sendall(message) TypeError: a bytes-like object is required, not 'str' s.sendall(message)

[issue27084] Add dir_fd and follow_symlinks kwargs to os.listdir and os.scandir

Evgeny Kapun added the comment: 1. Yes, it's possible to emulate dir_fd this way, so this is just for convenience. 2. If follow_symlinks is False, O_NOFOLLOW is passed to the underlying open(2) syscall. Of course, this doesn't make sense if a file descriptor is passed already. --

[issue27064] Make py.exe default to Python 3 when used interactively

Changes by Paul Moore : -- resolution: -> fixed status: open -> closed ___ Python tracker ___

[issue27064] Make py.exe default to Python 3 when used interactively

Changes by Zachary Ware : -- stage: patch review -> resolved ___ Python tracker ___

[issue27087] unable to use socket send and sendall due to type error

Eric V. Smith added the comment: You'll need to convert the string to bytes first, by using .encode(). -- nosy: +eric.smith resolution: -> not a bug stage: -> resolved status: open -> closed ___ Python tracker

[issue27086] Add closefd argument to os.listdir

Evgeny Kapun added the comment: Not a problem, just two unnecessary syscalls. Also, I think that many of those who pass a file descriptor to os.listdir don't need it afterwards, because after you fstat() a file descriptor (to discover that it points to a directory) and read the directory

[issue26673] Tkinter error when opening IDLE configuration menu (Tcl/Tk 8.6)

Roundup Robot added the comment: New changeset a873265366ba by Terry Jan Reedy in branch '2.7': Issue #26673: Protect IDLE from Linux fonts with reported default size 0. https://hg.python.org/cpython/rev/a873265366ba New changeset 1464df337152 by Terry Jan Reedy in branch '3.5': Issue #26673:

[issue26647] ceval: use Wordcode, 16-bit bytecode

Serhiy Storchaka added the comment: Warnings still emitted in debug build. -- ___ Python tracker ___ ___

[issue26673] Tkinter error when opening IDLE configuration menu (Tcl/Tk 8.6)

Terry J. Reedy added the comment: I would like to change the code a bit more (like using IntVar for size everywhere), but I don't want to do so without tests. The problem is that tk variables and font functions require a root window and none of the linux buildbots allow that. (They all run

[issue26927] test_mmap does not handle ValueError when no large file support

Xavier de Gaye added the comment: On linux with large file support, OSError is raised when the file system limit is exceeded, here with ext4: >>> f.seek(2**44 - 2**11) Traceback (most recent call last): File "", line 1, in OSError: [Errno 22] Invalid argument >>> f.seek(2**44 - 2**12)

[issue25413] ctypes (libffi) fails to compile on Solaris X86

Changes by Berker Peksag : -- nosy: +doko ___ Python tracker ___ ___

[issue27064] Make py.exe default to Python 3 when used interactively

Roundup Robot added the comment: New changeset 26c1e3dab624 by Paul Moore in branch 'default': Issue #27064: The py.exe launcher now defaults to Python 3. https://hg.python.org/cpython/rev/26c1e3dab624 -- nosy: +python-dev ___ Python tracker

[issue26647] ceval: use Wordcode, 16-bit bytecode

Demur Rumed added the comment: Compiler warnings can be fixed by using h instead of j in case RETURN_VALUE & casting CONST_LEN() to unsigned is safe As for the failing tests: I've finally figured out how to use Tools/freeze to fix __phello__ et al. I've brought up the trace test failure a few

[issue26865] Meta-issue: support of the android platform

Xavier de Gaye added the comment: issue #16255: subrocess.Popen needs /bin/sh but Android only has /system/bin/sh issue #16353: add function to os module for getting path to default shell -- dependencies: +subrocess.Popen needs /bin/sh but Android only has /system/bin/sh

[issue18383] test_warnings modifies warnings.filters when running with "-W default"

Serhiy Storchaka added the comment: $ ./python -We::UserWarning -m test.regrtest test_warnings Run tests sequentially 0:00:00 [1/1] test_warnings /home/serhiy/py/cpython/Lib/test/test_warnings/__init__.py:108: UserWarning: FilterTests.test_ignore_after_default self.module.warn(message,

[issue26857] gethostbyname_r() is broken on android

Stefan Krah added the comment: Okay thanks, let's assume api-level >= 21 for now. I've moved the include into pyport.h in order to save a little space everywhere. Could you check if that works? -- Added file: http://bugs.python.org/file42944/issue26857.diff

[issue25077] Compiler warnings: initialization from incompatible pointer type

Berker Peksag added the comment: libffi warnings have already been fixed in upstream libffi. See issue 23085 for updating our libffi copy. -- nosy: +berker.peksag ___ Python tracker

[issue27067] Improve curses tests

STINNER Victor added the comment: By the way, all buildbots now use -j1 option and so stdout and stderr are no more TTY on all buildbots: see issue #25285. So test_curses are not fully tested by buildbots anymore. Maybe we should have at least one buildbot testing test_curses with a TTY?

[issue27081] Multiprocessing is not robust against sys.stderr changes involving queues

Changes by ppperry : -- title: Multiprocessing is not robust against sys.stderr changes -> Multiprocessing is not robust against sys.stderr changes involving queues ___ Python tracker

[issue16255] subrocess.Popen needs /bin/sh but Android only has /system/bin/sh

Xavier de Gaye added the comment: The list of locations where '/bin/sh' is hard coded in the standard library: Lib/distutils/tests/test_build_scripts.py|68 col 31| ("#!/bin/sh\n" Lib/distutils/tests/test_install_scripts.py|57 col 38| write_script("shell.sh", ("#!/bin/sh\n"

[issue26647] ceval: use Wordcode, 16-bit bytecode

Demur Rumed added the comment: Sorry for the nuisance of uploading another patch so soon. wpyB modifies test_ctypes now that __phello__ is smaller, & fixes a typo in a comment I made & removes a blank line I had added in when adding in if(0) logic -- Added file:

[issue26944] android: test_posix fails

Xavier de Gaye added the comment: test_getgroups is ok on an android emulator running API 23 with the attached patch, and the test is skipped when running API 21. -- dependencies: +add is_android in test.support to detect Android platform -android: add platform.android_ver() keywords:

[issue26865] Meta-issue: support of the android platform

Xavier de Gaye added the comment: issue #22747: Interpreter fails in initialize on systems where HAVE_LANGINFO_H is undefined -- dependencies: +Interpreter fails in initialize on systems where HAVE_LANGINFO_H is undefined ___ Python tracker

[issue26857] gethostbyname_r() is broken on android

Roundup Robot added the comment: New changeset 09af54099973 by Stefan Krah in branch 'default': Issue #26857: The gethostbyaddr_r() workaround is no longer needed with https://hg.python.org/cpython/rev/09af54099973 -- ___ Python tracker

[issue26857] gethostbyname_r() is broken on android

Stefan Krah added the comment: Thanks! Closing again. -- resolution: -> fixed status: open -> closed ___ Python tracker ___

[issue27084] Add dir_fd and follow_symlinks kwargs to os.listdir and os.scandir

New submission from Evgeny Kapun: Many functions in os module support dir_fd and follow_symlinks keyword arguments. I think that os.listdir and os.scandir should do likewise. See also: issue25996. -- components: Library (Lib) messages: 266091 nosy: abacabadabacaba priority: normal

[issue27067] Improve curses tests

Zachary Ware added the comment: > Maybe we should have at least one buildbot testing test_curses with a TTY? My non-debug and installed Gentoo bot does run the tests with a PTY, so you can use its custom builders for experimentation (assuming it can complete a clone). --

[issue16255] subrocess.Popen needs /bin/sh but Android only has /system/bin/sh

Stefan Krah added the comment: I've also opened a feature request here: https://code.google.com/p/android/issues/detail?id=210812 -- ___ Python tracker

[issue25996] Add support of file descriptor in os.scandir()

Changes by Evgeny Kapun : -- nosy: +abacabadabacaba ___ Python tracker ___ ___

[issue27067] Improve curses tests

STINNER Victor added the comment: test_curses_no_atty.patch doesn't work for me using -j1 regrtest option :-/ haypo@smithers$ ./python -m test -u curses -j1 test_curses Run tests in parallel using 1 child processes Total duration: 0:00:01 Exception in thread Thread-1: Traceback (most recent

[issue18383] test_warnings modifies warnings.filters when running with "-W default"

Berker Peksag added the comment: LGTM -- stage: patch review -> commit review ___ Python tracker ___ ___

[issue27067] Improve curses tests

Roundup Robot added the comment: New changeset 433839643bbb by Serhiy Storchaka in branch '3.5': Temporary skip curses tests on non-tty (issue #27067). https://hg.python.org/cpython/rev/433839643bbb New changeset debe693c62b4 by Serhiy Storchaka in branch 'default': Temporary skip curses tests

[issue27071] unittest.TestCase.assertCountEqual is a very misleading name

Vitaly added the comment: As far as I can tell, issue #10242 deals with the renaming. I think that the arguments for the new name are flawed (sorry to be so blunt, but I don't know how else to express it). Regardless of whether it's Count or Counts, to most people it sounds like it's going

[issue26927] test_mmap does not handle ValueError when no large file support

Roundup Robot added the comment: New changeset bca81ea8f898 by Serhiy Storchaka in branch '3.5': Issue #26927: Fixed test_mmap on platforms with 32-bit off_t (like Android). https://hg.python.org/cpython/rev/bca81ea8f898 New changeset 6147a2c99db0 by Serhiy Storchaka in branch 'default': Issue

[issue26927] test_mmap does not handle ValueError when no large file support

Xavier de Gaye added the comment: f.seek(number) raises ValueError (ValueError: cannot fit 'int' into an offset-sized integer) when 'number' is greater than the size allowed by off_t. ValueError is not handled in test_mmap to detect that large file is not supported. With this patch,

[issue26673] Tkinter error when opening IDLE configuration menu (Tcl/Tk 8.6)

Odcar added the comment: Terry J. Reedy: That fixes the problem for me! Xubuntu 16.04, Python 2.7.11+, Tk 8.6.5. I have not tried it in Idle for Python 3 though. -- nosy: +Odcar ___ Python tracker

[issue27072] random.getrandbits is limited to 2**31-1 bits on 64-bit Windows

Serhiy Storchaka added the comment: The only case I know that would benefit is generating random data for tests. On my computer generating 2*28 bits with getrandbits() takes 2 sec (including 1 sec for converting from bytes to int), plus 1.4 sec for converting from int to bytes. Special API

[issue20699] Document that binary IO classes work with bytes-likes objects

Serhiy Storchaka added the comment: bytes-like-param.v6.patch LGTM. -- ___ Python tracker ___ ___

[issue26971] python v3.5.1: sys.paths not respecting DESTDIRS and DESTSHARED

Ned Deily added the comment: Sorry, I'm not able to reproduce your results on a vanilla Debian system. Building from scratch with ./configure --prefix=/usr and make steps results in: $ /usr/bin/python3.5 Python 3.5.1 (default, May 22 2016, 02:26:09) [GCC 4.9.2] on linux Type "help",

[issue27083] PYTHONCASEOK is ignored on Windows

New submission from Eryk Sun: importlib ignores the PYTHONCASEOK environment variable on Windows. _relax_case checks for b'PYTHONCASEOK' in os.environ, which is never true because os.environ is Unicode. On Windows, _make_relax_case should return a function that instead checks for

[issue26980] The path argument of asyncio.BaseEventLoop.create_unix_connection is not documented

Changes by Ned Deily : -- components: +asyncio nosy: +gvanrossum, haypo, yselivanov versions: -Python 3.4 ___ Python tracker ___

[issue26955] Implement equivalent to `pip.locations.distutils_scheme` in distutils

Sylvain Corlay added the comment: Ned, that is because these packages (lxml, cffi) have header files as `package_data`, instead of `headers`. This is why they are being copied into site-packages. People willing to include them must either rely on knowledge of their location, or a python

[issue26926] Large files are not supported on Android

Xavier de Gaye added the comment: Size of off_t is also 4 when building on API 23 with or without '-D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE=1 -D_LARGEFILE64_SOURCE=1' added to the CC envt variable. -- ___ Python tracker

[issue26331] Tokenizer: allow underscores for grouping in numeric literals

Georg Brandl added the comment: Thanks for the detailed review, Serhiy! Next try incoming. -- Added file: http://bugs.python.org/file42939/numeric_underscores_final_v8.diff ___ Python tracker

[issue26647] ceval: use Wordcode, 16-bit bytecode

Serhiy Storchaka added the comment: test_importlib and test_trace are failed with wpy9.patch. -- ___ Python tracker ___

[issue26858] android: setting SO_REUSEPORT fails

Xavier de Gaye added the comment: New patch test.asyncio_2.patch uses test.support.is_android and adds a dependency to issue #27027. -- dependencies: +add is_android in test.support to detect Android platform Added file: http://bugs.python.org/file42940/test.asyncio_2.patch

[issue27048] distutils._msvccompiler._get_vc_env() fails with UnicodeDecodeError if an env var is not encodable

Eryk Sun added the comment: When writing to a file or pipe, cmd's internal commands default to either the current codepage of the attached console or, for a detached process, the system locale codepage. It uses a best-fit encoding that tries to map characters to similar characters in the

[issue26926] Large files are not supported on Android

Xavier de Gaye added the comment: This patch skips the large file support test. -- dependencies: +add is_android in test.support to detect Android platform Added file: http://bugs.python.org/file42941/skip-large-file.patch ___ Python tracker