[issue7946] Convoy effect with I/O bound threads and New GIL
Michele added the comment: Attached ccbench-osx.log made today on OSX on latest svn checkout. Hope it helps -- nosy: +Michele Added file: http://bugs.python.org/file17393/ccbench-osx.log ___ Python tracker <http://bugs.python.org/issue7946> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue13294] http.server: HEAD request should not return a body
Michele Orrù added the comment: Well, actually SimpleHTTPRequesthandler extends BaseHTTPHandler with basic do_GET and do_HEAD methods. Unittests for http.server shows that this behavior is intended, since: Traceback (most recent call last): File "Lib/test/test_httpservers.py", line 639, in test_main() File "Lib/test/test_httpservers.py", line 633, in test_main SimpleHTTPRequestHandlerTestCase, File "/Users/maker/dev/cpython/Lib/test/support.py", line 1274, in run_unittest _run_suite(suite) File "/Users/maker/dev/cpython/Lib/test/support.py", line 1249, in _run_suite raise TestFailed(err) test.support.TestFailed: Traceback (most recent call last): File "Lib/test/test_httpservers.py", line 273, in test_head self.check_status_and_reason(response, 200) File "Lib/test/test_httpservers.py", line 242, in check_status_and_reason self.assertEqual(response.status, status) AssertionError: 501 != 200 So, imho this is not a bug. Anyway, I would propose a trivial patch to make http.server a little more elegant. -- keywords: +patch nosy: +maker Added file: http://bugs.python.org/file23711/issue13294.patch ___ Python tracker <http://bugs.python.org/issue13294> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue13294] http.server: HEAD request should not return a body
Michele Orrù added the comment: These tests shows how SimpleHTTPRequestHandler behaves: if the class contains a do_FOO method, it is called, otherwise error501 is raised. That's what Karl said with «Or to modify the library code that for any resources not yet defined.». Since SimpleHTTPRequestHandler.do_HEAD exists, and this is reflected in the correspondent unittest, I belive that this is not a bug. Patch attached. No problem if it's not committed now. I hope that someone in the noisy list will make a little cleanup one day :) > > -- > > ___ > Python tracker > <http://bugs.python.org/issue13294> > ___ > -- Added file: http://bugs.python.org/file23723/issue13294.patch ___ Python tracker <http://bugs.python.org/issue13294> ___diff -r a00bb30cf775 Lib/http/server.py --- a/Lib/http/server.pyTue Nov 15 16:12:49 2011 +0100 +++ b/Lib/http/server.pyWed Nov 16 11:15:45 2011 +0100 @@ -271,14 +271,11 @@ self.request_version = version = self.default_request_version self.close_connection = 1 requestline = str(self.raw_requestline, 'iso-8859-1') -if requestline[-2:] == '\r\n': -requestline = requestline[:-2] -elif requestline[-1:] == '\n': -requestline = requestline[:-1] +requestline = requestline.rstrip('\r\n') self.requestline = requestline words = requestline.split() if len(words) == 3: -[command, path, version] = words +command, path, version = words if version[:5] != 'HTTP/': self.send_error(400, "Bad request version (%r)" % version) return False @@ -304,7 +301,7 @@ "Invalid HTTP Version (%s)" % base_version_number) return False elif len(words) == 2: -[command, path] = words +command, path = words self.close_connection = 1 if command != 'GET': self.send_error(400, ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue13294] http.server: HEAD request should not return a body
Michele Orrù added the comment: As Ezio just pointed out, strip('\r\n') is still behaves differently from the previous code. Sorry for that. -- ___ Python tracker <http://bugs.python.org/issue13294> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue13642] urllib incorrectly quotes username and password in https basic auth
Michele Orrù added the comment: > Joonas, this issue seems easy to solve. Do you want to try to post a > patch?. Extra credits for patching testsuite too :). As far as I see, it would be sufficient to add unquote(passed) to _open_generic_http. Regarding unittests instead, there is already a method called test_userpass_inurl which could be extended with some tests on a password containing spaces ( Lib/test/test_urllib.py:263). But what I haven't yet understood is: does it really exists a user:pass in python.org? -- nosy: +maker ___ Python tracker <http://bugs.python.org/issue13642> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue13642] urllib incorrectly quotes username and password in https basic auth
Michele Orrù added the comment: There's no need to port your patch over python3k, since urllib behaves differently with http passwords - as you can see in the doc http://docs.python.org/dev/py3k/library/urllib.request.html#examples I would be glad to finish your password on 2.7 as soon as possible, joneskoo. Thanks. -- ___ Python tracker <http://bugs.python.org/issue13642> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue13642] urllib incorrectly quotes username and password in https basic auth
Michele Orrù added the comment: Patch attached. Note that now unquote is called with host using map(), and b64 encoded strings are no more hardcoded. Please tell me if those changes are acceptable - anyway they don't break any other unit tests. -- nosy: +ezio.melotti Added file: http://bugs.python.org/file24186/issue13642.patch ___ Python tracker <http://bugs.python.org/issue13642> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue13642] urllib incorrectly quotes username and password in https basic auth
Michele Orrù added the comment: Whoops, probably I tested using $ python instead of $ ./python.exe - Attaching two patches, one keeps using map(), but definitely changes unquote() behavior; the other simply asserts user_passwd exists before using unquote(). Well, concerning the class field abuse, HTTPConnection._buffer attribute might help? The point is that I can't find an easy way to get the HTTPConnection instance from an urlopen(). -- Added file: http://bugs.python.org/file24191/issue13642.patch ___ Python tracker <http://bugs.python.org/issue13642> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue13642] urllib incorrectly quotes username and password in https basic auth
Changes by Michele Orrù : Removed file: http://bugs.python.org/file24186/issue13642.patch ___ Python tracker <http://bugs.python.org/issue13642> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue13642] urllib incorrectly quotes username and password in https basic auth
Changes by Michele Orrù : Added file: http://bugs.python.org/file24192/issue13642_with_map.patch ___ Python tracker <http://bugs.python.org/issue13642> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue13359] urllib2 doesn't escape spaces in http requests
Michele Orrù added the comment: Patch attached for python3, with unit tests. -- nosy: +maker Added file: http://bugs.python.org/file24215/issue13359.patch ___ Python tracker <http://bugs.python.org/issue13359> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue13359] urllib2 doesn't escape spaces in http requests
Michele Orrù added the comment: Here the patch for python2. kiilerix, RFC 1738 explicitly says that the space character shall not be used. -- Added file: http://bugs.python.org/file24216/issue13359_py2.patch ___ Python tracker <http://bugs.python.org/issue13359> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue8898] The email package should defer to the codecs module for all aliases
Michele Orrù added the comment: In the sense that the alias for 'tactis' should be removed also in 2.7 and 3.2? -- title: The email package should defer to the codecs module for all aliases -> The email package should defer to the codecs module for all aliases ___ Python tracker <http://bugs.python.org/issue8898> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue8898] The email package should defer to the codecs module for all aliases
Changes by Michele Orrù : Removed file: http://bugs.python.org/file22064/unnamed ___ Python tracker <http://bugs.python.org/issue8898> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue8898] The email package should defer to the codecs module for all aliases
Michele Orrù added the comment: After discussing on IRC, it figured out that the best choice would be to use normalize_encoding plus ALIAS, as the attached patch does. -- Added file: http://bugs.python.org/file22094/issue8898_normalize.patch ___ Python tracker <http://bugs.python.org/issue8898> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue9654] merge PC/getpathp.c into Modules/getpath.c
Michele Orrù added the comment: In which cases it goes to PC/getpathp.c? I suppose it's Modules/getpath.c otherwise. Line 495 on getpathp.c let me guess it's not only for Windows. -- nosy: +maker ___ Python tracker <http://bugs.python.org/issue9654> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10424] better error message from argparse when positionals missing
Michele Orrù added the comment: Done. -- Added file: http://bugs.python.org/file22129/issue10424_2.patch ___ Python tracker <http://bugs.python.org/issue10424> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue8898] The email package should defer to the codecs module for all aliases
Michele Orrù added the comment: +1 What do you think? Ezio, David? -- title: The email package should defer to the codecs module for all aliases -> The email package should defer to the codecs module for all aliases ___ Python tracker <http://bugs.python.org/issue8898> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue8898] The email package should defer to the codecs module for all aliases
Michele Orrù added the comment: In that case, I could still take care of it; it would be really easy to do. So, it's up to you to tell me what is the best design choice. (: -- ___ Python tracker <http://bugs.python.org/issue8898> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue8898] The email package should defer to the codecs module for all aliases
Michele Orrù added the comment: Any idea about how to unittest mime.aliases? Also, since I've just created a new file, are there some buracratic issues? I mean, do I have to add something at the top of the file? (I'm just signing the Contributor Agreement) -- Added file: http://bugs.python.org/file22146/issue8898_2.patch ___ Python tracker <http://bugs.python.org/issue8898> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue8898] The email package should defer to the codecs module for all aliases
Changes by Michele Orrù : Added file: http://bugs.python.org/file22153/issue8898_3.patch ___ Python tracker <http://bugs.python.org/issue8898> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10424] better error message from argparse when positionals missing
Changes by Michele Orrù : Added file: http://bugs.python.org/file22155/issue10424.patch ___ Python tracker <http://bugs.python.org/issue10424> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue12757] undefined name in doctest.py
Michele Orrù added the comment: It is possible to retrieve the current module using _normalize_module(None), or instead use the test name (dt_test.name) just like in DocTestCase.shortDescription. Since there is no doc about it, IMHO we should use unittest's standard as guideline, which is: shortDescription() Returns a description of the test, or None if no description has been provided. The default implementation of this method returns the first line of the test method’s docstring, if available, or None. (i.e., DocTestCase._dt_test.name) What's your opinion? -- nosy: +maker ___ Python tracker <http://bugs.python.org/issue12757> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue6584] gzip module has no custom exception
Michele Orrù added the comment: The attached patch follows Ezio's hints. -- nosy: +maker Added file: http://bugs.python.org/file22956/6584_4.patch ___ Python tracker <http://bugs.python.org/issue6584> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue6584] gzip module has no custom exception
Changes by Michele Orrù : Added file: http://bugs.python.org/file22958/6584_5.patch ___ Python tracker <http://bugs.python.org/issue6584> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue6669] TarFile.getmembers fails at struct.unpack: unpack requires a string argument of length 4
Michele Orrù added the comment: Would it be better to use TarError as Sridhar suggested, or create a new class BadTarfile(TarError, IOError), following the convention used for gzip and zipfile? -- nosy: +maker ___ Python tracker <http://bugs.python.org/issue6669> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue6584] gzip module has no custom exception
Changes by Michele Orrù : -- nosy: +ezio.melotti ___ Python tracker <http://bugs.python.org/issue6584> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue6669] TarFile.getmembers fails at struct.unpack: unpack requires a string argument of length 4
Changes by Michele Orrù : -- nosy: +ezio.melotti ___ Python tracker <http://bugs.python.org/issue6669> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue8898] The email package should defer to the codecs module for all aliases
Michele Orrù added the comment: The attached patch adds aliases for latin_N in encodings.aliases, and fixes email.charset behaviour according to codecs.lookup, as requested. Tested on (Arch) Linux. Am I supposed to add any unittest? I'm wavering about where they should be placed (in encodings or email?). -- keywords: +patch nosy: +ezio.melotti, maker Added file: http://bugs.python.org/file22049/issue8898.patch ___ Python tracker <http://bugs.python.org/issue8898> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue8898] The email package should defer to the codecs module for all aliases
Changes by Michele Orrù : -- nosy: +eric.araujo ___ Python tracker <http://bugs.python.org/issue8898> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue8898] The email package should defer to the codecs module for all aliases
Changes by Michele Orrù : Removed file: http://bugs.python.org/file22049/issue8898.patch ___ Python tracker <http://bugs.python.org/issue8898> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue8898] The email package should defer to the codecs module for all aliases
Changes by Michele Orrù : Added file: http://bugs.python.org/file22053/issue8898.patch ___ Python tracker <http://bugs.python.org/issue8898> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue8898] The email package should defer to the codecs module for all aliases
Michele Orrù added the comment: Well, actually encodings.aliases links to the encoding _module name_, as described in the doc: """ Encoding Aliases Support This module is used by the encodings package search function to map encodings names to module names. """ So I've adjusted your snippet according to this, as you can see in the attachment. I've also slightly changed the imports as pep8 says: """ Yes: import os import sys No: import sys, os """ Anyway, running the test failed for two encodings, there are two bugs there, indeed. - mcbs has something broken in its imports; - tactis module is not present. Since they are really easy to fix, I haven't yet reported to the bugtraker. Let me know what should I do. Post on bugs.python.org bug and patch? Any new test specifically for the email module? -- Added file: http://bugs.python.org/file22057/unnamed ___ Python tracker <http://bugs.python.org/issue8898> ___ Regarding the tests, I don't see tests for the aliases anywhere, so something like: for alias, codec_name in encodings.aliases.items():   self.assertEqual(codecs.lookup(alias).name, codec_name) could be added somewhere to check that all the aliases in the dict map to the correct codec.Well, actually encodings.aliases links to the encoding _module name_, as described in the doc:""" Encoding Aliases Support   This module is used by the encodings package search function to  map encodings names to module names."""So I've adjusted your snippet according to this, as you can see in the attachment. I've also slightly changed the imports as pep8 says:"""Yes: import os import sysNo: import sys, os""" Anyway, running the test failed for two encodings, there are two bugs there, indeed.- mcbs has something broken in its imports;- tactis module is not present. Since they are really easy to fix, I haven't yet reported to the bugtraker. Let me know what should I do.Post on http://bugs.python.org";>bugs.python.org bug and patch? Any new test specifically for the email module? -- Michele Orrù ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue8898] The email package should defer to the codecs module for all aliases
Changes by Michele Orrù : Added file: http://bugs.python.org/file22058/fail_tactis.txt ___ Python tracker <http://bugs.python.org/issue8898> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue8898] The email package should defer to the codecs module for all aliases
Changes by Michele Orrù : Added file: http://bugs.python.org/file22059/issue8898_withtests.patch ___ Python tracker <http://bugs.python.org/issue8898> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue8898] The email package should defer to the codecs module for all aliases
Changes by Michele Orrù : Added file: http://bugs.python.org/file22060/fail_mcbs.txt ___ Python tracker <http://bugs.python.org/issue8898> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue8898] The email package should defer to the codecs module for all aliases
Michele Orrù added the comment: So, what do you prefer? Add a check for sys.platform, or just skip it? discussion on python-dev. So I'm +1 for just skipping it for now (with a XXX comment on the right maybe). -- title: The email package should defer to the codecs module for all aliases -> The email package should defer to the codecs module for all aliases Added file: http://bugs.python.org/file22064/unnamed ___ Python tracker <http://bugs.python.org/issue8898> ___ > - mcbs has something broken in its imports; mbcs is only available on Windows.So, what do you prefer? Add a check for sys.platform, or just skip it? > - tactis module is not present. I'm not sure what happened here: either the alias entry is wrong or the codec module was not committed. In either case, no one has complained about this encoding not working, so we can probably just remove it from the alias table. See http://bugs.python.org/issue1251921"; target="_blank">http://bugs.python.org/issue1251921 for a similar report and discussion. I don't have such autority, and probably such a choice will require a discussion on python-dev. So I'm +1 for just skipping it for now (with a XXX comment on the right maybe). -- Michele Orrù ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue8898] The email package should defer to the codecs module for all aliases
Michele Orrù added the comment: Sorry, I was told that email the bugtracker could not work properly. > > - mcbs has something broken in its imports; > mbcs is only available on Windows. So, what do you prefer? Add a check for sys.platform, or just skip it? > > - tactis module is not present. > I'm not sure what happened here: either the alias entry is wrong > or the codec module was not committed. > In either case, no one has complained about this encoding not working, > so we can probably just remove it from the alias table. See > http://bugs.python.org/issue1251921 for a similar report and > discussion. I don't have such autority, and probably such a choice will require a discussion on python-dev. So I'm +1 for just skipping it for now (with a XXX comment on the right maybe). -- ___ Python tracker <http://bugs.python.org/issue8898> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue8898] The email package should defer to the codecs module for all aliases
Changes by Michele Orrù : Removed file: http://bugs.python.org/file22057/unnamed ___ Python tracker <http://bugs.python.org/issue8898> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue8898] The email package should defer to the codecs module for all aliases
Michele Orrù added the comment: unittest.skip* are decorators, so useless in this case; also, AFAIS Lib/test/ uses sys.platform. I would suggest to put a try statement in encodings.mbcs, and raise an error in case the imported modules imported are not found. But this is another story. -- title: The email package should defer to the codecs module for all aliases -> The email package should defer to the codecs module for all aliases Added file: http://bugs.python.org/file22065/issue8898_skip.patch ___ Python tracker <http://bugs.python.org/issue8898> ___diff -r cc60d0283fad Lib/email/charset.py --- a/Lib/email/charset.py Fri May 20 16:55:06 2011 +0200 +++ b/Lib/email/charset.py Sun May 22 14:18:05 2011 +0200 @@ -10,6 +10,7 @@ ] from functools import partial +from codecs import lookup import email.base64mime import email.quoprimime @@ -63,36 +64,6 @@ 'utf-8': (SHORTEST, BASE64, 'utf-8'), } -# Aliases for other commonly-used names for character sets. Map -# them to the real ones used in email. -ALIASES = { -'latin_1': 'iso-8859-1', -'latin-1': 'iso-8859-1', -'latin_2': 'iso-8859-2', -'latin-2': 'iso-8859-2', -'latin_3': 'iso-8859-3', -'latin-3': 'iso-8859-3', -'latin_4': 'iso-8859-4', -'latin-4': 'iso-8859-4', -'latin_5': 'iso-8859-9', -'latin-5': 'iso-8859-9', -'latin_6': 'iso-8859-10', -'latin-6': 'iso-8859-10', -'latin_7': 'iso-8859-13', -'latin-7': 'iso-8859-13', -'latin_8': 'iso-8859-14', -'latin-8': 'iso-8859-14', -'latin_9': 'iso-8859-15', -'latin-9': 'iso-8859-15', -'latin_10':'iso-8859-16', -'latin-10':'iso-8859-16', -'cp949': 'ks_c_5601-1987', -'euc_jp': 'euc-jp', -'euc_kr': 'euc-kr', -'ascii': 'us-ascii', -} - - # Map charsets to their Unicode codec strings. CODEC_MAP = { 'gb2312': 'eucgb2312_cn', @@ -103,6 +74,8 @@ 'us-ascii':None, } +# Aliases defined by the user +ALIASES = dict() # Convenience functions for extending the above mappings @@ -220,9 +193,12 @@ input_charset = str(input_charset, 'ascii') except UnicodeError: raise errors.CharsetError(input_charset) -input_charset = input_charset.lower() -# Set the input charset after filtering through the aliases -self.input_charset = ALIASES.get(input_charset, input_charset) +# Set the input charset after filtering through its aliases defined in +# codecs library +try: +self.input_charset = lookup(input_charset).name +except LookupError: +self.input_charset = ALIASES.get(input_charset, input_charset) # We can try to guess which encoding and conversion to use by the # charset_map dictionary. Try that first, but let the user override # it. diff -r cc60d0283fad Lib/encodings/aliases.py --- a/Lib/encodings/aliases.py Fri May 20 16:55:06 2011 +0200 +++ b/Lib/encodings/aliases.py Sun May 22 14:18:05 2011 +0200 @@ -254,7 +254,7 @@ # hp_roman8 codec 'roman8' : 'hp_roman8', 'r8' : 'hp_roman8', -'csHPRoman8' : 'hp_roman8', +'cshproman8' : 'hp_roman8', # hz codec 'hzgb' : 'hz', @@ -298,6 +298,7 @@ 'iso_ir_157' : 'iso8859_10', 'l6' : 'iso8859_10', 'latin6' : 'iso8859_10', +'latin_6': 'iso8859_10', # iso8859_11 codec 'thai' : 'iso8859_11', @@ -308,6 +309,7 @@ 'iso_8859_13': 'iso8859_13', 'l7' : 'iso8859_13', 'latin7' : 'iso8859_13', +'latin_7': 'iso8859_13', # iso8859_14 codec 'iso_8859_14': 'iso8859_14', @@ -316,11 +318,13 @@ 'iso_ir_199' : 'iso8859_14', 'l8' : 'iso8859_14', 'latin8' : 'iso8859_14', +'latin_8': &
[issue8898] The email package should defer to the codecs module for all aliases
Changes by Michele Orrù : Removed file: http://bugs.python.org/file22065/issue8898_skip.patch ___ Python tracker <http://bugs.python.org/issue8898> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue8898] The email package should defer to the codecs module for all aliases
Changes by Michele Orrù : Added file: http://bugs.python.org/file22066/issue8898_skip.patch ___ Python tracker <http://bugs.python.org/issue8898> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10424] better error message from argparse when positionals missing
Michele Orrù added the comment: This issue seems already fixed. File: Lib/argparse.py 922 # if we didn't use all the Positional objects, there were too few 1923 # arg strings supplied. 1924 if positionals: 1925 self.error(_('too few arguments')) 1926 1927 # make sure all required actions were present 1928 for action in self._actions: 1929 if action.required: 1930 if action not in seen_actions: 1931 name = _get_action_name(action) 1932 self.error(_('argument %s is required') % name) -- nosy: +ezio.melotti, maker ___ Python tracker <http://bugs.python.org/issue10424> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10424] better error message from argparse when positionals missing
Michele Orrù added the comment: The attached patch solves this issue. I haven't added any unittest because test_argparse.py is quite huge - over 4300 lines-, and I was undecided between «ArgumentError tests» (4251) and «ArgumentTypeError tests» (4262). Any hint? However, file bug10424.py reproduces this bug. -- keywords: +patch Added file: http://bugs.python.org/file19646/issue10424.patch ___ Python tracker <http://bugs.python.org/issue10424> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10424] better error message from argparse when positionals missing
Changes by Michele Orrù : Added file: http://bugs.python.org/file19647/bug10424.py ___ Python tracker <http://bugs.python.org/issue10424> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10453] Add -h/--help option to compileall
Michele Orrù added the comment: I'm still working on this task; the attachment shows how I'm solving the bug. The patch is NOT yet completed, there are some problems with the unittests. Hoping that Eric will give me a help soon. -- keywords: +patch nosy: +ezio.melotti, maker Added file: http://bugs.python.org/file19679/issue10453.patch ___ Python tracker <http://bugs.python.org/issue10453> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10453] Add -h/--help option to compileall
Changes by Michele Orrù : Removed file: http://bugs.python.org/file19679/issue10453.patch ___ Python tracker <http://bugs.python.org/issue10453> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10453] Add -h/--help option to compileall
Michele Orrù added the comment: The new attached patch passes the unittest. -- Added file: http://bugs.python.org/file19682/issue10453.patch ___ Python tracker <http://bugs.python.org/issue10453> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10453] Add -h/--help option to compileall
Changes by Michele Orrù : Removed file: http://bugs.python.org/file19682/issue10453.patch ___ Python tracker <http://bugs.python.org/issue10453> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10453] Add -h/--help option to compileall
Changes by Michele Orrù : Added file: http://bugs.python.org/file19684/issue10453.patch ___ Python tracker <http://bugs.python.org/issue10453> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10453] Add -h/--help option to compileall
Michele Orrù added the comment: Eric, the unittests in Lib/test/test_compileall.py seems quite consistent to me, so for now I won't add anything. About adding a method for testing the '-h' argument, now that Lib/compileall.py uses argparse, it sounds trivial. EDIT: Kotan, I'm still of the same opinion. Anyway, I think you should use as template test.test_compileall.CommandLineTests.test_legacy_paths: so, check for returncode and use subprocess.call instead of subprocess.Popen. -- ___ Python tracker <http://bugs.python.org/issue10453> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10453] Add -h/--help option to compileall
Changes by Michele Orrù : Removed file: http://bugs.python.org/file19684/issue10453.patch ___ Python tracker <http://bugs.python.org/issue10453> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10453] Add -h/--help option to compileall
Michele Orrù added the comment: Unittest added; should be enough. -- Added file: http://bugs.python.org/file19702/issue10453.patch ___ Python tracker <http://bugs.python.org/issue10453> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10453] Add -h/--help option to compileall
Changes by Michele Orrù : Added file: http://bugs.python.org/file19703/issue10453_tests.patch ___ Python tracker <http://bugs.python.org/issue10453> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10453] Add -h/--help option to compileall
Changes by Michele Orrù : Added file: http://bugs.python.org/file19708/issue10453_final.patch ___ Python tracker <http://bugs.python.org/issue10453> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10453] Add -h/--help option to compileall
Michele Orrù added the comment: Yes, I was discussing about that on IRC. That's a matter of platform -on mine for example works-. He gave me a hand in solving this failure; now -I think- he's gonna apply that. -- ___ Python tracker <http://bugs.python.org/issue10453> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10453] Add -h/--help option to compileall
Michele Orrù added the comment: *discussing that on IRC with R. David Murray -- ___ Python tracker <http://bugs.python.org/issue10453> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10453] Add -h/--help option to compileall
Michele Orrù added the comment: Sorry. -- Added file: http://bugs.python.org/file19717/issue10453_noargs.patch ___ Python tracker <http://bugs.python.org/issue10453> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10453] Add -h/--help option to compileall
Michele Orrù added the comment: Yeah, maybe your is more readable. I suppose that failure was due to some missing arguments when calling compileall (line 225). The attached patch should fix this issue, but currently I have no Windows machines where to test. -- Added file: http://bugs.python.org/file19728/issue10453_quiet.patch ___ Python tracker <http://bugs.python.org/issue10453> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10424] better error message from argparse when positionals missing
Michele Orrù added the comment: Unittest added. -- Added file: http://bugs.python.org/file19729/issue10424.patch ___ Python tracker <http://bugs.python.org/issue10424> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10424] better error message from argparse when positionals missing
Changes by Michele Orrù : Removed file: http://bugs.python.org/file19646/issue10424.patch ___ Python tracker <http://bugs.python.org/issue10424> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10424] better error message from argparse when positionals missing
Michele Orrù added the comment: Ezio reviewed my patch; here there's the new version with some improvements. -- Added file: http://bugs.python.org/file19736/issue10424.patch ___ Python tracker <http://bugs.python.org/issue10424> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10424] better error message from argparse when positionals missing
Changes by Michele Orrù : Removed file: http://bugs.python.org/file19729/issue10424.patch ___ Python tracker <http://bugs.python.org/issue10424> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10453] Add -h/--help option to compileall
Michele Orrù added the comment: Thank you Stefan, these days I was a little busy and I hadn't the time to review my patch. I really appreciate you help. -- ___ Python tracker <http://bugs.python.org/issue10453> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue1100562] deepcopying listlike and dictlike objects
Michele Orrù added the comment: Unit tests added; tested both on python2.6 and python2.7. -- nosy: +maker versions: +Python 2.6 Added file: http://bugs.python.org/file17319/issue1100562.patch ___ Python tracker <http://bugs.python.org/issue1100562> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue13866] {urllib, urllib.parse}.urlencode should not use quote_plus
Changes by Michele Orrù : -- nosy: +maker ___ Python tracker <http://bugs.python.org/issue13866> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue1531415] parsetok.c emits warnings by writing to stderr
Michele Orrù added the comment: I am interested in solving this bug. If I understood correctly, that should be just a matter of spitting over sys.stdout whenever the user imports warnings.py, sys.stderr otherwise. Rewriting the code in C for python3.x would still be appreciated? -- nosy: +eric.araujo, ezio.melotti, maker ___ Python tracker <http://bugs.python.org/issue1531415> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue1531415] parsetok.c emits warnings by writing to stderr
Michele Orrù added the comment: For what I saw these days, Parser/tokenizer.c should import warnings.h (in order to use PyErr_WarnEx()), but Python/_warnings.c imports Python.h, that requires pgen ready. This leads to a circular dependency. Am I wrong / missing something? -- ___ Python tracker <http://bugs.python.org/issue1531415> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue1531415] parsetok.c emits warnings by writing to stderr
Michele Orrù added the comment: May somebody check for this? Otherwise the bug could be considered invalid. -- ___ Python tracker <http://bugs.python.org/issue1531415> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue8739] Update to smtpd.py to RFC 5321
Michele Orrù added the comment: I'm currently working on this issue. A little cleanup would be appreciated, or it would be better to split that on another issue? For what I saw, tests are in the form FooTest instead of TestFoo, smtpd imports modules used only in __main__, warnings can be handled the appropriate module, __import__ shall not be used. -- nosy: +maker ___ Python tracker <http://bugs.python.org/issue8739> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue8739] Update to smtpd.py to RFC 5321
Michele Orrù added the comment: Patch attached. A few considerations: in case of syntax error, the server responds with " MAIL FROM: [SP ] " according to http://tools.ietf.org/html/rfc5321#section-3.3 (instead of "MAIL FROM:"). Note that this could break something, as far as backwards compatibility is concerned. Looking at http://tools.ietf.org/html/rfc3030#section-4.2 , I was wondering whether the size parameter on MAIL FROM should have implications on the RCPT TO command (i.e., we should check that the foreign email accepts that size). Finally, right now the size parameter is not considered until greater than max_message_size. This could let the user push an email with a size greater than the on declared (but smaller than max_message_size). Tests for command line and cleanup will be on another issue. Waiting for directives :) , -- ù -- Added file: http://bugs.python.org/file24793/issue8739.patch ___ Python tracker <http://bugs.python.org/issue8739> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue1531415] parsetok.c emits warnings by writing to stderr
Michele Orrù added the comment: > This is not the proper workflow for bug tracking. "No one is working > on this right now" is not the same as "This bug is invalid". No one > worked on this ticket almost *seven years* after I filed it. You are right. Sorry for that. -- ___ Python tracker <http://bugs.python.org/issue1531415> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14261] Cleanup in smtpd module
New submission from Michele Orrù : This issue is related to 8739 . As pointed in http://bugs.python.org/issue8739#msg155385, « tests are in the form FooTest instead of TestFoo, smtpd imports modules used only in __main__, warnings can be handled the appropriate module, __import__ shall not be used » and, as bitdancer said in http://bugs.python.org/issue8739#msg153244 «there are no tests for the smtpd command line functionality» Also, note that currently the main has a bug: s/options\.max_message_size/size_limit/ -- components: Library (Lib) messages: 155448 nosy: Juhana.Jauhiainen, ezio.melotti, maker, r.david.murray priority: normal severity: normal status: open title: Cleanup in smtpd module type: enhancement versions: Python 3.3 ___ Python tracker <http://bugs.python.org/issue14261> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue8739] Update to smtpd.py to RFC 5321
Michele Orrù added the comment: David: yes, I did. About two weeks ago. Probably I'll take a look to those issues :) -- ___ Python tracker <http://bugs.python.org/issue8739> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue8739] Update to smtpd.py to RFC 5321
Michele Orrù added the comment: David, can you tag this issue as dependency for issue14261 ? -- ___ Python tracker <http://bugs.python.org/issue8739> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11959] smtpd cannot be used without affecting global state
Changes by Michele Orrù : -- nosy: +maker ___ Python tracker <http://bugs.python.org/issue11959> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14333] queue unittest errors
Changes by Michele Orrù : -- nosy: +maker ___ Python tracker <http://bugs.python.org/issue14333> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14333] queue unittest errors
Michele Orrù added the comment: This bug is becouse of Lib/unittest/loader.py:107 . So a quick fix would be to inherit from unittest.TestCase only in the exposed classes; but probably using a class decorator is the best solution. -- keywords: +patch Added file: http://bugs.python.org/file24910/issue14333_decorators.patch ___ Python tracker <http://bugs.python.org/issue14333> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14333] queue unittest errors
Changes by Michele Orrù : Added file: http://bugs.python.org/file24911/issue14333.patch ___ Python tracker <http://bugs.python.org/issue14333> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14364] Argparse incorrectly handles '--'
New submission from Michele Orrù : http://docs.python.org/library/argparse.html#arguments-containing The attached file shows different behaviours when using '--' immediately after an optional argument. tumbolandia:cpython maker$ python foo.py --test=-- foo [] tumbolandia:cpython maker$ python foo.py --test -- foo usage: foo.py [-h] [-t TEST] [yuri] foo.py: error: argument -t/--test: expected 1 argument(s) The same is for single-dash arguments. tumbolandia:cpython maker$ python foo.py -t -- foo usage: foo.py [-h] [-t TEST] [yuri] foo.py: error: argument -t/--test: expected 1 argument(s) tumbolandia:cpython maker$ python foo.py -t-- foo [] Obviously argparse should return an error in both cases. The bug is probably due to Lib/argparser.py:2211 -- files: foo.py messages: 156254 nosy: maker priority: normal severity: normal status: open title: Argparse incorrectly handles '--' versions: Python 2.7, Python 3.1 Added file: http://bugs.python.org/file24926/foo.py ___ Python tracker <http://bugs.python.org/issue14364> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue14364] Argparse incorrectly handles '--'
Michele Orrù added the comment: +1 also for me. I will try to work for a patch in the next days. :) -- ___ Python tracker <http://bugs.python.org/issue14364> ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue36986] tarfile: unexpected IsADirectoryError on extraction
Michele Angrisano added the comment: It looks like it has the same behavior of issue8958. -- nosy: +mangrisano ___ Python tracker <https://bugs.python.org/issue36986> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue36713] duplicate method definition in Lib/ctypes/test/test_unicode.py
Michele Angrisano added the comment: That method was already removed in cf44883. -- nosy: +mangrisano ___ Python tracker <https://bugs.python.org/issue36713> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue36713] duplicate method definition in Lib/ctypes/test/test_unicode.py
Michele Angrisano added the comment: The proper link is this: cf448832ebca7ed34809168660fa96c3c61f8abb. Sorry. -- ___ Python tracker <https://bugs.python.org/issue36713> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue36713] duplicate method definition in Lib/ctypes/test/test_unicode.py
Michele Angrisano added the comment: I'm on it. -- ___ Python tracker <https://bugs.python.org/issue36713> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue36713] duplicate method definition in Lib/ctypes/test/test_unicode.py
Change by Michele Angrisano : -- keywords: +patch pull_requests: +13440 stage: needs patch -> patch review ___ Python tracker <https://bugs.python.org/issue36713> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue37014] [First easy issue] fileinput module should document that openhook and mode are ignored when reading from stdin
Michele Angrisano added the comment: The Python's version chosen for this issue is 3.7. I think the suggest can be useful for 3.8 as well. Am I right? -- nosy: +mangrisano ___ Python tracker <https://bugs.python.org/issue37014> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue37014] [First easy issue] fileinput module should document that openhook and mode are ignored when reading from stdin
Change by Michele Angrisano : -- keywords: +patch pull_requests: +13457 stage: -> patch review ___ Python tracker <https://bugs.python.org/issue37014> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue37014] [First easy issue] fileinput module should document that openhook and mode are ignored when reading from stdin
Michele Angrisano added the comment: I've just made a PR for this issue. -- ___ Python tracker <https://bugs.python.org/issue37014> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue36461] timeit: Additional changes for autorange
Michele Angrisano added the comment: I agree with *target_time*. I'm working on it and soon I'm going to update the pr. -- nosy: +mangrisano ___ Python tracker <https://bugs.python.org/issue36461> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue18564] Fix Bluetooth address parser
Change by Michele Orrù : -- title: Integer overflow in the socket function parsing a Bluetooth address -> Fix Bluetooth address parser ___ Python tracker <https://bugs.python.org/issue18564> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue37074] os.stat() does not work for NUL and CON
Michele Angrisano added the comment: I've tried to reproduce this behavior on my Mac with python3.8 and python 3.7 but I couldn't. If 'nul' doesn't exist, it raises a FileNotFound exception as it should do. If 'nul' exists, it shows me the right output. Same behavior with 'con'. -- nosy: +mangrisano ___ Python tracker <https://bugs.python.org/issue37074> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue19184] dis module has incorrect docs for RAISE_VARARGS
Michele Angrisano added the comment: I'm working on it. -- nosy: +mangrisano ___ Python tracker <https://bugs.python.org/issue19184> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue19184] dis module has incorrect docs for RAISE_VARARGS
Change by Michele Angrisano : -- keywords: +patch pull_requests: +13544 stage: needs patch -> patch review pull_request: https://github.com/python/cpython/pull/13652 ___ Python tracker <https://bugs.python.org/issue19184> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue37086] time.sleep error message misleading
Michele Angrisano added the comment: The doc (3.7) says that the argument "may be a floating point number to indicate a more precise sleep time." I think that TypeError message is right because you can choose how much precision you need but it's optional. What do you think about that? -- nosy: +mangrisano ___ Python tracker <https://bugs.python.org/issue37086> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue37074] os.stat() does not work for NUL and CON
Change by Michele Angrisano : -- nosy: -mangrisano ___ Python tracker <https://bugs.python.org/issue37074> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue37086] time.sleep error message misleading
Change by Michele Angrisano : -- nosy: -mangrisano ___ Python tracker <https://bugs.python.org/issue37086> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue36986] tarfile: unexpected IsADirectoryError on extraction
Change by Michele Angrisano : -- nosy: -mangrisano ___ Python tracker <https://bugs.python.org/issue36986> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue37150] Do not allow to pass FileType class object instead of instance in add_argument
Michele Angrisano added the comment: Reading the examples in the doc, it's clear the behavior when FileType takes an argument. What's the behavior of FileType when is called without any argument? -- nosy: +mangrisano ___ Python tracker <https://bugs.python.org/issue37150> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue37150] Do not allow to pass FileType class object instead of instance in add_argument
Michele Angrisano added the comment: Yes, I meant that. Thanks! :) -- ___ Python tracker <https://bugs.python.org/issue37150> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com