Senthil Kumaran sent...@uthcode.com added the comment:
On Mon, Jul 9, 2012 at 8:50 PM, Chris Jerdonek rep...@bugs.python.org wrote:
Sure, if someone is open to reviewing it.
Yes, please go ahead. I can review it and I believe others should be
able to as well. As a first short, I think, it may
Serhiy Storchaka storch...@gmail.com added the comment:
+sys.stdin = self.console = _RPCFile(self.get_remote_proxy(stdin))
write to sys.stdin?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13532
Chris Jerdonek chris.jerdo...@gmail.com added the comment:
Thanks, Senthil. That is my plan. I should be able to have code with tests in
no later than a week.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15302
Chris Jerdonek chris.jerdo...@gmail.com added the comment:
Patch attached.
I also added a note about building the Dev Guide documentation. Unlike the
main Python documentation, `make html` for the Dev Guide doesn't seem to
install Sphinx automatically.
--
keywords: +patch
Added
Changes by Chris Jerdonek chris.jerdo...@gmail.com:
--
stage: - patch review
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15311
___
___
Changes by Mark Dickinson dicki...@gmail.com:
--
nosy: +mark.dickinson
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15310
___
___
New submission from Tomi Pieviläinen tomi.pievilainen+launch...@iki.fi:
/usr/local/lib/python3.2/lib2to3/Grammar3.2.3.final.0.pickle is readable only
by root after installing with make install from the source package. This makes
at least installing distribute fail, and thus makes virtualenv,
Tomi Pieviläinen tomi.pievilainen+launch...@iki.fi added the comment:
Also PatternGrammar3.2.3.final.0.pickle is missing the read permissions.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15317
Chris Jerdonek chris.jerdo...@gmail.com added the comment:
By the way, issue 15300 has a related patch that is ready to review today.
Assuming that one is okay, it would make sense to commit first because it
overlaps with the changes I'll be doing here.
Issue 15305 is another related issue
Craig McQueen pyt...@craig.mcqueen.id.au added the comment:
That's definitely an improvement. It gets further, but on my PC, the compile
fails:
...
c:/mingw/bin/../lib/gcc/mingw32/4.5.0/../../../../mingw32/bin/ld.exe: cannot
find -lmsvcr100
collect2: ld returned 1 exit status
error: command
Amaury Forgeot d'Arc amaur...@gmail.com added the comment:
Looks very similar to issue15111.
--
nosy: +amaury.forgeotdarc
resolution: - duplicate
status: open - closed
superseder: - Wrong ImportError message with importlib
___
Python tracker
Changes by Chris Jerdonek chris.jerdo...@gmail.com:
--
nosy: +cjerdonek
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15111
___
___
Craig McQueen pyt...@craig.mcqueen.id.au added the comment:
I downloaded the latest MinGW, and now it tells me:
...
c:\mingw\bin\gcc.exe -mno-cygwin -mdll -O -Wall -IC:\Python33\include
-IC:\Python33\PC -c python3/src/_cobs_ext.c -o bui
ld\temp.win32-3.3\Release\python3\src\_cobs_ext.o
Craig McQueen pyt...@craig.mcqueen.id.au added the comment:
It sounds as though the option '-mno-cygwin' is related to issue #12641.
Does that mean I need to find a version of MinGW that is old enough to support
the option '-mno-cygwin', but new enough to include a library for msvcr100?
Craig McQueen pyt...@craig.mcqueen.id.au added the comment:
I've come across this issue when trying to build extensions for Python 3.3 on
Windows, needing a recent enough MinGW to provide a library for msvcr100. See
issue #15315.
--
nosy: +cmcqueen1975
Martin v. Löwis mar...@v.loewis.de added the comment:
I'm not a Cygwin user myself, so it would be good if you could figure this out
somehow. Feel free to ask on Cygwin lists how this is supposed to work. Our
requirement is that the resulting pyd needs to link with msvcr100.dll. It may
well
Stefan Krah stefan-use...@bytereef.org added the comment:
Could you please state the Python version and line numbers in
memoryobject.c?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15309
Martin v. Löwis mar...@v.loewis.de added the comment:
I have gcc 3.4.4 in my cygwin installation, and it still needs the -mno-cygwin
option, else the resulting binary will link with cygwin1.dll (which is
undesired).
--
___
Python tracker
Martin v. Löwis mar...@v.loewis.de added the comment:
I just installed the cygwin gcc4 package; this gives me gcc 4.5.3. In this
version, -mno-cygwin is still recognized, and gives this error message:
gcc: The -mno-cygwin flag has been removed; use a mingw-targeted cross-compiler.
So it seems
Tim Golden m...@timgolden.me.uk added the comment:
I can confirm that this isn't a problem for 3.x. No-one's listed in the experts
list for the httplib module but Senthil appears to be the de factor maintainer
so I've added him to the call for his opinion. My feeling would be to do the
Martin v. Löwis mar...@v.loewis.de added the comment:
The 64-bit compiler is actually called x86_64-w64-mingw32-gcc.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12641
___
Antoine Pitrou pit...@free.fr added the comment:
I don't really like the pre-parsed option hack. Can't we think of something
simpler? For example, why would the child be involved, rather than simply the
parent setting the right cwd?
Also, I don't think there's any point in
Brett Cannon br...@python.org added the comment:
That cannot import name message seems to only come from
Python/ceval.c:import_from() which raises that exception when an AttributeError
is raised by an 'import from' statement (I think). This happens when an 'import
from' asks for a name on the
Brett Cannon br...@python.org added the comment:
Ooh, first customer of importlib.find_loader()! Obviously if the API doesn't
work for you let me know since it's a 3.3 addition and can be tweaked to meet
your needs.
--
nosy: +brett.cannon
___
Roundup Robot devn...@psf.upfronthosting.co.za added the comment:
New changeset dc18a2a66d16 by Brett Cannon in branch 'default':
Issue #15111: When a module was imported using a 'from import'
http://hg.python.org/cpython/rev/dc18a2a66d16
--
nosy: +python-dev
Changes by Brett Cannon br...@python.org:
--
assignee: - brett.cannon
resolution: - fixed
stage: needs patch - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15111
Roger Serwy roger.se...@gmail.com added the comment:
sys.stdin has a write method, but it should raise io.UnsupportedOperation: not
writable when passed a string. It looks like IDLE has allowed writes to stdin
even before Martin's patch. I'll open a separate issue for this case.
--
New submission from Roger Serwy roger.se...@gmail.com:
This is a follow-up to issue13532 which fixed problems with IDLE's sys.stdout
and sys.stderr implementation.
Presently, sys.stdin has a write method, but does not raise
io.UnsupportedOperation: not writable when passed a string. Instead,
Roger Serwy roger.se...@gmail.com added the comment:
Running input from IDLE now raises an error. Attached is a fix to _RPCFile to
allow readline (and isatty) to function properly.
--
priority: normal - high
resolution: fixed -
status: closed - open
Added file:
samwyse samw...@gmail.com added the comment:
Thanks, Eric. Based on what you said, I was able to get the desired behavior
by creating a metaclass.
--
Added file: http://bugs.python.org/file26343/Issue15289.py
___
Python tracker
Roger Serwy roger.se...@gmail.com added the comment:
Issue13532 has an applied patch (422242dbce30) for the subprocess that makes
the stdout, stderr, and stdin inherit from io.TextIOBase. This includes the
errors attribute, albeit set to ''.
The fix does not apply when running IDLE without a
Serhiy Storchaka storch...@gmail.com added the comment:
sys.stdin.readable()
False
sys.stdout.writable()
False
I think the issue is far from a solution.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13532
André Cruz an...@cabine.org added the comment:
Can anyone confirm what is missing for this patch to be committed?
Is it just test and documentation changes or is something wrong with the code
changes as well?
--
nosy: +edevil
___
Python tracker
Carl Nobile carl.nob...@gmail.com added the comment:
I was told some time ago that it was documentation changes. And, if I
remember correctly CONTINUE (100) was not ignored, it was actually broken.
Data was being read from stdin when a CONTINUE was received and this should
never happen based on
Martin v. Löwis mar...@v.loewis.de added the comment:
Folks: one issue at a time - pretty pretty please.
This issue was In IDLE, sys.stdout and sys.stderr can write any pickleable
object. Will anybody dispute that this issue is fixed?
Roger: please create a new issue for input being broken
New submission from Roger Serwy roger.se...@gmail.com:
Per Martin's request in msg165197, this issue has been separated from
Issue13532.
The newly introduced _RPCFile object in run.py (422242dbce30) inherits from
io.TextIOBase which has readline as one of its methods. As a result, IDLE's
Roger Serwy roger.se...@gmail.com added the comment:
I tried issue4832_rev2.patch on Ubuntu 11.04 with Tk 8.5 and it still behaves
as Terry described.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4832
R. David Murray rdmur...@bitdance.com added the comment:
As mentioned, the first step is to create some tests that can validate the
current behavior, so that changes don't break things. This is a non-trivial
task. I know from experience with a similar refactoring that even seemingly
simple
Matthias Klose d...@debian.org added the comment:
proposed patch, tested with configure, build and install
--
keywords: +needs review, patch
Added file: http://bugs.python.org/file26345/sysconfigdata.diff
___
Python tracker rep...@bugs.python.org
Serhiy Storchaka storch...@gmail.com added the comment:
There is a simpler solution of this issue.
--
nosy: +storchaka
type: - behavior
Added file: http://bugs.python.org/file26346/idle_stdin_no_rpcfile.patch
___
Python tracker
New submission from Chris Jerdonek chris.jerdo...@gmail.com:
My understanding is that generators are not thread-safe. For example, see
http://stackoverflow.com/a/1131458/262819
However, regrtest.main() seems to access a generator from multiple threads when
run in multiprocess mode:
def
Roger Serwy roger.se...@gmail.com added the comment:
I like Serhiy's patch. It works for me.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15319
___
André Cruz an...@cabine.org added the comment:
As far as I can see, the patch does add some documentation changes. What
exactly is missing?
As for the bug, if I understood correctly, what you are saying is that when
ignore_continue is True, and the server sends a 100 Continue response, the
Antoine Pitrou pit...@free.fr added the comment:
The patch is rather ugly, I think. You should arrange for sysconfigdata to be
generated directly at the right place instead (not sure how, perhaps it should
be done from setup.py).
Also, there's no need for this to be a release blocker.
Carl Nobile carl.nob...@gmail.com added the comment:
Yes, exactly. I was not the one who posted the original bug report, but I
found it when I ran into the same problem. I was not exactly sure if the
original poster had the same issues as I had. I do know that my fix to the
code eliminated some
Stefan Krah stefan-use...@bytereef.org added the comment:
I see nothing wrong in memory_ass_sub(). In 2.7:
if (destbuf + bytelen srcbuf || srcbuf + bytelen destbuf)
/* No overlapping */
memcpy(destbuf, srcbuf, bytelen);
else
memmove(destbuf, srcbuf, bytelen);
André Cruz an...@cabine.org added the comment:
Carl: that would be great. Do you use it regularly? Any other problems?
Python devs: can anyone confirm me what still needs to be done so that this
patch can be considered for merging into trunk? Thanks.
--
Craig McQueen pyt...@craig.mcqueen.id.au added the comment:
It would be great if this could be sorted out in time for Python 3.3. Otherwise
I don't think we'll be able to use MinGW to build extensions in Windows. Unless
there is a version of MinGW which supports the -mno-cygwin option, as well
Changes by Antoine Pitrou pit...@free.fr:
--
nosy: +benjamin.peterson
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15317
___
___
Python-bugs-list
Senthil Kumaran sent...@uthcode.com added the comment:
The patch seems good. I apologize that this has been sitting stale for a long
time. Since this is a new feature, I am not sure if putting to 3.3 might be a
good idea. This is a feature for httplib, so it may not make it to 2.7.x, but
can
Craig McQueen pyt...@craig.mcqueen.id.au added the comment:
I've succeeded in building an extension for Python 3.3 (at least, on Windows
XP, 32-bit; haven't tried any 64-bit), by the hack of copying libmsvcr100.a
from a recent MinGW release (20120426) into an older MinGW release (20101030).
I
New submission from Mark Hammond skippy.hamm...@gmail.com:
Note the error message in the title is only for Python 2.x - Python 3.x shows
an empty string instead, but otherwise seems identical.
This was first brought to my attention via
Changes by Antoine Pitrou pit...@free.fr:
--
nosy: +loewis
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15321
___
___
Python-bugs-list mailing
Terry J. Reedy tjre...@udel.edu added the comment:
I agree that the _RPCFile wrapping of the stdin proxy should be undone unless
and until there is a positive reason for it -- it solves a problem -- and it is
better tested. But reversion does not solve pre-existing problems. As noted by
Terry J. Reedy tjre...@udel.edu added the comment:
To me, this is part of #15319. sys.stdin is broken in being writable but not
readable. Unwrapping is just the first, trivial step to fixing it. So I think
we should close this in favor of the other.
--
Roger Serwy roger.se...@gmail.com added the comment:
I agree. I'll close it.
--
resolution: - duplicate
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15318
___
Changes by Roger Serwy roger.se...@gmail.com:
--
superseder: - IDLE - input() is broken.
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15318
___
Roger Serwy roger.se...@gmail.com added the comment:
Including issue15318 where stdin is writable. The proper solution to that issue
and this one are likely the same.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15319
Roger Serwy roger.se...@gmail.com added the comment:
Same difference for readlines and writelines. I wonder how input works if it
does not call sys.stdin.read()
Eventually IDLE makes a call to PyShell's readline.
--
___
Python tracker
New submission from Chris Jerdonek chris.jerdo...@gmail.com:
sysconfig.get_config_var('srcdir') seems to return the current working
directory rather than a directory related to the source directory.
For example (starting from the repository root)--
$ mkdir foo
$ cd foo
$ ../python.exe
Python
Carl Nobile carl.nob...@gmail.com added the comment:
André,
As I said I'm not sure if I am fixing the same thing that this bug was
originally posted for. However, after looking at my code I realized that I
just did a quick work around for my situation and it shouldn't be put into
any python
Chris Jerdonek chris.jerdo...@gmail.com added the comment:
Thanks for your feedback, Antoine, and for causing me to rethink the patch.
This patch is drastically simpler.
Incidentally, I found a reason for this issue in issue 15322.
regrtest calculates the build folder relative to
Terry J. Reedy tjre...@udel.edu added the comment:
Ah, sys.stdin.readline() works. So the object being proxied has readline (and
write and writelines) but not read and readlines. I presume the proxy is in the
user process and the actual stdin/out/err objects are in the idle process,
Changes by Terry J. Reedy tjre...@udel.edu:
--
title: IDLE - input() is broken. - IDLE - sys.stdin is broken.
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15319
___
New submission from Brian Jones bkjo...@gmail.com:
In Python 3.3.0b1, if the number of calls to a mock is, say, zero, and you call
assert_called_once_with() on that mock, it will throw an exception which says
Expected to be called once. Called 0 times.
This is nice, but it would be nicer if
Changes by Arfrever Frehtes Taifersar Arahesis arfrever@gmail.com:
--
nosy: +Arfrever
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15230
___
Changes by Arfrever Frehtes Taifersar Arahesis arfrever@gmail.com:
--
nosy: +Arfrever
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15272
___
Changes by Arfrever Frehtes Taifersar Arahesis arfrever@gmail.com:
--
nosy: +Arfrever
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15314
___
Chris Jerdonek chris.jerdo...@gmail.com added the comment:
I have a minor suggestion. I would suggest--
+ msg = (Expected %s to be called once. Called %s times. %
+(repr(self._mock_name) or 'mock', self.call_count))
so that one can distinguish between self._mock_name not being defined
Changes by Éric Araujo mer...@netwok.org:
--
nosy: +eric.araujo
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14797
___
___
Python-bugs-list
Changes by Ramchandra Apte maniandra...@gmail.com:
--
nosy: +ramchandra.apte
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15319
___
___
Roger Serwy roger.se...@gmail.com added the comment:
PyShell.py's PyShell object has the readline method, at line 1080 in the most
recent code. It's meant for use with and without a subprocess. (See also
Issue14254)
The IDLEfork project long ago created the subprocess and the RPC plumbing for
72 matches
Mail list logo
