[issue28914] selectmodule build fails
Chris Rose <off...@offby1.net> added the comment: Ach; ignore the libc version below; that's a PEBKAC error on my part. The libc version on the system that's failing is older (and hard to get; just trust me, it's older) and doesn't have the relevant epoll value. -- ___ Python tracker <rep...@bugs.python.org> <https://bugs.python.org/issue28914> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue28914] selectmodule build fails
Chris Rose <off...@offby1.net> added the comment: Is this patch mergeable? I'm trialing the 3.7.0a4 build on some systems here and am unable to build due to this issue, on libc 2.12: ± /lib/libc.so.6 GNU C Library stable release version 2.12, by Roland McGrath et al. Copyright (C) 2010 Free Software Foundation, Inc. This is free software; see the source for copying conditions. There is NO warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. Compiled by GNU CC version 4.4.6 20110731 . Compiled on a Linux 3.2.5 system on 2017-10-20. Available extensions: The C stubs add-on version 2.1.2. crypt add-on version 2.1 by Michael Glad and others GNU Libidn by Simon Josefsson Native POSIX Threads Library by Ulrich Drepper et al BIND-8.2.3-T5B RT using linux kernel aio -- nosy: +offby1 ___ Python tracker <rep...@bugs.python.org> <https://bugs.python.org/issue28914> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20752] Difflib should provide the option of overriding the SequenceMatcher
Chris Rose added the comment: What's the word on this change? -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20752 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20752] Difflib should provide the option of overriding the SequenceMatcher
Chris Rose added the comment: As suggested: SYN -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20752 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20752] Difflib should provide the option of overriding the SequenceMatcher
Chris Rose added the comment: Ping? I'd like to know if the proposed solution passes muster, so that I can get this into ... well, whatever the right revision would be. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20752 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20752] Difflib should provide the option of overriding the SequenceMatcher
Changes by Chris Rose off...@offby1.net: -- nosy: +r.david.murray ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20752 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20752] Difflib should provide the option of overriding the SequenceMatcher
Chris Rose added the comment: Updated according to review. -- Added file: http://bugs.python.org/file34868/difflib-sm.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20752 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20752] Difflib should provide the option of overriding the SequenceMatcher
Chris Rose added the comment: Addressed comments regarding documentation and assertion formats in the test. -- Added file: http://bugs.python.org/file34869/difflib-sm.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20752 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20752] Difflib should provide the option of overriding the SequenceMatcher
Chris Rose added the comment: Patch against tip -- Added file: http://bugs.python.org/file34871/difflib-sm.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20752 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20752] Difflib should provide the option of overriding the SequenceMatcher
Chris Rose added the comment: As a historical record, it should be noted that this is driven by an actual use case: I was experimenting with using Bazaar's patience diff implementation, and I saw that in order for them to use a custom sequence matcher, they had to essentially copy-paste and modify the stdlib diff methods in order to inject their own sequence matchers. That struck me as a bad thing, and that's pretty much what led to this. I welcome a discussion of the API itself; there's definitely a bit of an odd challenge in describing the usage of the matcher variants when both are used (in line_matcher and char_matcher roles). A possible approach would be to consider matcher factories to take _just_ a junk function, nothing else, and use the SequenceMatcher API's set_seqs method to actually provide the sequences in all cases. This fits the character use case, which reuses the matcher, and the line use case which does not. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20752 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20246] buffer overflow in socket.recvfrom_into
Chris Rose added the comment: Is there an ETA for a 2.7.7 release with this fix? -- nosy: +offby1 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20246 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20752] Difflib should provide the option of overriding the SequenceMatcher
New submission from Chris Rose: It should be possible to provide subclasses or implementations of the SequenceMatcher interface to difflib's various methods that do formatting (for example, unified_diff/context_diff et al). I've included a patch for it that satisfies my personal itch on the subject (attached). I'd like to have this reviewed for inclusion in 3.5. -- components: Library (Lib) files: difflib-sm.patch keywords: patch messages: 212036 nosy: offby1 priority: normal severity: normal status: open title: Difflib should provide the option of overriding the SequenceMatcher type: enhancement versions: Python 3.5 Added file: http://bugs.python.org/file34203/difflib-sm.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20752 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue20752] Difflib should provide the option of overriding the SequenceMatcher
Chris Rose added the comment: I've regenerated the patch with doc additions and Misc/ACKS changed as well. -- Added file: http://bugs.python.org/file34204/difflib-sm.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20752 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11965] Simplify context manager in os.popen
Chris Rose off...@offby1.net added the comment: I'm pretty sure that the os._wrap_close wrapper is not the same thing as the Popen context manager. I don't think it's useful to try refactor this. As Antoine points out, the current wrapper serves a very different purpose. -- nosy: +offby1 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11965 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11970] distutils command 'upload' crashes when --show-response is selected
Chris Rose off...@offby1.net added the comment: No, not 100% sure, but my read of the 2.7 branch code certainly seemed to suggest that it was fixed. in distutil/commands/upload.py: Line 193 gets the response unconditionally as 'r' from the http object Line 201 uses 'r' to show the response. I might be missing a form that the 'r' object can take that isn't going to have a read() method, but I doubt it. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11970 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11970] distutils command 'upload' crashes when --show-response is selected
Chris Rose off...@offby1.net added the comment: ... oh, except I'm an idiot, and I think I'm reading tip code there. Yep, I'm an idiot. Okay, but this is still a dupe of #10367 -- resolution: invalid - duplicate ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11970 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11970] distutils command 'upload' crashes when --show-response is selected
Chris Rose off...@offby1.net added the comment: I can see that this is only an error in the 2.7.1 release, and is fixed in distutils on the 2.7 branch (along with other bugs, too, from what I can see.) I'm closing it as invalid, given that. -- resolution: - invalid status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11970 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11970] distutils command 'upload' crashes when --show-response is selected
New submission from Chris Rose off...@offby1.net: When running distutils like so: .tox/py27/bin/python setup.py -v bdist_egg upload --show-response Eventually, after everything else spools by, this pops up: Using PyPI login from /Users/offline/.pypirc Submitting dist/PyHamcrest-1.5-py2.7.egg to http://pypi.python.org/pypi Upload failed (401): You must be identified to edit package information Traceback (most recent call last): File setup.py, line 69, in module setup(**all_params) File /usr/local/Cellar/python/2.7.1/lib/python2.7/distutils/core.py, line 152, in setup dist.run_commands() File /usr/local/Cellar/python/2.7.1/lib/python2.7/distutils/dist.py, line 953, in run_commands self.run_command(cmd) File /usr/local/Cellar/python/2.7.1/lib/python2.7/distutils/dist.py, line 972, in run_command cmd_obj.run() File /usr/local/Cellar/python/2.7.1/lib/python2.7/distutils/command/upload.py, line 60, in run self.upload_file(command, pyversion, filename) File /usr/local/Cellar/python/2.7.1/lib/python2.7/distutils/command/upload.py, line 193, in upload_file msg = '\n'.join(('-' * 75, r.read(), '-' * 75)) NameError: global name 'r' is not defined -- assignee: tarek components: Distutils files: upload-failed.log messages: 134908 nosy: eric.araujo, offby1, tarek priority: normal severity: normal status: open title: distutils command 'upload' crashes when --show-response is selected type: crash versions: Python 2.7 Added file: http://bugs.python.org/file21845/upload-failed.log ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11970 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com