[issue7946] Convoy effect with I/O bound threads and New GIL

2010-05-18 Thread Michele

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

2011-11-16 Thread Michele Orrù

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

2011-11-18 Thread Michele Orrù

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

2011-11-18 Thread Michele Orrù

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

2011-12-30 Thread Michele Orrù

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

2012-01-08 Thread Michele Orrù

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

2012-01-09 Thread Michele Orrù

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

2012-01-10 Thread Michele Orrù

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

2012-01-10 Thread Michele Orrù

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

2012-01-10 Thread Michele Orrù

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

2012-01-12 Thread Michele Orrù

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

2012-01-12 Thread Michele Orrù

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

2011-05-22 Thread Michele Orrù

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

2011-05-22 Thread Michele Orrù

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

2011-05-24 Thread Michele Orrù

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

2011-05-24 Thread Michele Orrù

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

2011-05-26 Thread Michele Orrù

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

2011-05-26 Thread Michele Orrù

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

2011-05-26 Thread Michele Orrù

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

2011-05-27 Thread Michele Orrù

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

2011-05-27 Thread Michele Orrù

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

2011-05-27 Thread Michele Orrù

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

2011-08-18 Thread Michele Orrù

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

2011-08-20 Thread Michele Orrù

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

2011-08-20 Thread Michele Orrù

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

2011-08-20 Thread Michele Orrù

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

2011-08-24 Thread Michele Orrù

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

2011-08-24 Thread Michele Orrù

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

2011-05-21 Thread Michele Orrù

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

2011-05-21 Thread Michele Orrù

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

2011-05-21 Thread Michele Orrù

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

2011-05-21 Thread Michele Orrù

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

2011-05-21 Thread Michele Orrù

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

2011-05-21 Thread Michele Orrù

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

2011-05-21 Thread Michele Orrù

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

2011-05-21 Thread Michele Orrù

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

2011-05-22 Thread Michele Orrù

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

2011-05-22 Thread Michele Orrù

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

2011-05-22 Thread Michele Orrù

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

2011-05-22 Thread Michele Orrù

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

2011-05-22 Thread Michele Orrù

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

2011-05-22 Thread Michele Orrù

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

2010-11-19 Thread Michele Orrù

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

2010-11-19 Thread Michele Orrù

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

2010-11-19 Thread Michele Orrù

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

2010-11-20 Thread Michele Orrù

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

2010-11-20 Thread Michele Orrù

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

2010-11-20 Thread Michele Orrù

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

2010-11-20 Thread Michele Orrù

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

2010-11-20 Thread Michele Orrù

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

2010-11-20 Thread Michele Orrù

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

2010-11-20 Thread Michele Orrù

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

2010-11-20 Thread Michele Orrù

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

2010-11-20 Thread Michele Orrù

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

2010-11-20 Thread Michele Orrù

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

2010-11-20 Thread Michele Orrù

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

2010-11-20 Thread Michele Orrù

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

2010-11-20 Thread Michele Orrù

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

2010-11-20 Thread Michele Orrù

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

2010-11-21 Thread Michele Orrù

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

2010-11-21 Thread Michele Orrù

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

2010-11-21 Thread Michele Orrù

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

2010-11-21 Thread Michele Orrù

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

2010-11-25 Thread Michele Orrù

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

2010-05-13 Thread Michele Orrù

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

2012-02-21 Thread Michele Orrù

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

2012-02-27 Thread Michele Orrù

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

2012-03-05 Thread Michele Orrù

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

2012-03-11 Thread Michele Orrù

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

2012-03-11 Thread Michele Orrù

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

2012-03-12 Thread Michele Orrù

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

2012-03-12 Thread Michele Orrù

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

2012-03-12 Thread Michele Orrù

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

2012-03-16 Thread Michele Orrù

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

2012-03-16 Thread Michele Orrù

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

2012-03-17 Thread Michele Orrù

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

2012-03-17 Thread Michele Orrù

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

2012-03-17 Thread Michele Orrù

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

2012-03-17 Thread Michele Orrù

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 '--'

2012-03-18 Thread Michele Orrù

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 '--'

2012-03-18 Thread Michele Orrù

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

2019-05-21 Thread Michele Angrisano


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

2019-05-23 Thread Michele Angrisano


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

2019-05-23 Thread Michele Angrisano


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

2019-05-23 Thread Michele Angrisano


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

2019-05-23 Thread Michele Angrisano


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

2019-05-24 Thread Michele Angrisano


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

2019-05-24 Thread Michele Angrisano


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

2019-05-24 Thread Michele Angrisano


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

2019-05-25 Thread Michele Angrisano


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

2019-05-26 Thread Michele Orrù

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

2019-05-28 Thread Michele Angrisano


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

2019-05-29 Thread Michele Angrisano


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

2019-05-29 Thread Michele Angrisano


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

2019-05-29 Thread Michele Angrisano


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

2019-06-02 Thread Michele Angrisano


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

2019-06-02 Thread Michele Angrisano


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

2019-06-02 Thread Michele Angrisano


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

2019-06-04 Thread Michele Angrisano


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

2019-06-04 Thread Michele Angrisano


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



  1   2   3   >