Austin Bingham austin.bing...@gmail.com added the comment:
Yes, in some sense that's what I'm thinking of. But one problem with
this straightforward approach is that it doesn't scale well. If I've
got many TestCases, each if which I want to parameterize, I have to
create subclasses for each
Eli Bendersky eli...@gmail.com added the comment:
OK, I can reproduce the problem on a clean virtual machine running a pristine
XP home SP2. No Python was previously installed.
Steps to reproduce:
1. Install Python 3.2.1 MSI from http://www.python.org/download/releases/3.2.1/
2. Run IDLE from
Ezio Melotti ezio.melo...@gmail.com added the comment:
Several buildbot are failing after the commit:
==
FAIL: test_get_original_stdout (test.test_support.TestSupport)
angus an...@amcinnes.info added the comment:
I'm experiencing a related problem:
---
from urllib.request import urlopen
print(urlopen('https://mtgox.com/').read())
---
prints b'' rather than the page content.
It looks like mtgox.com always sends 'Connection: Keep-Alive'. So some hack
like
Changes by Senthil Kumaran sent...@uthcode.com:
--
assignee: - orsenthil
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12576
___
___
Senthil Kumaran sent...@uthcode.com 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 rep...@bugs.python.org
Robert Xiao nneon...@gmail.com added the comment:
S3 also doesn't send any kind of connection header at all.
x-amz-id-2: WWuo30Fk2inKVcC5dH4GOjvHxnqMa5Q2+AduPm2bMhL1h3GqzOR0EPwUv0biqv2V
x-amz-request-id: 3CCF6B6A000E6446
Date: Sat, 23 Jul 2011 06:42:45 GMT
x-amz-meta-s3fox-filesize: 27692
Eli Bendersky eli...@gmail.com added the comment:
Am I missing something, or is there no explicit command to kill the subprocess
on Windows in PyShell.py
The kill_subprocess method (which does get invoked) of ModifiedInterpreter is:
def kill_subprocess(self):
try:
Martin v. Löwis mar...@v.loewis.de added the comment:
I give up. Call it 'linux3', then.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12326
___
New submission from Sjoerd de Vries sjdv1...@gmail.com:
When you specify cfile to be in the current directory, an error occurs (line
133).
I have fixed the file, see attached
--
components: Library (Lib)
files: py_compile.py
messages: 140940
nosy: sjdv1982
priority: normal
severity:
Arfrever Frehtes Taifersar Arahesis arfrever@gmail.com added the comment:
Please create a patch in unified format.
--
nosy: +Arfrever
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12618
Georg Brandl ge...@python.org added the comment:
Done (in rev e32f140a020b).
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11435
___
___
Georg Brandl ge...@python.org added the comment:
index does create targets, but they are not accessible for creating a link *to*
it. They are only used for links from the indices.
--
___
Python tracker rep...@bugs.python.org
Matthias Klose d...@debian.org added the comment:
this does sound very ugly.
so we get now another mostly unmaintained platform directory? unfortunately the
generated header files are almost never updated during a releaes cycle.
and we repeat the mistakes that some constants differ on some
Changes by Georg Brandl ge...@python.org:
--
nosy: +georg.brandl, loewis
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12605
___
___
Sjoerd de Vries sjdv1...@gmail.com added the comment:
The attached file just works.
You can diff with trunk, or wherever python devs store the latest version.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12618
Georg Brandl ge...@python.org added the comment:
It might work right now, but in case the file changes before your change can be
processed, we will lose the previous changes if we just copy in your new file.
IOW, you're making our work much harder and your change is less likely to be
applied.
Georg Brandl ge...@python.org added the comment:
Recognizing ction: close as Connection: close is exactly what those servers
do *not* want you to do.
--
nosy: +georg.brandl
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12576
Ned Deily n...@acm.org added the comment:
Doesn't unix_terminate() also get called on Windows? If so, what does
os.kill() do on Windows? The docs for os.kill say New in version 3.2: Windows
support. Perhaps this was being skipped before and now has some negative
effect?
--
STINNER Victor victor.stin...@haypocalc.com added the comment:
It's too late to fix sys.platform.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12326
___
Éric Araujo mer...@netwok.org added the comment:
Peter’s patch now uses iter(thing) instead of len(thing) to assess
sequenciness. I made this comment:
You can iterate over an iterator (which is not a sequence). Here I
don’t know if the code talks about sequence because it pre-dates
Éric Araujo mer...@netwok.org added the comment:
Same bug. I’ve added debug prints to find out the rights (UNIX permission
system) of the files, and they’re very strange: --wxrw--wt (the read bit is
missing, and the t is strange). It should be -rwxr-xr-x, like other programs.
The bug is
New submission from Arfrever Frehtes Taifersar Arahesis
arfrever@gmail.com:
I suggest that platform-specific modules be automatically regenerated during
installation. I'm attaching a patch.
--
files: python-regenerate_platdir.patch
keywords: patch
messages: 140952
nosy: Arfrever,
Sjoerd de Vries sjdv1...@gmail.com added the comment:
Makes no sense to me: since I don't have the trunk version, I can only diff -c
against 3.2 release. Doing a diff against trunk is 1 sec of work for you.
But I am just being a helpful user; so if a diff is what you want, here it is.
No
Eli Bendersky eli...@gmail.com added the comment:
Indeed, unix_terminate is invoked on Windows, and since Windows now has
os.kill it runs. However, it appears that the actual os.kill call throws
OSError, saying:
[Error 87] The parameter is incorrect
--
Arfrever Frehtes Taifersar Arahesis arfrever@gmail.com added the comment:
See issue #12619 for potential solution for platform-specific modules.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12326
Éric Araujo mer...@netwok.org added the comment:
I confirm fixing 0o755 makes the tests pass for me.
Your code gives one warning:
build_scripts.py:241: BytesWarning: str() on a bytes instance
hdr = #!%(executable)s%(options)s\n % locals()
The object with the name executable or options is
Éric Araujo mer...@netwok.org added the comment:
(Autotools/make newbie here) Why during install and not build?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12619
___
Arfrever Frehtes Taifersar Arahesis arfrever@gmail.com added the comment:
I don't care when they will be regenerated.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12619
___
Arfrever Frehtes Taifersar Arahesis arfrever@gmail.com added the comment:
It's a context patch, not a unified patch, and it is reversed :) .
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12618
Changes by Daniel Urban urban.dani...@gmail.com:
--
nosy: +durban
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12617
___
___
Python-bugs-list
Petri Lehtinen pe...@digip.org added the comment:
Should the component really be extension modules? I don't believe _sre.c is a
part of the API, and the problem is embedding Python as a whole.
--
___
Python tracker rep...@bugs.python.org
Senthil Kumaran sent...@uthcode.com 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
Matthias Klose d...@debian.org added the comment:
is auto-generation wanted?
are you sure that you can't end up with bad syntax?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12619
___
Roundup Robot devn...@psf.upfronthosting.co.za added the comment:
New changeset e171db785c37 by Senthil Kumaran in branch '3.2':
Fix closes issue12581 - Increase the urllib.parse test coverage. Patch by
Petter Haggholm.
http://hg.python.org/cpython/rev/e171db785c37
New changeset fcccda3c546f
Georg Brandl ge...@python.org added the comment:
Well, we can work with this patch. Thanks.
Yes, we can diff against 3.2, but for that you would have at least needed to
specify
* that you worked on a released version
* and that that version is 3.2
Otherwise, we cannot know what to diff
Eli Bendersky eli...@gmail.com added the comment:
Here's a simple reproducer for the same problem, without the context of IDLE.
As far as I understand, what IDLE's spawn and then kill process are doing is:
import os
from signal import SIGTERM
pid = os.spawnv(os.P_NOWAIT, notepad.exe,
Georg Brandl ge...@python.org added the comment:
Yes, this is a bug in bytearray and should be fixed.
--
assignee: docs@python -
components: -Documentation
nosy: +georg.brandl
versions: +Python 2.7, Python 3.2 -Python 3.1
___
Python tracker
Senthil Kumaran sent...@uthcode.com 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
Georg Brandl ge...@python.org added the comment:
Hmm, the docs say Any other value for sig will cause the process to be
unconditionally killed by the TerminateProcess API [...]
What happens if you try to use other signals (like signal.SIGKILL) instead of
SIGTERM?
--
Georg Brandl ge...@python.org added the comment:
The other question is if it is an access control problem.
win32_kill tries to open the process with PROCESS_ALL_ACCESS, while IMO
PROCESS_TERMINATE would suffice.
--
___
Python tracker
Eli Bendersky eli...@gmail.com added the comment:
Georg, I'm now debugging into win32_kill, and it's an error in OpenProcess, so
this *could* be a security issue.
The process is started with _spawnv, so maybe this causes problems opening it
with OpenProcess later.
--
Charles-François Natali neolo...@free.fr added the comment:
This behavior was introduced (exposed by?) r68460.
After a quick look, I don't see anything wrong with r68460, but it might very
well have exposed this problem which was lurking before.
When Python is compiled without threads, the
Eli Bendersky eli...@gmail.com added the comment:
According to
http://msdn.microsoft.com/en-us/library/7zt1y878%28v=vs.80%29.aspx, on Windows
_spawnv in async mode (P_NOWAIT) returns the process _handle_, not the process
ID.
win32_kill uses OpenProcess, passing it pid to obtain the handle,
Georg Brandl ge...@python.org added the comment:
Hmm, on the other hand there may be valid use cases for using os.kill() with a
PID. Argh.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12540
Eli Bendersky eli...@gmail.com added the comment:
I don't think there's a problem with os.spawnv and os.kill - they do what their
docs describe.
IMHO, the solution should be to change IDLE so that it uses subprocess.Popen
for both starting and killing the child process.
--
New submission from Charles-François Natali neolo...@free.fr:
Trivial patch.
In Python/ceval.c, when compiled with threads, make the `pendingbusy` flag used
to guard against reentrant calls static to Py_MakePendingCalls().
Also, make it an int and rename it to `busy` to be consistent with the
Roundup Robot devn...@psf.upfronthosting.co.za added the comment:
New changeset 0018a28583f4 by Eli Bendersky in branch 'default':
Issue #11049: skip a test that fails on some buildbots
http://hg.python.org/cpython/rev/0018a28583f4
--
___
Python
Changes by Benjamin Peterson benja...@python.org:
--
dependencies: +add a AST validator
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12608
___
higery shoulderhig...@gmail.com added the comment:
An octal literal in 3.x is 0o755. Decimal 755 means 0o1363, which is not
good :)
Thank you for your reminding. The reason I made this mistake is that I'm not
familiar with the right way to set permission code in Python3+ .
--
higery shoulderhig...@gmail.com added the comment:
The object with the name executable or options is bytes, which you should
explicitly convert to a string with decode. I also don’t like using
locals(), but that’s a personal style thing.
Thanks for your test, I'll amend it.
--
Eli Bendersky eli...@gmail.com added the comment:
OK, the bots are green again after the last push.
Sorry about that, folks. I tested this on two different machines (Ubuntu Win
XP) before pushing the original commit.
--
___
Python tracker
Roundup Robot devn...@psf.upfronthosting.co.za added the comment:
New changeset c741ba9e37ef by Nadeem Vawda in branch '3.2':
Issue #10883: Fix socket leaks in urllib.request.
http://hg.python.org/cpython/rev/c741ba9e37ef
New changeset d68765bd6490 by Nadeem Vawda in branch 'default':
Merge:
Steffen Daode Nurpmeso sdao...@googlemail.com added the comment:
Even PEP 3151 won't help.
I don't understand. If the syscall supposed to flush the disk's buffer
cache fails - be it fcntl() or sync_file_range() - I think the error
should be propagated, not silently ignored and replaced
higery shoulderhig...@gmail.com added the comment:
Your test should catch stdout (see other packaging tests for how to do that),
so that people or buildbots running the tests don’t see “Hello world!”, and
so that you can run asserts for the output.
Thanks. Got it - captured_stdout
Arfrever Frehtes Taifersar Arahesis arfrever@gmail.com added the comment:
The new patch creates platform-specific modules at build time and verifies
their syntax.
--
Added file: http://bugs.python.org/file22726/python-regenerate_platdir.patch
Changes by Arfrever Frehtes Taifersar Arahesis arfrever@gmail.com:
Removed file: http://bugs.python.org/file22723/python-regenerate_platdir.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12619
Antoine Pitrou pit...@free.fr added the comment:
this does sound very ugly.
so we get now another mostly unmaintained platform directory?
unfortunately the generated header files are almost never updated
during a releaes cycle.
I would be +1 to deprecate this stuff, but that's quite
Charles-François Natali neolo...@free.fr added the comment:
and Darwin).
It would have been useful...
interp 0x0, thread state 0x81855380:
Program received signal SIGSEGV, Segmentation fault.
Here, the interpreter state is NULL, and this shouldn't happen.
It could be a bug linked to
Roundup Robot devn...@psf.upfronthosting.co.za added the comment:
New changeset dbf1e1a27427 by Nadeem Vawda in branch '2.7':
Issue #10883: Fix socket leaks in urllib.request.
http://hg.python.org/cpython/rev/dbf1e1a27427
--
___
Python tracker
Changes by higery shoulderhig...@gmail.com:
Added file: http://bugs.python.org/file22727/c5692393c621.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12394
___
R. David Murray rdmur...@bitdance.com added the comment:
Exhaustion of the iterator is easily solved by simply retaining a reference to
it and iterating that (which is what I had in mind). However, I had not
thought about the problem of an *in*exhaustable iterator, and to cover that
case len
Vlad Riscutia riscutiav...@gmail.com added the comment:
Updated patch to reflect review feedback. Allocation strategy is now specified
as string in Python code.
I kept asserts in CanContinueField/CanExpandField because, as I said, default
case should never be hit. Input is validated in
Sjoerd de Vries sjdv1...@gmail.com added the comment:
Good to hear that the patch is helpful.
Again, I am just trying to be a helpful user, making a (very very little)
contribution to make Python better. I am not a Python dev at all: Python is
already awesome enough for me, no desire to
Stefan Krah stefan-use...@bytereef.org added the comment:
Unassigning: I tested on OpenBSD 4.5 and get other (probably unrelated
errors).
--
assignee: skrah -
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12560
Changes by Antoine Pitrou pit...@free.fr:
--
nosy: +haypo, neologix
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12560
___
___
Python-bugs-list
Antoine Pitrou pit...@free.fr added the comment:
Looks obviously fine :)
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12620
___
___
Antoine Pitrou pit...@free.fr added the comment:
The improvements are welcome, but I'm not sure they're ok for 3.2 or 2.7, since
they're technically a new feature.
OTOH, one could argue that better debuggability trumps the no-feature rule.
--
nosy: +pitrou
stage: - patch review
Changes by Antoine Pitrou pit...@free.fr:
--
nosy: +neologix
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12603
___
___
Python-bugs-list mailing
Roundup Robot devn...@psf.upfronthosting.co.za added the comment:
New changeset cda93720c06d by Charles-François Natali in branch 'default':
Issue 12620: Make pendingbusy flag static to Py_MakePendingCalls().
http://hg.python.org/cpython/rev/cda93720c06d
--
nosy: +python-dev
Changes by Charles-François Natali neolo...@free.fr:
--
resolution: - fixed
stage: patch review - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12620
Darren Dale dsdal...@gmail.com added the comment:
Here is a new version of the patch, addressing points raised in the review of
the previous version.
--
Added file: http://bugs.python.org/file22729/abc_descriptor.patch
___
Python tracker
Changes by Michael Foord mich...@voidspace.org.uk:
--
nosy: -michael.foord
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11610
___
___
aaugustin aymeric.augus...@polyconseil.fr added the comment:
I noticed an inconsistency in the docs, and I think it falls in the Language
Reference section of this ticket. In the definition of the import statement,
after:
| from module import *
we should add:
| from
Charles-François Natali neolo...@free.fr added the comment:
Demo:
cf@neobox:~/cpython$ ./python -c import pydoc;
print(pydoc.synopsis('Lib/os.py'))
OS routines for Mac, NT, or Posix depending on what system we're on.
[51835 refs]
cf@neobox:~/cpython$ touch -t 19700101 Lib/os.py
Changes by Antoine Pitrou pit...@free.fr:
--
title: text files returned by subprocess.Popen with universal_newlines=True are
not iterable - TextIOWrapper should fall back on read() if read1() doesn't
exist
___
Python tracker rep...@bugs.python.org
Antoine Pitrou pit...@free.fr added the comment:
So, as the title indicates, I think we should make TextIOWrapper work with raw
IO objects. The reason is so that write() can behave in a totally unbuffered
way, which is necessary for Popen to behave appropriately.
--
Michael Foord mich...@voidspace.org.uk added the comment:
Having a TestCase factory would be pretty easy, and solve the scaling
problems.
For example:
def make_testcase_classes():
for backend in backends:
yield type(
'{}Test'.format(backend.name),
Antoine Pitrou pit...@free.fr added the comment:
Hmm, one problem is that the C TextIOWrapper buffers writes. We would need an
additional constructor parameter to prevent that :/
--
___
Python tracker rep...@bugs.python.org
New submission from Petri Lehtinen pe...@digip.org:
bytes.{find,rfind} reads s[start:end]; should be B[start:end]
bytearray.{find,rfind} reads s[start,end]; should be B[start:end]
{str,unicode}.{find,rfind} reads s{start:end]; should be S[start:end]
--
assignee: docs@python
Antoine Pitrou pit...@free.fr added the comment:
Another possibility is to provide read1() on RawIO, as a synonym of read().
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12591
___
Antoine Pitrou pit...@free.fr added the comment:
Here is a first patch allowing TextIOWrapper to work with raw IO objects, and
adding a write_through flag.
--
keywords: +patch
Added file: http://bugs.python.org/file22731/textio_rawio.patch
___
New submission from Antoine Pitrou pit...@free.fr:
http://docs.python.org/dev/library/unittest.html#unittest.TextTestRunner
doesn't mention the failfast argument, even though it's used by
unittest.main().
--
assignee: michael.foord
components: Documentation
messages: 141004
nosy:
Amaury Forgeot d'Arc amaur...@gmail.com added the comment:
write_through is not used in _pyio.py, is it expected?
--
nosy: +amaury.forgeotdarc
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12591
Antoine Pitrou pit...@free.fr added the comment:
write_through is not used in _pyio.py, is it expected?
Yup, because it is actually always write-through (writes are not
cached). The argument is only there so that the signature is the same as
the C version.
--
Stefan Krah stefan-use...@bytereef.org added the comment:
I know, the OpenBSD libpthread has problems. I listed some possible
candidates in #8712.
For instance:
Amaury Forgeot d'Arc amaur...@gmail.com added the comment:
Looks good, then.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12591
___
___
Antoine Pitrou pit...@free.fr added the comment:
This additional patch improves universal_newlines support in subprocess (still
broken with the select- and poll-based loops in communicate()).
--
Added file: http://bugs.python.org/file22732/spnewlines.patch
Terry J. Reedy tjre...@udel.edu added the comment:
Eli, nice detective work. What I understand is that there was a latent
platform-dependent buglet that presumably got exposed by a recent change in
process handling, as Ned suggested.
idlelib/PyShell.py, class
Roundup Robot devn...@psf.upfronthosting.co.za added the comment:
New changeset 9144014028f3 by Antoine Pitrou in branch '3.2':
Issue #12591: Allow io.TextIOWrapper to work with raw IO objects (without
http://hg.python.org/cpython/rev/9144014028f3
New changeset c3b47cdea0d1 by Antoine Pitrou in
Stefan Krah stefan-use...@bytereef.org added the comment:
I forgot: readline support is enabled, and there are known problems:
http://marc.info/?t=12832732933r=1w=2
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue8714
Roundup Robot devn...@psf.upfronthosting.co.za added the comment:
New changeset 5cc536fbd7c1 by Antoine Pitrou in branch '3.2':
Issue #12591: Improve support of universal newlines in the subprocess
http://hg.python.org/cpython/rev/5cc536fbd7c1
New changeset b616396fa170 by Antoine Pitrou in
New submission from Antoine Pitrou pit...@free.fr:
The select() and poll() loop implementations of Popen.communicate() call
os.write() instead of the write() method on the stdin pipe, meaning any newline
translation *and* unicode-to-bytes encoding step is skipped.
To use the write() method on
Antoine Pitrou pit...@free.fr added the comment:
Ok, this should be fixed now. universal newlines support is still broken with
communicate(), I've opened issue12623 for that.
--
resolution: - fixed
stage: needs patch - committed/rejected
status: open - closed
Petri Lehtinen pe...@digip.org added the comment:
Attached a patch with the following changes:
Allow an integer argument in range(0, 256) for the following bytes and
bytearray methods: count, find, index, rfind, rindex. Initially, only
count and index were targeted, but as index is implemented
New submission from Antoine Pitrou pit...@free.fr:
This patch adds a new --failfast (-G) argument to regrtest which stops running
the tests as soon as a test fails (when using unittest-based test cases).
Useful with long test suites such as test_io and test_subprocess.
--
components:
Roundup Robot devn...@psf.upfronthosting.co.za added the comment:
New changeset 9f847dfba217 by Antoine Pitrou in branch '3.2':
Issue #12624: It is now possible to fail after the first failure when
http://hg.python.org/cpython/rev/9f847dfba217
New changeset 01d18277c40b by Antoine Pitrou in
Changes by Antoine Pitrou pit...@free.fr:
--
resolution: - invalid
stage: patch review - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12624
___
New submission from Antoine Pitrou pit...@free.fr:
Seen on an OpenIndiana buildbot:
==
FAIL: testInstallHandler (unittest.test.test_break.TestBreak)
--
Changes by Sandro Tosi sandro.t...@gmail.com:
--
resolution: invalid - fixed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12624
___
___
1 - 100 of 128 matches
Mail list logo