Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue17552>
___
___
Python-bugs-list mailing list
Unsubscribe:
Ross Lagerwall added the comment:
That text was from the POSIX 2008 spec:
http://pubs.opengroup.org/onlinepubs/9699919799/functions/readdir.html
The following text from my copy of the readdir manpage gives an indication of
how you *should* allocate struct dirent when using readdir_r:
"&qu
Ross Lagerwall added the comment:
Hi,
There shouldn't be a problem with the existing implementation since, according
to posix:
"""
The pointer returned by readdir() points to data which may be overwritten by
another call to readdir() on the same directory stream. This data
Ross Lagerwall added the comment:
> Ross, the select() result for a large number of ready FDs was
> completely skewed because of a bug spotted by Antoine (complexity
> was much worse than it ought to be).
Ah, that makes a little more
Ross Lagerwall added the comment:
Interesting benchmark. There is no gain for epoll with a large number of
ready fds (as expected) but at least it is no worse than poll. Poll offers
a large improvement on select, in this case.
$ ./python selector_bench.py -r 2 -m 1000 -t pipe
Trying with 2
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue16853>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue16850>
___
___
Python-bugs-list mailing list
Unsubscribe:
New submission from Ross Lagerwall:
test_posix.test_getgroups() fails on some systems:
http://buildbot.python.org/all/builders/AMD64%20Mountain%20Lion%20%5BSB%5D%203.2
This could be related to #16661.
--
components: Tests
messages: 177601
nosy: rosslagerwall
priority: normal
severity
Ross Lagerwall added the comment:
getgrouplist() is new in 3.3. Those failures are from getgroups() failing. I'll
open a separate issue for that.
--
status: open -> closed
___
Python tracker
<http://bugs.python.org
Changes by Ross Lagerwall :
--
assignee: -> rosslagerwall
resolution: -> fixed
stage: needs patch -> committed/rejected
status: open -> closed
___
Python tracker
<http://bugs.python
Ross Lagerwall added the comment:
Is that fixed now? I simplified the test to check for a non-empty list being
returned.
--
___
Python tracker
<http://bugs.python.org/issue16
Ross Lagerwall added the comment:
I wouldn't think so. A call to setgroups can add or remove groups for
the calling process. If it removes groups, then getgrouplist() won't
return a subset of getgroups().
--
___
Python trac
Ross Lagerwall added the comment:
It seems like getgrouplist returns the information from the system
database whereas getgroups (and consequently id -G) returns the
supplementary groups for the calling process.
I'm not exactly sure how getgrouplist() can be effectively tested
Changes by Ross Lagerwall :
--
resolution: -> invalid
status: open -> closed
___
Python tracker
<http://bugs.python.org/issue16639>
___
___
Python-bugs-
Changes by Ross Lagerwall :
--
assignee: -> rosslagerwall
nosy: +rosslagerwall
resolution: -> fixed
stage: -> committed/rejected
status: open -> closed
versions: +Python 3.2, Python 3.4
___
Python tracker
<http://bugs.python
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue16140>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Ross Lagerwall :
--
resolution: -> fixed
stage: -> committed/rejected
status: open -> closed
___
Python tracker
<http://bugs.python.or
Ross Lagerwall added the comment:
This didn't get picked up by Antoine's daily refleak test run because test
curses only runs when stdout is a TTY.
--
___
Python tracker
<http://bugs.python.o
New submission from Ross Lagerwall:
[1/1] test_curses
beginning 6 repetitions
123456
.
test_curses leaked [1, 1, 1] references, sum=3
1 test failed:
test_curses
[154814 refs]
--
assignee: rosslagerwall
messages: 169973
nosy: rosslagerwall
priority: low
severity: normal
status: open
Ross Lagerwall added the comment:
I sent a review through on rietveld; I'm attaching a patch with the changes so
that it compiles and passes the tests.
--
Added file: http://bugs.python.org/file27053/issue15798_v2.patch
___
Python tracker
Ross Lagerwall added the comment:
The attached patch + test seems to fix the issue.
It's not very elegant.
--
keywords: +patch
Added file: http://bugs.python.org/file27042/issue15798.patch
___
Python tracker
<http://bugs.python.org/is
Ross Lagerwall added the comment:
It's caused by the following check in _posixsubprocess.c:
if (close_fds && errpipe_write < 3) { /* precondition */
PyErr_SetString(PyExc_ValueError, "errpipe_write must be >= 3");
return NULL;
}
which was w
Changes by Ross Lagerwall :
--
resolution: -> fixed
stage: needs patch -> committed/rejected
status: open -> closed
___
Python tracker
<http://bugs.python.or
New submission from Ross Lagerwall:
results for fa745ed89b7a on branch "default"
test_capi leaked [2, 2, 2] references, sum=6
Command line was: ['./python', '-m', 'test.regrtest', '-uall', '
Ross Lagerwall added the comment:
Sorry, I didn't mean that it's impossible; I meant that it shouldn't happen and
it should be fixed :-)
Unfortunately I don't have access to an OS X box to test.
--
___
Python tracker
<
Ross Lagerwall added the comment:
Well the app has an infinite recursion.
This shows on Fedora Linux 17:
"""
Request Method: GET
Request URL:http://127.0.0.1:8000/
Django Version: 1.4
Exception Type: RuntimeError
Exception Value:
maximum
Ross Lagerwall added the comment:
Attached is a diff between dir(os) in 3.2 and 3.3
--
keywords: +patch
nosy: +rosslagerwall
Added file: http://bugs.python.org/file26676/oschanges.diff
___
Python tracker
<http://bugs.python.org/issue15
Ross Lagerwall added the comment:
I can't actually remember why I disabled waitid for OS X - that was message was
rather a long time ago :-(
Unfortunately, I don't currently have access to an OS X machine to test it.
A google search shows the following comment in the v8 javascr
Changes by Ross Lagerwall :
--
resolution: -> duplicate
status: open -> closed
superseder: -> thread-safety issue in regrtest.main()
___
Python tracker
<http://bugs.python.o
Ross Lagerwall added the comment:
Are there any webbrowser unit tests?
(this could probably use the new subprocess.DEVNULL constant in 3.3)
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue15
Ross Lagerwall added the comment:
It looks like this broke the build bots:
http://buildbot.python.org/all/builders/AMD64%20Ubuntu%20LTS%202.7/builds/66/steps/test/logs/stdio
--
assignee: -> orsenthil
nosy: +rosslagerwall
status: closed ->
Ross Lagerwall added the comment:
Ah, I see you've already opened a new issue for that (issue15284).
--
___
Python tracker
<http://bugs.python.org/is
Ross Lagerwall added the comment:
Thanks.
test_socket seems to be broken in all branches when running with
net.ipv6.conf.all.disable_ipv6 = 1 but I'll open a new issue for that.
--
assignee: -> rosslagerwall
nosy: +rosslagerwall
resolution: -> fixed
stage: -> comm
Ross Lagerwall added the comment:
This looks like the kind of optimization that depends hugely on what kernel
you're using. Maybe on FreeBSD/Solaris/whatever, standard os.walk() is faster?
If this micro-optimization were to be accepted, someone would have to be keen
enough to test
Ross Lagerwall added the comment:
Yeah, after I submitted the patch, I was unsure if that was a good idea or if
it should try and use gdbm in native mode if possible.
--
___
Python tracker
<http://bugs.python.org/issue15
Ross Lagerwall added the comment:
Attached is a patch which fixes the issue on Fedora 17.
If this doesn't break other OSes I'll commit it for 2.7, 3.2 and 3.3.
--
keywords: +patch
versions: +Python 2.7, Python 3.2
Added file: http://bugs.python.org/file25943/
Ross Lagerwall added the comment:
The gdbm provided with Fedora 17 provides /usr/include/ndbm.h.
This makes setup.py think that it should try link with -lndbm when it actually
requires -lgdbm_compat.
A workaround is to specify --with-dbmliborder=gdbm to force gdbm to be used.
I'll tr
Ross Lagerwall added the comment:
Unfortunately, it seems like it's still failing on the RHEL 6 buildbot.
--
status: pending -> open
___
Python tracker
<http://bugs.python.org
Ross Lagerwall added the comment:
Yeah, I'm pretty sure it means if and only if.
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/is
Ross Lagerwall added the comment:
> Personally, I would factor out the code for Popen.communicate() in to a >
> Communicator class which wraps a Popen object and has a method
>
>communicate(input, timeout=None) -> (bytes_written, output, error)
How would this diff
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue1191964>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Ross Lagerwall :
--
assignee: astrand ->
___
Python tracker
<http://bugs.python.org/issue1191964>
___
___
Python-bugs-list mailing list
Unsubscri
Ross Lagerwall added the comment:
Closed issue14872 as a duplicate of this.
--
assignee: astrand ->
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue1
Ross Lagerwall added the comment:
See also issue1260171.
Closing as a duplicate of that.
--
resolution: -> duplicate
stage: -> committed/rejected
status: open -> closed
superseder: -> subprocess: more general (non-buffering) communication
type: -&
Ross Lagerwall added the comment:
Well if you're *certain* that the process is only using one stream, then you
can just use read/write on that stream.
If not, it probably means you have to use either threads or select/poll.
This is a known issue with subprocess; there are a few proposa
Ross Lagerwall added the comment:
The attached patch fixed the test for me on Fedora 16.
It was necessary for the `define` to be after the -ba switch.
I don't know why this wouldn't work on RHEL6 then...
--
keywords: +patch
Added file: http://bugs.python.org/file25650/distu
Ross Lagerwall added the comment:
Patch seems good (although it doesn't apply cleanly).
Why do you not provide a structure to decode the bytes? I thought relying on
ctypes in the stdlib was not advised...
--
nosy: +rosslagerwall
___
Python tr
Ross Lagerwall added the comment:
Nice work, thanks!
--
assignee: lars.gustaebel -> rosslagerwall
resolution: -> fixed
stage: -> committed/rejected
status: open -> closed
___
Python tracker
<http://bugs.python
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue13031>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Ross Lagerwall :
--
resolution: -> invalid
status: open -> closed
___
Python tracker
<http://bugs.python.org/issue14584>
___
___
Python-bugs-
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue14626>
___
___
Python-bugs-list mailing list
Unsubscribe:
Ross Lagerwall added the comment:
The server encodes the response if the length of the response exceeds
self.encode_threshold, 1400 (presumably there's no point in compression if the
data fits in the MTU anyway).
--
nosy: +rosslagerwall
___
P
Ross Lagerwall added the comment:
In any case, it should be OK to remove libffi_arm_wince?
Is WinCE supported?
--
___
Python tracker
<http://bugs.python.org/issue12
Ross Lagerwall added the comment:
If it is in a non-standard location, try setting the environment variables:
LDFLAGS linker flags, e.g. -L if you have libraries in a
nonstandard directory
CPPFLAGS(Objective) C/C++ preprocessor flags, e.g. -I if you
have headers in a nonstandard
Ross Lagerwall added the comment:
I'm happy to remove the bit about *installing* autoconf altogether.
Do you think the Autoconf section (about regenerating configure) should stay
where it is or be moved somewhere else?
--
___
Python tracker
Ross Lagerwall added the comment:
The first bad revision is:
changeset: 72818:27a36b05caed
branch: 3.2
user:Éric Araujo
date:Sat Oct 08 00:34:13 2011 +0200
summary: Fix distutils byte-compilation to comply with
Ross Lagerwall added the comment:
> Thanks Ross. I don't think this is worth a news item, even though the
> bug was shipped in an alpha. If someone disagrees please add one.
I did add it to the [Tests] section in 9c2b710da3c7. Hardly worth it, but ...
--
nosy: +ro
Ross Lagerwall added the comment:
The output of running rpmbuild from bash:
"""
$ rpmbuild
RPM version 4.9.1.2
Copyright (C) 1998-2002 - Red Hat, Inc.
This program may be freely redistributed under the terms of the GNU GPL
Usage: rpmbuild [-v?] [-bp] [-bc] [-bi] [-bl] [-ba] [
New submission from Ross Lagerwall :
On an up to date Fedora 16:
== CPython 3.3.0a1+ (default:d528b2d2+, Mar 29 2012, 18:04:26) [GCC 4.6.3
20120306 (Red Hat 4.6.3-2)]
== Linux-3.3.0-4.fc16.x86_64-x86_64-with-fedora-16-Verne little-endian
== /home/ross/src/cpythondev/temp/build
Ross Lagerwall added the comment:
Here's the patch ;-)
--
keywords: +patch
nosy: +rosslagerwall
Added file: http://bugs.python.org/file25055/generator.patch
___
Python tracker
<http://bugs.python.org/is
Ross Lagerwall added the comment:
Thanks!
--
resolution: -> fixed
stage: -> committed/rejected
status: open -> closed
___
Python tracker
<http://bugs.python.or
Changes by Ross Lagerwall :
--
assignee: -> rosslagerwall
nosy: +rosslagerwall
resolution: -> fixed
stage: -> committed/rejected
status: open -> closed
type: -> enhancement
___
Python tracker
<http://bugs.py
Ross Lagerwall added the comment:
Perhaps any errors that occur during supports_extended_attributes should cause
it to just return false?
--
nosy: +benjamin.peterson, rosslagerwall
___
Python tracker
<http://bugs.python.org/issue14
Ross Lagerwall added the comment:
I tested the code from msg107484 on Fedora 16 with no change in locale.
Probably OK to close?
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue8
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue14229>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue14127>
___
___
Python-bugs-list mailing list
Unsubscribe:
Ross Lagerwall added the comment:
Cool, thanks for reporting and debugging the issue :-)
--
___
Python tracker
<http://bugs.python.org/issue14079>
___
___
Pytho
Ross Lagerwall added the comment:
This was hopefully fixed in e5a94b56d6bc.
Was it one of the buildbots that was failing?
--
___
Python tracker
<http://bugs.python.org/issue14
Ross Lagerwall added the comment:
This appears to be a buffering issue with the tr program. Replace with ["cat",
"-"] and it works whether the close() is in or not.
To fix this, you need to open up the child process so that it is connected to a
tty. man 4 pts if you want
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue14001>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue13981>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue13964>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue10487>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue10482>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue12157>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue13937>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue13817>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue13893>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue13876>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue13846>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue13845>
___
___
Python-bugs-list mailing list
Unsubscribe:
Ross Lagerwall added the comment:
FreeBSD has a /dev/fd as well as a procfs (deprecated AFAIK).
However, both may not be mounted so a patch would *need* to at least fallback
to the current functionality.
--
___
Python tracker
<h
Ross Lagerwall added the comment:
Thanks for the patch.
However, this cannot as far as I understand be used for the subprocess
implementation due to the limitation of what can be called after a fork() and
before an exec().
Take a look at #8052 for some more discussion of this
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue13779>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Ross Lagerwall :
--
nosy: +ned.deily, ronaldoussoren
___
Python tracker
<http://bugs.python.org/issue13777>
___
___
Python-bugs-list mailing list
Unsub
Ross Lagerwall added the comment:
The reason I made it like that was that it seemed "closer" to the fdopendir()
function which steals the fd for internal use.
However, I agree that it makes more sense to dup() it.
Patch looks good.
--
Ross Lagerwall added the comment:
> I'm currently leaning towards the simple 4-tuple approach
I would also take that approach. It seems simplest to me.
--
___
Python tracker
<http://bugs.python.org
Ross Lagerwall added the comment:
> I see two options:
> 1. rewind the directory stream in fdlistdir()
> 2. document this
>
> Here's a patch for option 1.
Yeah, looks good.
--
___
Python tracker
<http://bug
Ross Lagerwall added the comment:
Thanks!
--
assignee: -> rosslagerwall
resolution: -> fixed
stage: -> committed/rejected
status: open -> closed
type: -> behavior
___
Python tracker
<http://bugs.pyt
Changes by Ross Lagerwall :
--
dependencies: +Add a generic directory walker method to avoid symlink attacks
___
Python tracker
<http://bugs.python.org/issue4
Ross Lagerwall added the comment:
> Has there already been done any work? Ross mentioned he wanted to take a stab?
Unfortunately, I'm rather busy at the moment but when I get some free time and
if no one else wants to work on it then I'l
Ross Lagerwall added the comment:
The patch looks right and seems to fix the issue. Thanks :-)
--
___
Python tracker
<http://bugs.python.org/issue13713>
___
___
New submission from Ross Lagerwall :
806cfe39f729 introduced a regression for http.client read(len).
To see this:
$ ./python test.py
$ wget http://archives.fedoraproject.org/pub/archive/fedora/linux/core/1/SRPMS/
$ diff index.html index2.html
This is a difference in the files (which there
Changes by Ross Lagerwall :
Added file: http://bugs.python.org/file24129/itest.py
___
Python tracker
<http://bugs.python.org/issue12364>
___
___
Python-bugs-list mailin
Ross Lagerwall added the comment:
Retrieving the result of a future after the executor has been shut down can
cause a hang.
It seems like this regression was introduced in a76257a99636. This regression
exists only for ProcessPoolExecutor.
The problem is that even if there are pending work
Ross Lagerwall added the comment:
I'll try & investigate Solaris a bit...
Also, what should be the default terminal size?
Gnome-terminal and xterm seem to default to 80x24, not 80x25.
--
___
Python tracker
<http://bugs.python.or
Ross Lagerwall added the comment:
Nice patch :-)
I think the two function approach works well.
Since you have already checked that termsize is not NULL, Py_DECREF can be used
instead of Py_CLEAR.
Would it not be better to use sys.__stdout__ instead of 1 in the documentation
and to use
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue13694>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue13684>
___
___
Python-bugs-list mailing list
Unsubscribe:
Ross Lagerwall added the comment:
Removed the warnings.
Thanks.
--
assignee: -> rosslagerwall
nosy: +rosslagerwall
resolution: -> fixed
stage: -> committed/rejected
status: open -> closed
versions: +Python 3.3
___
Python tr
1 - 100 of 326 matches
Mail list logo
