New submission from Ross <[EMAIL PROTECTED]>:
When installing Python using any of the following stand-alone installers:
python-2.5.2.amd64.msi
python-2.5.1.amd64.msi
python-2.5.2.msi
all the files and folders are installed in C:\ instead of C:\Python25\
as specified in the installer. Cr
Ross <[EMAIL PROTECTED]> added the comment:
log now attached
Added file: http://bugs.python.org/file9655/python.zip
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.pytho
Ross <[EMAIL PROTECTED]> added the comment:
Checking Progress. This is a big of a show-stopper as it prevents me
from using a number of python dependent packages.
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.pytho
Ross <[EMAIL PROTECTED]> added the comment:
using Orca, I modified the .msi file and python now appears to be
working. I made the following change:
Property -> SecureCustomProperty
Changed value from
REMOVEOLDSNAPSHOT;REMOVEOLDVERSION
to
REMOVEOLDSNAPSHOT;REMOVEOLDVERSION;TARGETD
New submission from Ross :
It would be most helpful if a method could be included in the TarFile
class of the tarfile module and the ZipFile class of the zipfile module
that would remove a particular file (either given by a name or a
TarInfo/ZipInfo object) from the archive.
Usage to remove a
Changes by Ross :
--
components: +IO
___
Python tracker
<http://bugs.python.org/issue6818>
___
___
Python-bugs-list mailing list
Unsubscribe:
http://mail.pyth
Ross added the comment:
Slight change to:
"Such a method should probably only apply to archives that are in append
mode as write mode would erase the original archive and read mode should
render the archive immutable."
The method should probably still apply to an archive in write mo
Ross added the comment:
In light of Lars's comment on the matter, perhaps this functionality
could be added to zip files only. Surely it can be done, considering
that numerous utilities and even Windows Explorer provide such
functionality. I must confess that I am unfamiliar with the
New submission from Ross:
If convert_charrefs is set to true the final data section is not return by
feed(). It is held until the next tag is encountered.
---
from html.parser import HTMLParser
class MyHTMLParser(HTMLParser):
def __init__(self):
HTMLParser.__init__(self
Ross added the comment:
That would make sense.
Might also be worth mentioning the difference in behaviour with
convert_charrefs = True/False as that was what led me to think this was a bug.
--
___
Python tracker
<http://bugs.python.org/issue23
Ross Lagerwall added the comment:
Updated patch based on Eric's comments:
Store _supports_safe_rmdir at the module level.
Move imports up to module level
Skip test on non-threading build
--
Added file: http://bugs.python.org/file23261/i4489_v4.
Ross Lagerwall added the comment:
Attached is a patch which adds dirfd= as a keyword argument.
--
keywords: +patch
Added file: http://bugs.python.org/file23272/i12797.patch
___
Python tracker
<http://bugs.python.org/issue12
Ross Lagerwall added the comment:
http://bugs.python.org/review/4489/diff/3383/10563#newcode319
Lib/test/test_shutil.py:319: @unittest.skipIf(threading == None, 'requires
threading')
On 2011/10/07 19:29:47, eric.araujo wrote:
> You can just say skipUnless(threading, 'm
Ross Lagerwall added the comment:
#11454 is another case where pre-parsing and pickling the regular expressions
in the email module may improve import time considerably.
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue13
Ross Lagerwall added the comment:
I see this requires dpkg-architecture which isn't always available. While it
isn't hard to install it, it isn't very clear that this is the cause of the nis
and crypt modules failing to build on a fresh install of 11.10.
What would be nice i
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue13195>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue13196>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue13197>
___
___
Python-bugs-list mailing list
Unsubscribe:
Ross Lagerwall added the comment:
I guess that would make it more general...
I'll play around with it for a bit. It mustn't become too hard to use though
since the original point was to simplify the opening of files :-)
--
___
Pyth
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue13263>
___
___
Python-bugs-list mailing list
Unsubscribe:
Ross Lagerwall added the comment:
Some functions would be easy to split off into separate modules conceptually
like the sched_* functions and the cap_* functions (see #1615158).
However, certain groups of functions like the *at functions cover a lot of
different functionality and don&#
Ross Lagerwall added the comment:
What would you envisage the API for the custom opener to look like?
--
___
Python tracker
<http://bugs.python.org/issue12
Ross Lagerwall added the comment:
Before I implement it properly, is this the kind of api that's desired?
"""
import os
import io
class MyOpener:
def __init__(self, dirname):
self.dirfd = os.open(dirname, os.O_RDONLY)
def open(self, path, flags, m
Ross Lagerwall added the comment:
The attached patch adds the opener keyword + tests.
--
Added file: http://bugs.python.org/file23550/opener.patch
___
Python tracker
<http://bugs.python.org/issue12
Ross Lagerwall added the comment:
Updated patch:
* checks for long overflow
* raises original exception if opener returns null
* makes it explicit that "opener" must return an open file descriptor.
I don't think that mode should be passed in since it is not specified in the
par
Ross Lagerwall added the comment:
Thanks (and for the English lesson ;-) )
--
assignee: -> rosslagerwall
resolution: -> fixed
stage: needs patch -> committed/rejected
status: open -> closed
___
Python tracker
<http://bugs.python
Ross Lagerwall added the comment:
"""
import time
import sys
t = time.gmtime(time.time())
s = time.strftime('%Z', t)
print(s)
time.mktime((-1, 1, 1, 0, 0, 0, -1, -1, -1))
t = time.gmtime(time.time())
s = time.strftime('%Z', t)
print(s)
""&qu
Ross Lagerwall added the comment:
It outputs:
SAST
LMT
LMT
An equivalent C program to the first test:
"""
#include
#include
#include
int main() {
time_t t;
struct tm *tmp;
t = time(NULL);
tmp = localtime(&t);
char str[200];
strftime(str, siz
Ross Lagerwall added the comment:
Thanks!
--
assignee: -> rosslagerwall
nosy: +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/issue6397>
___
___
Python-bugs-list mailing list
Unsubscribe:
Ross Lagerwall added the comment:
Thanks Charles, I'll take your comments into account and take a look at making
a general walker method.
--
___
Python tracker
<http://bugs.python.org/i
Ross Lagerwall added the comment:
+ increases this value, c:func:`devpoll` will return a possible
+ incomplete list of active file descriptors.
I think this should change to:
+ increases this value, c:func:`devpoll` will return a possibly
+ incomplete list of active file descriptors
Ross Lagerwall added the comment:
Is write()ing a devpoll fd a blocking operation in the kernel?
Does it need to have Py_BEGIN_ALLOW_THREADS around it?
The same question applies for open()ing it.
Obviously, the ioctl() call *is* blocking
Ross Lagerwall added the comment:
Also, you can use Py_RETURN_NONE instead of:
+Py_INCREF(Py_None);
+return Py_None;
--
___
Python tracker
<http://bugs.python.org/issue6
Ross Lagerwall added the comment:
That was thorough :-) Seems OK though.
+if (n < size) {
+PyErr_SetString(PyExc_IOError, "failed to write all pollfds. "
+"Please, report in http://bugs.python.org/";);
If n < size, it's not a Pytho
Ross Lagerwall added the comment:
Has it been reported?
--
___
Python tracker
<http://bugs.python.org/issue13309>
___
___
Python-bugs-list mailing list
Unsub
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue13424>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue13517>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue13559>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue13564>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue13530>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue13578>
___
___
Python-bugs-list mailing list
Unsubscribe:
Ross Lagerwall added the comment:
getstatusoutput() is broken given that it doesn't work on windows yet it should.
I'd also recommend leaving the behavior as is and deprecating the function (and
getoutput() while we're at it).
A related bug (#10197) also recomme
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
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue13684>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue13694>
___
___
Python-bugs-list mailing list
Unsubscribe:
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
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:
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
Changes by Ross Lagerwall :
Added file: http://bugs.python.org/file24129/itest.py
___
Python tracker
<http://bugs.python.org/issue12364>
___
___
Python-bugs-list mailin
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
Ross Lagerwall added the comment:
The patch looks right and seems to fix the issue. Thanks :-)
--
___
Python tracker
<http://bugs.python.org/issue13713>
___
___
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
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:
Thanks!
--
assignee: -> rosslagerwall
resolution: -> fixed
stage: -> committed/rejected
status: open -> closed
type: -> behavior
___
Python tracker
<http://bugs.pyt
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:
> 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:
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.
--
Changes by Ross Lagerwall :
--
nosy: +ned.deily, ronaldoussoren
___
Python tracker
<http://bugs.python.org/issue13777>
___
___
Python-bugs-list mailing list
Unsub
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue12158>
___
___
Python-bugs-list mailing list
Unsubscribe:
Ross Lagerwall added the comment:
Python 3.3 (as yet unreleased) supports the lutimes function:
http://docs.python.org/dev/py3k/library/os.html#os.lutimes
Python 2.7 is not getting any more features so it will not be added.
I'm changing the title to the second part of your que
Ross Lagerwall added the comment:
Out of interest, is there any reason that the configure check for pipe2 is a
special case near the bottom of configure.in instead of with all the other
function checks in the AC_CHECK_FUNCS[] section in the middle?
I know this patch didn't write the conf
Ross Lagerwall added the comment:
>> Also, the pure python implementation of subprocess for posix can now >> be
>> updated to use pipe2 if it exists (previously on _posixsubprocess.c
>> used it).
> I don't understand the last part :-)
> What do you suggest
Ross Lagerwall added the comment:
Could this be related to http://bugs.python.org/issue6559#msg123958?
Or Issue10826?
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue12
Ross Lagerwall added the comment:
> These failures were specific to Solaris/OpenIndiana: they were "door files"
Yeah, I saw that but thought maybe the reason was similar, some library
function in the child process was opening a file and not closing it properly.
> using -m
Ross Lagerwall added the comment:
All stable buildbots appear to be green, so closing...
--
nosy: +rosslagerwall
resolution: -> fixed
status: open -> closed
___
Python tracker
<http://bugs.python.org/i
Ross Lagerwall added the comment:
Attached is a patch for review.
It adds st_atim, st_ctim and st_mtim.
They are defined even when the underlying system does not have nanosecond
precision.
--
keywords: +patch
stage: -> patch review
Added file: http://bugs.python.org/file22
Ross Lagerwall added the comment:
Here is an updated patch that uses the atim, ctim and mtim variables (which are
assigned based on various #ifdefs).
This should now work on Linux, BSD and Windows.
--
Added file: http://bugs.python.org/file22279/issue11941_2.patch
Ross Lagerwall added the comment:
Thanks!
--
resolution: -> accepted
stage: commit review -> committed/rejected
status: open -> closed
___
Python tracker
<http://bugs.python.o
Ross Lagerwall added the comment:
> Nit: when a patch gets committed and the issue closed, the preferred
> resolution is "fixed" ;)
> (see http://docs.python.org/devguide/triaging.html#resolution)
Sure, I will try remember that.
--
___
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue12326>
___
___
Python-bugs-list mailing list
Unsubscribe:
Ross Lagerwall added the comment:
> That's exactly my point.
> Code checking sys.platform against 'linux2' is already broken, there's
> no point in complicating the code further, or adding a new constant.
> If you want to check for a specific operating system, t
Changes by Ross Lagerwall :
--
status: pending -> closed
___
Python tracker
<http://bugs.python.org/issue10635>
___
___
Python-bugs-list mailing list
Unsubscri
Ross Lagerwall added the comment:
> The right approach is to use sigblock/sigsetmask before creating the
> process, and then again after creating it. Unfortunately, these aren't
> exposed from the signal module.
Since 3.3, pthread_sigmask is exposed so the right approach ca
Ross Lagerwall added the comment:
Here's a patch to add the two functions (with docs and tests).
You'll need to run autoreconf before compiling.
--
assignee: -> rosslagerwall
keywords: +patch
nosy: +rosslagerwall
stage: -> patch review
type: -> feature reque
Ross Lagerwall added the comment:
Thanks for the review.
> Why do you wait until the end of PyInit_signal() to set initialized to 1?
Fixed.
> If this variable is only present ... but I see that the init function of the
> posixmodule.c has also a static initialized variable.
I simply
Ross Lagerwall added the comment:
> What do you mean? signal.SIGPOLL exists in Python 3.3.
Right, si_band added.
> - test_sigwaitinfo(), test_sigtimedwait_poll(),
> test_sigwaitinfo_interrupted() are called from a child process. In
> test_wait(), I chose to write manually to std
Ross Lagerwall added the comment:
Thanks!
--
assignee: -> rosslagerwall
nosy: +rosslagerwall
resolution: -> fixed
stage: -> committed/rejected
status: open -> closed
versions: +Python 3.3
___
Python tracker
<http://bugs.python
Ross Lagerwall added the comment:
>From the FreeBSD 6.4 buildbot:
"""
[172/356/2] test_signal
error: 'NoneType' object has no attribute 'si_signo'
error: False is not true
test test_signal failed -- multiple errors occurred; run in verbose mode for
Ross Lagerwall added the comment:
The tests seem to be working fine on all the buildbots now...
Thanks!
--
resolution: -> accepted
stage: commit review -> committed/rejected
status: open -> closed
___
Python tracker
<http://bug
New submission from Ross Lagerwall :
6d6099f7fe89 introduced a regression.
It causes the following code to hang fairly reliably for me:
"""
import concurrent.futures
import math
def is_prime(n):
print(sqt(81)) < Note the type error!
de
Ross Lagerwall added the comment:
Further analysis seems to indicate that it is a race between the shutdown code
and the processes finishing.
This code hangs:
"""
executor = concurrent.futures.ProcessPoolExecutor(2)
executor.map(is_prime, [1, 2])
executor.shutdown()
"&q
Ross Lagerwall added the comment:
Yes, the patch does appear to fix the issue.
Thanks
--
___
Python tracker
<http://bugs.python.org/issue12456>
___
___
Pytho
Ross Lagerwall added the comment:
Looks good, the patch seems to fix the problem.
This section of code indicates that the accepting socket shouldn't be in the
write set...
def handle_write_event(self):
if self.accepting:
# Accepting sockets shouldn't get a w
Ross Lagerwall added the comment:
I think this is a duplicate of #3177.
--
nosy: +rosslagerwall
resolution: -> duplicate
status: open -> closed
___
Python tracker
<http://bugs.python.org/i
Ross Lagerwall added the comment:
Closed #12522 as a duplicate.
It contains a link to a discussion on python-ideas requesting the feature.
--
nosy: +rosslagerwall
versions: +Python 3.3 -Python 3.2
___
Python tracker
<http://bugs.python.
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue12494>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue12517>
___
___
Python-bugs-list mailing list
Unsubscribe:
Ross Lagerwall added the comment:
> I don't like the idea of adding an argument which doesn't have a
> counterpart in the POSIX version (especially to address such corner
> cases).
Indeed, it seems rather messy for a corner case that may well not exist.
If there are defini
Ross Lagerwall added the comment:
> That's what I thought, it's really uncommon: in that case, I'm
> reluctant to making such a change, for the reason explained above.
> Ross, Victor?
Even if Linux did accept a a length of 0 (which it doesn't since 2.6.12), what
Ross Lagerwall added the comment:
It is indeed a problem.
It seems like the problem comes about due to the "swapping" of fds.
i.e. using stdout as stderr. The reverse appears to work due to the order in
which the dup() calls are performed.
Attached is a patch which fixes
Ross Lagerwall added the comment:
Thanks!
--
assignee: -> rosslagerwall
resolution: -> fixed
stage: -> committed/rejected
status: open -> closed
versions: +Python 3.3
___
Python tracker
<http://bugs.python
Ross Lagerwall added the comment:
Attached is a patch which tests all combinations and includes a testcase.
--
nosy: +pitrou
Added file: http://bugs.python.org/file22747/i12607_v2.patch
___
Python tracker
<http://bugs.python.org/issue12
Ross Lagerwall added the comment:
communicate() requires setting stdin, stdout and stderr to subprocess.PIPE
which defeats the purpose of the test: setting them to 0, 1 and 2 (in various
orders) so that they need to be swapped...
--
___
Python
Changes by Ross Lagerwall :
--
assignee: -> rosslagerwall
resolution: -> fixed
stage: -> committed/rejected
status: open -> closed
type: -> behavior
___
Python tracker
<http://bugs.pyt
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue12650>
___
___
Python-bugs-list mailing list
Unsubscribe:
Ross Lagerwall added the comment:
The patches look good and seem to fix the issue.
--
___
Python tracker
<http://bugs.python.org/issue12650>
___
___
Python-bug
Ross Lagerwall added the comment:
> Formally I believe that the 'accepted' resolution is for committed feature
> requests
That's what I also thought originally.
Perhaps this should be discussed on Python-Dev to see what people think it
should mean?
--
Changes by Ross Lagerwall :
--
nosy: +rosslagerwall
___
Python tracker
<http://bugs.python.org/issue12702>
___
___
Python-bugs-list mailing list
Unsubscribe:
Ross Lagerwall added the comment:
An initial patch.
The first problem is that before, os.path.realpath('') == os.path.realpath('.')
but this is not true for realpath(3).
Thus, the test suite does not run because it relies on this behaviour.
--
keywords: +pa
1 - 100 of 408 matches
Mail list logo
