Ross Lagerwall added the comment:
Thanks for the comments.
I implemented sethostid() - its not actually in the posix spec, but linux/*bsd
have it although the signature for FreeBSD is a bit different.
I implemented gethostname(). Both get/sethostname() now use FSDefault encoding.
gethostname
Ross Lagerwall added the comment:
This patch takes out sethostname() and gethostname(). I'll open up a new issue
to add sethostname() to the socket module.
--
Added file: http://bugs.python.org/file20311/10812_v5.patch
___
Python tracker
New submission from Ross Lagerwall :
This patch adds sethostname to the socket module (since socket has
gethostname()).
--
components: Library (Lib)
files: sethostname.patch
keywords: patch
messages: 125761
nosy: giampaolo.rodola, loewis, rosslagerwall
priority: normal
severity: normal
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue10822>
___
___
Python-bugs-list mailing list
Unsubscribe:
New submission from Ross Lagerwall :
Attached is a patch which implements os.sendfile for unix systems (linux,
freebsd, apple, solaris, dragonfly).
It takes the iov initialization code and off_t parsing from i10812.
It encapsulates all the functionality from the various sendfiles which means
Ross Lagerwall added the comment:
Ok, I figured it out to link with sendfile on solaris. Here is the updated
patch.
--
Added file: http://bugs.python.org/file20352/sendfile_v2.patch
___
Python tracker
<http://bugs.python.org/issue10
Ross Lagerwall added the comment:
It seems to work fine on OpenIndiana (running normally or as root).
According to the posix specification, "It is implementation-defined whether
getgroups() also returns the effective group ID in the grouplist array."
But, id -G prints all group ID
Ross Lagerwall added the comment:
This issue has been fixed on 3.2.
--
nosy: +rosslagerwall
versions: -Python 3.2
___
Python tracker
<http://bugs.python.org/issue4
New submission from Ross Lagerwall :
I think there is a small docs bug: it says that multiprocessing.Semaphore is "a
bounded semaphore". Shouldn't it says that it is just "a semaphore".
Attached is a patch to fix this.
--
components: Library (Lib)
files:
Ross Lagerwall added the comment:
Just to be clear:
There are 3 different interfaces.
The basic one with the offset included & no headers/trailers is supported by
all the platforms, including Linux.
The one with offset as None is only supported by Linux.
The one with headers/trailers/flag
Ross Lagerwall added the comment:
I've just tried it against r87935 and it applies cleanly.
Perhaps you didn't apply the patch correctly (it requires "-p1" since it was a
Mercurial diff), try:
patch -p1 < sendfile_v2.patch
With regards to the different arguments,
Changes by Ross Lagerwall :
--
assignee: -> d...@python
components: +Documentation
nosy: +d...@python
___
Python tracker
<http://bugs.python.org/issu
Ross Lagerwall added the comment:
Oh sorry, that was because it changed configure.in so "autoreconf" needs to be
run to regenerate configure & pyconfig.h.in.
I thought that patches weren't meant to include the regenerated files.
Especially since differences in the versio
Ross Lagerwall added the comment:
How about this?
Instead of just losing the data that's been read so far in readline(), this
patch adds the data as a new field to the exception that is thrown - this way
the semantics remain exactly the same but the data is not discarded when a
ti
Ross Lagerwall added the comment:
> That complicates things quite a bit,
> especially given that it has to be grafted on at least two layers of the
> IO stack (the raw IO layer, and the buffered IO layer).
Also the TextIO layer I think.
> That's my opinion too. So, instead, o
Ross Lagerwall added the comment:
Attached patch disallows further reads after a timeout.
--
Added file: http://bugs.python.org/file20398/i7322.patch
___
Python tracker
<http://bugs.python.org/issue7
Ross Lagerwall added the comment:
Shouldn't this be closed? CGIHTTPServer *has* been updated to use subprocess on
windows and the dependency has been closed as wont fix.
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.py
Ross Lagerwall added the comment:
I think all that is needed is a documentation patch. Attached is a doc patch
which changes the doc to explicitly describe what happens on unix & windows (as
described by Peter).
--
components: +Documentation
keywords: +patch
nosy: +rosslager
Ross Lagerwall added the comment:
After trying to reproduce this bug in 2.7.1 & 3.2b2 and failing, I think this
should be closed (even the OP couldn't reproduce it in anything other than 2.4).
--
nosy: +rosslagerwall
___
Python track
New submission from Ross Lagerwall :
If a mmap length 0 is used and an offset is used as well, the size to mmap() is
calculated as the size of the file instead of the size of the file minus
offset. This means that trying to access a certain part does not result in an
index error but a
Ross Lagerwall added the comment:
I think this can be closed.
msync() is only called in mmap.flush() and it is checked for an error.
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue2
Ross Lagerwall added the comment:
Attached is a fix to make offset use off_t. This means that mmap will work with
offset > 2GB on 32bit systems.
It also fixes that mmap.size() returns the correct value for files > 2GB on
32bit systems.
The first issue of msg78055 was fixed in issu
Ross Lagerwall added the comment:
>From what I can see, this issue is in memoryview and allows memoryview to
>export a readonly buffer as writable (because memoryview.getbuffer() removes
>the writable flag from flags before calling the underlying buffer).
This causes segfaults w
Ross Lagerwall added the comment:
Attached is an updated patch with a simpler test.
--
Added file: http://bugs.python.org/file20440/i10451_v2.patch
___
Python tracker
<http://bugs.python.org/issue10
Ross Lagerwall added the comment:
And a simple fix for the test_getargs2 test - it wraps the memoryview around a
bytearray.
--
Added file: http://bugs.python.org/file20441/testfix.patch
___
Python tracker
<http://bugs.python.org/issue10
Ross Lagerwall added the comment:
I tried to reproduce the bug on 2.7 & 3.2 using libuuid version 2.17.2 and up
to 100 threads but couldn't.
Perhaps there was an issue with the uuid library function that he was using and
threading?
--
nosy: +ross
Ross Lagerwall added the comment:
Fixed small #ifdef error with fstatat.
--
Added file: http://bugs.python.org/file20460/i4761_v6.patch
___
Python tracker
<http://bugs.python.org/issue4
Ross Lagerwall added the comment:
A few small fixes for OS X:
It has no return value for sethostid() and sets different errno if permission
denied,
waitid() is broken - so its disabled,
the timeval struct used in futimes and lutimes is defined slightly differently.
--
Added file: http
New submission from Ross Lagerwall :
The fix for issue10916 commited in r88022 introduces this line:
map_size = st.st_size - offset;
If offset > st.st_size, map_size is negative. This should cause the mmap system
call to return -1 and set errno.
However, given a certain size of offset, si
Ross Lagerwall added the comment:
Yes I think subprocess is working correctly.
Since this feature request is 2 years old now without any interest, I think it
should be closed. If the functionality is needed, it can always be programmed
by the user when needed.
--
nosy
Ross Lagerwall added the comment:
Attached is a patch based on the original patch, meant to have better
performance.
On my PC, this:
import sys, time, uuid
def uu(n):
t = time.time()
for x in range(n):
uuid.uuid1()
print('%.3f microseconds' % ((time.
Ross Lagerwall added the comment:
Attached is a new sendfile patch which fixes the issue with FreeBSD (and Mac OS
X & DragonFly BSD from what I can see).
With regards to anacrolix's request, I think what Martin said in msg126049.
i.e. if we want to provide a unifying layer on top of
Ross Lagerwall added the comment:
Attached is an updated patch that uses keyword arguments.
Using an offset with Linux was always supported although I have cleaned up the
documentation a bit to make that clearer.
E.g. the following script sends part of a file over a socket (shows using an
Ross Lagerwall added the comment:
OK, updated documentation and tests.
> Why special case these? Why can't Mac OS X and FreeBSD write those manually
> into the output file descriptor.
These can be a crucial part of certain protocols such as HTTP to ensure that a
minimal am
Ross Lagerwall added the comment:
This has been fixed in 2.7 and 3.2 so I think that this issue can be closed.
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue9
Ross Lagerwall added the comment:
The docs should be updated. This has been noted in msg54949 and
http://www.enricozini.org/2009/debian/python-pipes/
Perhaps this example will make it clear:
import subprocess
p1 = subprocess.Popen(["yes"], stdout=subprocess.PIPE)
p2 = subprocess.Po
Ross Lagerwall added the comment:
For trailers to work, I think the line:
self.assertEqual(data, "abcde12345")
should be:
self.assertEqual(data, b"abcde12345")
Also not that tests like this:
if not sys.platform.startswith('linux'):
perhaps should also in
Ross Lagerwall added the comment:
With no changes, I get:
==
FAIL: test_trailers (test.test_os.TestSendfile)
--
Traceback (most recent call last):
File "
Ross Lagerwall added the comment:
OK, this new patch applies cleanly, uses support.unlink and hexadecimal
constants.
I left the off_t handling as is (it seems to work on *nix testing). Perhaps
someone can handle the Windows side?
--
Added file: http://bugs.python.org/file20722
Ross Lagerwall added the comment:
OK, I'm happy to not return the file offset. However, I still think that
headers and trailers should remain as is since this matches the native
interface very closely.
--
___
Python tracker
Ross Lagerwall added the comment:
> will this patch support off_t for "mmap.resize" as well?
> ftruncate uses off_t for the length of a file as well.
No, it doesn't because resize() resizes the amount mmapped in memory which
can't be more than ssize_t anyway. Howeve
Ross Lagerwall added the comment:
32-bit computers can address up to 4GiB of memory and 64-bit computers can
address much more than this. mmap() allows a file to be mapped to a location in
memory - the actual amount of memory that exists doesn't matter. This is the
reason why a 5GiB fil
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue13779>
___
___
Python-bugs-list mailing list
Unsubscribe:
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
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
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue13845>
___
___
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/issue13876>
___
___
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/issue13817>
___
___
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/issue12157>
___
___
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/issue10487>
___
___
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/issue13981>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue14001>
___
___
Python-bugs-list mailing list
Unsubscribe:
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
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:
Cool, thanks for reporting and debugging the issue :-)
--
___
Python tracker
<http://bugs.python.org/issue14079>
___
___
Pytho
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue14127>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue14229>
___
___
Python-bugs-list mailing list
Unsubscribe:
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
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
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:
Thanks!
--
resolution: -> fixed
stage: -> committed/rejected
status: open -> closed
___
Python tracker
<http://bugs.python.or
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
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:
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] [
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 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:
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:
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:
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:
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
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue14626>
___
___
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/issue13031>
___
___
Python-bugs-list mailing list
Unsubscribe:
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
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:
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:
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:
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:
Closed issue14872 as a duplicate of this.
--
assignee: astrand ->
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue1
Changes by Ross Lagerwall :
--
assignee: astrand ->
___
Python tracker
<http://bugs.python.org/issue1191964>
___
___
Python-bugs-list mailing list
Unsubscri
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue1191964>
___
___
Python-bugs-list mailing list
Unsubscribe:
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
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:
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:
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:
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:
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:
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:
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:
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:
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:
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
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:
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
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
201 - 300 of 326 matches
Mail list logo
