Senthil Kumaran added the comment:
This is in 3.3. Thanks for the patches.
--
nosy: +orsenthil
resolution: -> fixed
status: open -> closed
___
Python tracker
<http://bugs.python.org/i
Changes by Senthil Kumaran :
--
nosy: +orsenthil
___
Python tracker
<http://bugs.python.org/issue11012>
___
___
Python-bugs-list mailing list
Unsubscribe:
Senthil Kumaran added the comment:
Fixed this in 2.7.
--
nosy: +orsenthil
resolution: -> fixed
status: open -> closed
___
Python tracker
<http://bugs.python.org/i
Senthil Kumaran added the comment:
Updated the doc with Sandro Tosi's suggested changes in all the codelines.
--
nosy: +orsenthil
resolution: -> fixed
status: open -> closed
___
Python tracker
<http://bugs.python.
Senthil Kumaran added the comment:
The original issue was invalid.
Incorporated Michele Orrù's code style changes into the trunk and other
codelines.
--
resolution: -> fixed
status: open -> closed
title: http.server: HEAD request should not return a body -> http.serve
Senthil Kumaran added the comment:
The concern here is if the request line had something like this.
Method SP Request-URI SP HTTP-Version \r\n
The previous behavior would have resulted in
Method SP Request-URI SP HTTP-Version
That is removing only the final \r\n, whereas the
Senthil Kumaran added the comment:
Here is the patch against 3.2. Something is wrong with the mercurial at the
moment where 3.2 is shown as inactive. Once that is corrected, I shall commit
and push this to 3.2 and cpython codelines.
This bug is not present in 2.7 as different logic is
Changes by Senthil Kumaran :
--
resolution: -> fixed
stage: test needed -> committed/rejected
status: open -> closed
___
Python tracker
<http://bugs.python.or
Senthil Kumaran added the comment:
Some review comments. Instead of doing the inline unquote like this -
-auth = base64.b64encode(user_passwd).strip()
+auth = base64.b64encode(unquote(user_passwd)).strip()
It is better to do the explicitly above the b64 encoding step
Senthil Kumaran added the comment:
Sorry, this escaped for so long. First thing is, having different values for
HTTP_PROXY and HTTP_proxy and http_proxy is plain wrong at the user side. They
all should be same and in that case the possible problem which Kirikaza and
Neil Muller state is
Senthil Kumaran added the comment:
Joonas and Michele - The fix along with the tests is in for 2.7 line. W.r.t to
tests having it in the class level buf seems to be only easy way, for other it
seemed to be that FakeSocket and FakeConnection stuff need some major change
(and resulted in
Senthil Kumaran added the comment:
This is fixed. Thanks for your contribution.
--
resolution: -> fixed
stage: needs patch -> committed/rejected
status: open -> closed
___
Python tracker
<http://bugs.python.o
Senthil Kumaran added the comment:
Justin,
The patch and logic is okay. We can have this is 3.3.
- I find that loop_actions as not appropriate name for the new method. It fails
to give a intuitive meaning of what is supposed to do. request_action,
request_action_continued or anything else
Senthil Kumaran added the comment:
The feature is in 3.3. I did not remove the handle_timeout method from the
Mixin class, because it might be used in the production by the existing
servers. It is not appropriate to remove methods all of sudden without
deprecation warnings and also it is not
Senthil Kumaran added the comment:
This is fixed in 3.3 now. Keeping it open for test_socketserver update. After a
ForkingServer service call, it should be asserted the collect_children routine
is run.
--
resolution: -> fixed
stage: patch review -> committed/rejected
ve
Senthil Kumaran added the comment:
Victor - Sure, I understand Issue #xxx: desc must be useful while generation
reST docs.
--
___
Python tracker
<http://bugs.python.org/issue11
New submission from Senthil Kumaran :
socketserver.ForkingMixin class has a collect_children method, that waits for
the children to exit.
But, that waits for any child process and not just the ones spawned the
socketserver alone.
try:
pid, status = os.waitpid(0, 0
Changes by Senthil Kumaran :
--
nosy: +charles-francois.natali
___
Python tracker
<http://bugs.python.org/issue12184>
___
___
Python-bugs-list mailing list
Unsub
Senthil Kumaran added the comment:
I think, if you speak to RM, you can just have this change in. Don't we get "no
commits please" email request from RM with hg or is the branch already cut?
--
___
Python tracker
<http://bugs.pyt
Senthil Kumaran added the comment:
Ori, which platform did you try to reproduce this issue. I tried in all active
codelines (cpython to all through 2.5) from hg and can't able to reproduce this
bug on Linux. If someone can reproduce, can you provide exact instruc
Senthil Kumaran added the comment:
+libpython*.so* is fine, but please don't remove the rej and orig. By
convention and practice, those files are meant to be in .ignore files to
prevent accidental checkins. Usually when the conflict occurs you are notified
immediately and you fix
Senthil Kumaran added the comment:
I strictly have not faced any problem with those rej and orig files being
present in the .hgignore. I also thought keeping the .hgignore in the repo was
for distribution and never knew that it can affect local views (If you remove
.rej and .orig your local
Senthil Kumaran added the comment:
By "for distribution", I meant that everyone who checks out ( clones)
from hg.cpython.org repository share the same .hgignore file, instead
of each person having to create their own file.
I got an impression from the earlier message that hg
Senthil Kumaran added the comment:
I tried to a line .hgignore inside .hgignore file, so that it can ignore itself
(weird concept, but wanted to try if it works) and users can make local
customization of it without any problem. This has been suggest at many places
in the Internet. I tried
Senthil Kumaran added the comment:
well, urlopen does return an file like object for socket connection which
closes itself when it goes out of scope, as you raised this bug, I think a more
explicit context manager like behavior can be tried. But I am afraid that it
would complex to implement
Senthil Kumaran added the comment:
We had support for chunked transfer encoding for POST method recently, which is
exposed via urllib2 wrapper function. PUT is not exposed via urllib2 and users
should use httplib. This feature of chunked transfer can be added to PUT by
taking the body of the
Senthil Kumaran added the comment:
Hello Stephane,
> I use HTTPConnection to simulate Apple Finder WebDAV client. When
> this WebDAV client do PUT request, it transmit data in chunked
> encoding mode and not set Content-Length HTTP field.
>
> Do you understand my context ?
I
Senthil Kumaran added the comment:
Both the changes suggested in the patch are fine. I think, it is okay to remove
the executable function. It is undocumented as it is, and have not seen any
standalone use of it. A note in NEWS can help.
--
nosy: +orsenthil
Senthil Kumaran added the comment:
I forgot completely, but in Python3, Issue5418 had already added
support to addinfourl. It is now possible to write code like
import urllib.request
with urllib.request.urlopen('http://www.python.org') as req:
res = req.read()
But yeah, unl
Senthil Kumaran added the comment:
Yes, it is a feature. Sorry that I have not paid attention to this. The Windows
(IIS) part led me to delay as I did not have any to test. Let me take this up
and see through it in 3.3.
--
___
Python tracker
Senthil Kumaran added the comment:
I see you missed the doseq parameter which is applicable in this case.
print(urllib.parse.urlencode(dictQuery, encoding='iso8859-1',doseq=True))
a=b&c=d&c=e&b=c
--
nosy: +orsenthil
resolution: -> inval
Senthil Kumaran added the comment:
Joseph, resolution applies for the bug not the process. tc. :)
--
resolution: accepted -> invalid
___
Python tracker
<http://bugs.python.org/issu
Senthil Kumaran added the comment:
Sorin, can you please open another report with more details and how some
condition in httplib breaks PyAMF. We will see through that it is fixed.
Commenting on an invalid closed issue is confusing.
--
___
Python
Senthil Kumaran added the comment:
Hi Sorin,
On Sat, Jun 25, 2011 at 07:54:24PM +, sorin wrote:
> type(message_body) is str ... even if I tried to manually force
> Python for use bytes. It seams that in 2.7 bytes are alias to str.
> Due to this the code will fail to run only on 2.
Senthil Kumaran added the comment:
Eric, sure go ahead. As I am not affected by remains of .rej and .orig (neither
have I found their usefulness (yet)), so my concerns can be stepped over. Plus,
I have found a way to ignore files via patterns in .hgrc.
I already see you and RDM proposing the
Senthil Kumaran added the comment:
Fixed in branch 2.7. Thanks Kristan. Eric, the documentation patch does not
apply to 3.1 because those functions are not present in 3.1.
--
nosy: +orsenthil
status: open -> closed
___
Python tracker
&l
Senthil Kumaran added the comment:
On Sat, Jun 25, 2011 at 08:38:04PM +, Senthil Kumaran wrote:
> As I am not affected by remains of .rej and .orig (neither have I
> found their usefulness (yet)), so my concerns can be stepped over.
Just a side-note. I wondered why is my case so,
Senthil Kumaran added the comment:
The module documentation for the SimpleHTTPServer class says that.
"""
This class serves files from the current directory and below, directly
mapping the directory structure to HTTP requests.
"""
As it is already docume
Senthil Kumaran added the comment:
On Sun, Jun 26, 2011 at 10:44:59AM +, Mark Mc Mahon wrote:
> So we need a doc patch? Should we update the older 'What's new in
> Python 2.x' for added attribs?
The Documentation patch may be applied only 2.7 and 3.2/3.3 releases
now.
Senthil Kumaran added the comment:
On Sun, Jun 26, 2011 at 03:12:30PM +, Sandro Tosi wrote:
> The patch is fine: but would you be interested in trying to write a
> unittest for select.epoll.register ? it would be really nice to
This is covered in test_epoll.py
--
nosy: +ors
Senthil Kumaran added the comment:
Eric, you mean for the whole of sqlite3 docmentation or the part in the
footnote?
--
___
Python tracker
<http://bugs.python.org/issue10
Senthil Kumaran added the comment:
Yes, I agree with you. Good Suggestion. Thanks!
--
___
Python tracker
<http://bugs.python.org/issue10403>
___
___
Python-bug
Senthil Kumaran added the comment:
As Santosa said, this was discovered and fixed in 2.7 itself, surprised that
you discovered it in any release Jon. If it was from a checkout, please update.
--
resolution: -> invalid
stage: -> committed/rejected
status: open -&g
Senthil Kumaran added the comment:
AFAIR, that capitalize part is somewhere a requirement in RFC, if the server
did not behave in proper manner, it may not be a good idea for the client to
change (or be permissive the flag).
--
nosy: +orsenthil
Senthil Kumaran added the comment:
Sorry, not "Capitalize", but the "Title" part. One can some bugs which lead to
this change in the urllib2.
--
___
Python tracker
<http://bug
Senthil Kumaran added the comment:
Fixed the indentation issues. We could not use automatic reST
footnotes because two links refer to same footnote, so left it as
such.
--
___
Python tracker
<http://bugs.python.org/issue10
Senthil Kumaran added the comment:
Hello Stefen,
Yes, I was able to reproduce the issue without the patch and at the
same time, noticed that the problem was not observed in other branches
(py3k), then saw the differences where the possible socket close
delays can happen and fixed it. As it
Senthil Kumaran added the comment:
What's new docs are usually for what is coming up new in the upcoming
releases. It is not updated once the release is done. Bug fixes and
related docs are updated in documentation.
--
___
Python tracker
I am not sure that is comprehensive enough. It is just organized into
OS specific sections and gives little details further on.
So, -1 to this, but +1 to the general idea of having detailed install
and get started documentation.
___
Python-bugs-list maili
Have fixed this. Even looking at the code, your suggestion was
correct. But even before the fix, I was not able to reproduce the bug
in Ubuntu and perhaps it was getting masked. May I ask, how did you
invoke idle that this bug surfaced and what was the platform?
Changes by Senthil Kumaran :
--
resolution: -> fixed
stage: patch review -> committed/rejected
status: open -> closed
___
Python tracker
<http://bugs.python.or
Changes by Senthil Kumaran :
--
nosy: +orsenthil
___
Python tracker
<http://bugs.python.org/issue12484>
___
___
Python-bugs-list mailing list
Unsubscribe:
Senthil Kumaran added the comment:
It should be documented that in 2.x series the file-like object does not
support context management protocol. I have added the superseder issue number,
please see the note as if you really want in 2.x, how you can have it using
contextlib and in 3.x the
New submission from Senthil Kumaran :
Update the term 'attribute' in the glossary
http://docs.python.org/dev/glossary.html#term-attribute so that the reader can
understand that the term attribute in Python can mean both 'data-attribute' and
a 'callable' meth
Changes by Senthil Kumaran :
--
nosy: +orsenthil
___
Python tracker
<http://bugs.python.org/issue12149>
___
___
Python-bugs-list mailing list
Unsubscribe:
Senthil Kumaran added the comment:
With the patch applied, test_urllib2net fails at test_ftp test case
when a valid and invalid url are presented in sequence. I think test
needs a change or a further look is needed at the patch.
--
___
Python
Senthil Kumaran added the comment:
Is this not implicit? Do we really need this stmt? I am -0 on this. If you
agree, you can close this issue.
--
nosy: +orsenthil
title: Clarify struct doc for format 's'. -> Clarify struct doc for format 's',
when it is me
Senthil Kumaran added the comment:
That is for the string formatting character, correct?
In this issue, we are talking about format character for struct and
not just 's' but 'c','h','b','i&#
Changes by Senthil Kumaran :
--
assignee: -> orsenthil
nosy: +orsenthil
___
Python tracker
<http://bugs.python.org/issue12541>
___
___
Python-bugs-list mai
Senthil Kumaran added the comment:
Hello Eric, I missed noticing Alexander's comments in the reitveld, I looked
only at tracker then. I see that some of them can be addressed. Like using
members (components) of the field, instead of attributes when it is not an
attribute. Shall corre
Senthil Kumaran added the comment:
Can you please provide an example snippet with your expectation of the behavior?
--
assignee: -> orsenthil
nosy: +orsenthil
___
Python tracker
<http://bugs.python.org/issu
Also, it does not say or "advertise" about POST, it gives examples for
GET. I find Ezio's suggestion using bugs.python.org as a good one.
___
Python-bugs-list mailing list
Unsubscribe:
http://mail.python.org/mailman/options/python-bugs-list/archive%40mai
Senthil Kumaran added the comment:
On Sun, Jul 17, 2011 at 12:07:44AM +, Ugra Dániel wrote:
> For example IMDb's "Cneonction: close" (not a typo) feature. But
This is a mistake at the server and urllib relies on the
Connection: close header at some point in time in
TypeError should be okay. But I am still -0 on that. It would be good
to hear a strong argument from the user that how did he end up passing
unicode to shlex.split? It is for parsing command line args for
programs and personally have not seen those cases. Or did he want
unicode everywhere if we was
Changes by Senthil Kumaran :
--
status: open -> closed
___
Python tracker
<http://bugs.python.org/issue10403>
___
___
Python-bugs-list mailing list
Unsubscri
Senthil Kumaran added the comment:
I would be +1 if the basic example also highlights setUp and tearDown methods.
Those are useful ones for a new comer to know via an example snippet and not
just with explanation.
--
nosy: +orsenthil
___
Python
Senthil Kumaran added the comment:
Should information on Windows behavior, which you experienced be included too?
How about -
It uses :func:`os.rename` to perform the move. If that fails, for example
because src and dst are on different filesystems or in case of Windows where
rename cannot
Senthil Kumaran added the comment:
Would you like to give an example snippet (server+client) which can help
reproduce this behavior?
--
assignee: -> orsenthil
nosy: +orsenthil
___
Python tracker
<http://bugs.python.org/issu
Senthil Kumaran added the comment:
Not a problem. I am closing this, but if you find enough evidence that
something is a bug, feel free to reopen this or another report.
--
resolution: -> invalid
stage: -> committed/rejected
status: open -&g
Changes by Senthil Kumaran :
Removed file: http://bugs.python.org/file22690/unnamed
___
Python tracker
<http://bugs.python.org/issue12570>
___
___
Python-bugs-list mailin
Oh, I thought we never rely on exception "message" for anything
important. However this seems to be an exception for that exception.
:-)
___
Python-bugs-list mailing list
Unsubscribe:
http://mail.python.org/mailman/options/python-bugs-list/archive%40mai
On Mon, Jul 18, 2011 at 02:07:42PM +, Éric Araujo wrote:
> I’d put the class definition just before the methods. (I would even
> refactor the reST to use nested class/method combo...
This is a good suggestion. It would good to do some point in time soon.
Thanks!
__
Senthil Kumaran added the comment:
Hi Petter, writing tests are ofcourse a good way to start. As long as the tests
increase the coverage, those are most welcome. Thanks!
--
assignee: -> orsenthil
___
Python tracker
<http://bugs.pyth
Hello Eric,
On Tue, Jul 19, 2011 at 01:05:59PM +, Éric Araujo wrote:
> Alternate idea: use example.org. People won’t be able to actually
> run the example, but is it really important?
The whole idea is present a usable example. Please don't suggest
unusable example. example.org does not sup
Senthil Kumaran added the comment:
Thanks for the patch, Bharadwaj.
--
___
Python tracker
<http://bugs.python.org/issue12524>
___
___
Python-bugs-list mailin
Changes by Senthil Kumaran :
--
assignee: -> orsenthil
___
Python tracker
<http://bugs.python.org/issue12576>
___
___
Python-bugs-list mailing list
Unsubscri
Senthil Kumaran added the comment:
I am against hacks like tion: close. Under worst case, we shall revert
the change which caused this regression in the first place.
--
___
Python tracker
<http://bugs.python.org/issue12
Senthil Kumaran added the comment:
On Sat, Jul 23, 2011 at 09:37:27AM +, Éric Araujo wrote:
> I’ve had a look at the docstring and the reST docs, and they clearly
> say that sequences are supported, not arbitrary iterables.
Yeah. At the first cut, when I saw the suggestion of ite
Senthil Kumaran added the comment:
Thanks a lot for the patch, Petter Haggholm.
I was initially hesitant to have separate tests for functions/methods
which are helper functions and not necessarily have documented api.
My thought was that those should be tested as part of the public api.
But
Senthil Kumaran added the comment:
These are not on head revision in cpython and 3.2. Where did you find this bug?
Please provide further details along with snippet and/or file.
--
nosy: +orsenthil
stage: needs patch ->
___
Python tracker
&l
Okay, got it. Thanks. I was a subtle one within the description. I
missed it in the first place.
___
Python-bugs-list mailing list
Unsubscribe:
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
Senthil Kumaran added the comment:
I propose the attached patch as fix to this issue. All it does is, moves the
code of getting http response to the finally block of the http request. It
closes the sockets if the getting the response fails for some reason, otherwise
it proceeds normally
urlretrieve is a helper function from urllib module. The way to use
it is:
>>> import urllib.request
>>> urllib.request('http://bugs.python.org')
('/tmp/tmpe873xe', )
>>> import os
>>> os.stat('/tmp/tmpe873xe')
posix.stat_result(st_mode=33152, st_ino=4462517, st_dev=2054,
st_nlink=1, st_uid=1000,
Senthil Kumaran added the comment:
urlretrieve is a helper function from urllib module. The way to use
it is:
('/tmp/tmpe873xe', )
>>> import os
>>> os.stat('/tmp/tmpe873xe')
Works!
--
___
Python track
Senthil Kumaran added the comment:
Tracker stripped off the code. Here is it.
import urllib.request
urllib.request.urlretrieve('http://bugs.python.org')
('/tmp/tmpe873xe', )
import os
os.stat('/tmp/tmpe873xe')
posix.stat_result(st_mode=33152, st_ino=4462517, st
Changes by Senthil Kumaran :
--
assignee: -> orsenthil
nosy: +orsenthil
___
Python tracker
<http://bugs.python.org/issue12480>
___
___
Python-bugs-list mai
Senthil Kumaran added the comment:
Thanks for the patch, Neil.
--
nosy: +orsenthil
___
Python tracker
<http://bugs.python.org/issue11439>
___
___
Python-bug
Senthil Kumaran added the comment:
When shutil.copy2 already says that it's behavior is equivalent to cp -p, will
adding this sentence
+ Symbolic links are not preserved. The contents and metadata of the
+ linked files are copied instead.
Not actively confuse the user? Because cp
Senthil Kumaran added the comment:
Cleartree seems more equivalent to a shell expression, as you pointed out to
find and -delete,than a utility. We might find those useful when work on CLI as
opposed to programmatically needing them via APIs. (Won't you agree?) Chin, can
you suggest ce
Changes by Senthil Kumaran :
--
title: [new function] shutil.cleartree -> Add a new shutil.cleartree function
to shutil module
___
Python tracker
<http://bugs.python.org/issu
Senthil Kumaran added the comment:
I propose we close this, because it adds a little value as a separate bug. If
there are specific sections in the docs that can be improved, we can deal with
that.
--
status: open -> pending
___
Python trac
Senthil Kumaran added the comment:
Thanks for the bug report, Popa Claudiu and Patch Santoso Wijaya.
Ouch, pretty ugly bug - shows the code lacked test coverage.
--
___
Python tracker
<http://bugs.python.org/issue12
Senthil Kumaran added the comment:
Petri, Thanks for the patch. It would also be helpful to track -
"shutil.copytree() use lutimes in Python 3.3 to copy symlink metadata if
symlinks=True". You can raise a feature request if it is not already raise
Senthil Kumaran added the comment:
Hi Ezio,
It was intentional that I did not merge it to default. We want
shutil.copystat to use lutimes which is available in 3.3 and in which
case, we will have to remove this once that change is done
Senthil Kumaran added the comment:
I propose that we remove 'accepted' as the meaning seems ambiguous and
it is of little practical (/tracking) use.
--
nosy: +orsenthil
___
Python tracker
<http://bugs.python.o
The orientation depends upon the mode, which is explained further
down.
http://docs.python.org/py3k/library/turtle.html#turtle.left
So, the correct fix would be:
"would turn clockwise/counterclockwise depending upon the mode."
___
Python-bugs-list maili
Senthil Kumaran added the comment:
I thought "fixed" was enough. A kind of binary state where we say, okay we have
it and otherwise no, we don't. I know for feature requests the terminology does
not make sense, but I believe I have worked with bug trackers where fixed was
the
The reason for that seems that svn scheme did not support relative paths
to being with (at least when urlparse was originally written).
>From 1.5 onwards (released sometime in 2008), it has the support for
relative urls and can work with urljoin (by giving relative paths).
http://subversion.apach
Senthil Kumaran added the comment:
Sandro, perhaps got overlooked. :) But yeah, why wait. Done.
--
nosy: +orsenthil
___
Python tracker
<http://bugs.python.org/issue11
Changes by Senthil Kumaran :
--
nosy: +markm
___
Python tracker
<http://bugs.python.org/issue12653>
___
___
Python-bugs-list mailing list
Unsubscribe:
301 - 400 of 2044 matches
Mail list logo
