Tim Golden added the comment:
Patch attached with code & test which fixes this within _subprocess.c at least
for Windows
--
assignee: -> tim.golden
Added file: http://bugs.python.org/file18398/3210.r83741.patch
___
Python tracker
Changes by Tim Golden :
--
nosy: +tim.golden
___
Python tracker
<http://bugs.python.org/issue9503>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Tim Golden :
--
nosy: +tim.golden
___
Python tracker
<http://bugs.python.org/issue4453>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Tim Golden :
--
nosy: +tim.golden
___
Python tracker
<http://bugs.python.org/issue6040>
___
___
Python-bugs-list mailing list
Unsubscribe:
Tim Golden added the comment:
Not necessarily; pyreadline takes over from the standard console
functionality on Windows in order to emulate a more Unix-ish
approach. I prefer the Windows default.
There's nothing to stop someone downloading and installing
pyreadline as a third-party packa
Tim Golden added the comment:
Although I'm the implementer of the patch (the concept
was discussed way back on c.l.py after a naive poster's
original request) I'm probably +0 myself. It's an attempt
to replace os.access' next-to-useless behaviour on Windows
with some
Tim Golden added the comment:
That's (still...) a known issue with Windows file associations and
redirects:
http://support.microsoft.com/kb/321788
In theory it was fixed way back when. In practise...
On 30/06/2010 15:04, Vojtech Fried wrote:
>
> Vojtech Fried added the com
Tim Golden added the comment:
I think we're saying the same thing :)
The simplest thing to do here is to create a win_ismount function
in posixmodule.c which does the attributes / reparse tag dance and
returns True/False and use that wherever it's needed to support this
concept und
Tim Golden added the comment:
Error no longer occurs
--
resolution: -> fixed
status: open -> closed
___
Python tracker
<http://bugs.python.org/
Changes by Tim Golden :
--
resolution: -> out of date
status: open -> closed
___
Python tracker
<http://bugs.python.org/issue8869>
___
___
Python-bugs-list
Tim Golden added the comment:
... of course you still need to get the reparse tag to determine whether this
is a mount point so the file attributes alone in this case are not enough.
--
___
Python tracker
<http://bugs.python.org/issue9
Tim Golden added the comment:
All we need to do is check the FILE_ATTRIBUTE_REPARSE_POINT
in the file attributes. Frustratingly, we grab file attributes
a dozen times in posixpath.c only to throw most of it away.
Is there a case for adding an "attributes" function to os.path
which e
Tim Golden added the comment:
Switching to Python 3.2 as this essentially constitutes a behaviour change and
2.6 is in bugfix mode and 2.7 is about to enter rc2. It would certainly be
possible to use one of the volume APIs under the covers. Would you be willing
to offer a patch to, say
Changes by Tim Golden :
--
assignee: -> d...@python
components: +Documentation
nosy: +d...@python
stage: -> patch review
versions: +Python 3.2
___
Python tracker
<http://bugs.python.org/
New submission from Tim Golden :
The reference to PyDateTime_IMPORT in the C API datetime docs is marked
up as cfunction, giving it a pair of brackets which render it invalid.
There is also no indication of where in the code it should be invoked.
Patch attached.
--
files
Tim Golden added the comment:
I'm unlikely to get to it soon. If there's no urgency I can
look at it later. FWIW, it's not something I'm especially
familiar with.
On 12/06/2010 01:02, STINNER Victor wrote:
>
> STINNER Victor added the comment:
>
> Tim: are you
Tim Golden added the comment:
Since execfile is basically shorthand for exec (open (filename).read ()),
and since open (filename) *does* support the full range of filepath
syntax on Windows, and since execfile has been removed in py3k in favour
of exec (open ...)), and since Python 2.x is
Tim Golden added the comment:
Sorry, typing too fast:
http://bugs.python.org/issue7443 - test.support.unlink issue on Windows
platform
at least insofar as the issue applies to Windows. I imagine that the
OS X thingis completely different
Tim Golden added the comment:
This is basically issue 7743 which is a combination of:
* Using the same filename for all tests in one process
* Something (TSvn / Virus Checker) having a delete-share handle
* Not renaming the file before removing it in test.support.unlink
MvL suggested a change
Changes by Tim Golden :
--
nosy: -tim.golden
___
Python tracker
<http://bugs.python.org/issue8273>
___
___
Python-bugs-list mailing list
Unsubscribe:
New submission from Tim Golden :
If test_heapq is run before test_import on the current py3k head,
test_import will fail as per the attached traceback.
python -m test.regrtest -W test_heapq test_import > test_import.log
At a glance I can't see any obvious reason why test_heapq should
New submission from Tim Golden :
test_linecache in the current py3k branch is failing on my WinXP machine with
ERROR_SHARING_VIOLATION.
The attached trivial patch appears to fix the problem, altho' I'm unfamiliar
with the module in question so it may be that there's
Tim Golden added the comment:
Well that's embarrassing: I updated but didn't rebuild. Sorry for the noise;
all tests passing now. Please close the call.
--
___
Python tracker
<http://bugs.python.
Tim Golden added the comment:
Yes:
C:\temp>\work_in_progress\make-snapshots\branches
Python 3.2a0 (py3k:80030, Apr 13 2010, 11:13:13)
Type "help", "copyright", "credits" or "license"
>>>
New submission from Tim Golden :
test_pickle failing on WinXP
http://svn.python.org/projects/python/branches/py3k/Lib r80044
==
ERROR: test_unicode (__main__.CPicklerTests
Changes by Tim Golden :
--
nosy: +mark.dickinson
___
Python tracker
<http://bugs.python.org/issue8385>
___
___
Python-bugs-list mailing list
Unsubscribe:
New submission from Tim Golden :
There is a reference to _winreg left in test_winsound. Trivial patch attached
renames this to winreg.
--
components: Tests
files: test_winsound.patch
keywords: patch
messages: 103042
nosy: tim.golden
severity: normal
status: open
title: _winreg
Tim Golden added the comment:
> Then we shouldn't use DeleteFile in the first place to delete the file,
> but instead CreateFile, with DELETE access (and FILE_SHARE_DELETE
> sharing). If that fails, we need to move the file to the bin
> (see unlink_nt for details).
I see what
Tim Golden added the comment:
I'm afraid that the problem doesn't lie in the unlink: DeleteFile
succeeds. The problem is that the file is only marked for delete
until such time as the last SHARE_DELETE handle on it is closed.
Until that time, an attempt to (re)create the file for anyt
Tim Golden added the comment:
I put together a trivial patch against the 2.7 trunk (basically: I added
a os.rename before the os.remove in test_support.unlink) and reran my
test harness with test_zipfile... and it still failed because, of course,
test_zipfile calls shutil.rmtree which bypasses
Tim Golden added the comment:
In one window run the attached script (assumes you have pywin32 installed) with
a parameter of the directory the TESTFN file will end up in. Then run, eg,
test_zipfile in another window. For me:
c:\temp> watch_dir.py C:\work_in_progress\make-snapshots\tr
Tim Golden added the comment:
This is basically a rerun of this discussion a couple of years ago:
http://mail.python.org/pipermail/python-dev/2008-April/078333.html
The problem certainly still happens against trunk -- I have a semi-aggressive
test-harness which can cause it to reproduce
Tim Golden added the comment:
Seeing the same thing on 32-bit WinXP on x86
On 07/04/2010 14:34, Stefan Krah wrote:
>
> New submission from Stefan Krah:
>
> On Windows/amd64, I get loads of pickling errors in test_multiprocessing.
>
> Type 1 error:
>
> Tracebac
Tim Golden added the comment:
On 31/03/2010 14:20, Michael Foord wrote:
> - TESTFN
>
> This is a global for setting the directory temporary files are created in?
> Don't think I like the global approach. Which functions is it used by?
It's used *all over the place*. I
Tim Golden added the comment:
Just think about it for a minute:
t = (1, 2)
print id (t), t
t += (1, 2, 3)
print id (t), t
Not mutating, merely creating a new new object
and giving it the same name
--
nosy: +tim.golden
___
Python tracker
<h
Tim Golden added the comment:
Sorry; the email interface messed that up. The code
which triggered the error was:
import _winreg
list (
_winreg.IterValue (
_winreg.OpenKey (_winreg.HKEY_CURRENT_USER, "Console")
)
)
--
Tim Golden added the comment:
Traceback (most recent call last):
File "", line 1, in
WindowsError: [Error 6] The handle is invalid
I suspect that .iterkeys / .itervalues would be more
acceptable spellings as those mirror the dict methods.
Whether the idea of extending _winreg
Changes by Tim Golden :
--
nosy: +tim.golden
___
Python tracker
<http://bugs.python.org/issue7344>
___
___
Python-bugs-list mailing list
Unsubscribe:
Tim Golden added the comment:
Don't know what exact release OS X ships with, but it
looks like you're being bitten by this:
http://bugs.python.org/issue5261
--
nosy: +tim.golden
___
Python tracker
<http://bugs.python.
Tim Golden added the comment:
Attached is a patch against r73685 of the documentation for subprocess
which adds some information about using shell=True on Windows. I plan
to do some more general-purpose docs for subprocess on Windows, but as
I've failed to get round to them for a year
Tim Golden added the comment:
Making something executable on Windows has nothing to
do with file permissions. You can set them as much
as you like, but executability is determined by file
associations, possibly in association with PATHEXT
settings. AFAICT, the current Python installer does
this
Tim Golden added the comment:
What's superior about .tar.gz? (This is a genuine question).
--
nosy: +tim.golden
title: Native (and default) tarfile support for setup.py sdist in distutils on
Windows -> Native (and default) tarfile support for setup.py sdist in
dist
Tim Golden added the comment:
Eric Smith wrote:
> Eric Smith added the comment:
>
>> So is this a cosmetic issue or a functional issue?
>
> It's a cosmetic issue.
>
>> Also, even if it could figure that out, how would it know whether
>> a part
Tim Golden added the comment:
I doubt there's any real likelihood of the official Python 2.5.4 being
generated with a different compiler: it would mean, in principle, that all
binary extensions would have be recompiled to ensure there were no issues. I
suggest you ask on the Python ma
Changes by Tim Golden :
--
assignee: -> tarek
components: +Distutils
nosy: +tarek
type: -> behavior
versions: +Python 2.7
___
Python tracker
<http://bugs.python.org/
Index: bdist_msi.py
===
--- bdist_msi.py(revision 72507)
+++ bdist_msi.py(working copy)
@@ -141,6 +141,8 @@
bdist_base = self.get_finalized_command('bdist').bdist_base
self.bdist_dir = os.path.
Tim Golden added the comment:
Edmund Eyles wrote:
> New submission from Edmund Eyles :
>
> The documentation for the webbrowser module at
> http://www.python.org/doc/2.5/lib/module-webbrowser.html refers to the
> use of the BROWSER environment variable as a means to control whi
New submission from Tim Golden :
tools/msi/msi.py is still trying to copy the README file from the
lib/test directory. This file was removed in r70872. Patch attached
against r71393 of msi.py.
--
components: Build, Demos and Tools, Windows
files: msi.r71393.patch
keywords: patch
Tim Golden added the comment:
Well it may be uncommon, but it's perfectly easily reproducible:
Just try to reinstall / repair from a file not called
python-2.5.2.msi. Maybe uninstall: haven't tried.
If Martin doesn't get there, I'll try to look at the log and
see
Tim Golden added the comment:
Is there mileage for glob.glob to grow a dialect
param, with a default value to keep it backwards
compatible? Otherwise, presumably, proponents of
some other xsh variant will come forward with
their scheme of matching, and regex-followers
with theirs and so on
Tim Golden added the comment:
Can I nudge this one a bit? It causes an interpreter crash and the patch
seems good (subject to someone else's review).
--
___
Python tracker
<http://bugs.python.org/i
Tim Golden added the comment:
Erik Sandberg wrote:
> Erik Sandberg added the comment:
>
> Did you test your code?
Several times, cutting and pasting into the Python interpreter.
But I missed the fact that you were running Python 2.5
Python 2.6.1 (r261:67517, Dec 4 2008, 16:51
Tim Golden added the comment:
Erik Sandberg wrote:
> Erik Sandberg added the comment:
>
> I experimented further, the only way to run a .bat file whose name
> contains funny characters, seems to be:
>
> subprocess.call('""f(o.bat""', shell=Tr
Tim Golden added the comment:
Adding Tarek to nosy list as he added the code earlier this month
--
nosy: +tarek
title: distutils.test_util fails to restore os.uname -> distutils.test_util
fails to restore os.uname; causes test_platform to f
New submission from Tim Golden :
lib\distutils\tests\test_util.py, run as part of the full testsuite,
creates a stub os.uname on an OS which doesn't support it natively.
However, it fails to restore it correctly -- ie fails to delete the
attribute. As a result, test_platform and test_p
New submission from Tim Golden :
The msi.py determines which files to carry over into the installer for
the lib\test directory. zipdir.zip was added recently for test_zipfile
and this isn't picked up. The attached patch adds it in.
--
components: Installation
files: msi-zipdir.
New submission from Tim Golden :
tools\msi\merge.py attempts to import config and fails with an
ImportError if it doesn't exist (which it doesn't by default). msi.py
catches this exception and ignores it.
The attached patch carries the same behaviour over to merge.py
--
Tim Golden added the comment:
Effectively a duplicate of http://bugs.python.org/issue2421
(which has been sitting around unapplied for a few months)
I certainly don't mind which one goes in, but I think
one should be closed in favour of the other. (And that
one should be ap
Tim Golden added the comment:
Hirokazu Yamamoto wrote:
> Hirokazu Yamamoto added the comment:
>
> I reconsidered this issue. When mmap is anonymous,
> self->file_handle == INVALID_HANDLE_VALUE (-1), so we should not call
> SetFilePointer and SetEndOfFile for this handle
Tim Golden added the comment:
>From me, yes of course, but I assume you want another
core dev for a 2nd opinion.
___
Python tracker
<http://bugs.python.org/iss
Tim Golden added the comment:
Have a look at issue 2733
http://bugs.python.org/issue2733
where I've just proposed a patch in this area. I'm also
not sure exactly what's going on, but I have patched what
I believe is a linked pair of bugs in that code.
--
no
Tim Golden added the comment:
Can't reproduce under Python 2.6 or Python 2.5.2. Likelihood is a virus
checker / indexer process.
--
nosy: +tim.golden
___
Python tracker
<http://bugs.python.org/i
Tim Golden added the comment:
Patch attached to mmapmodule.c and test_mmap.py
--
keywords: +patch
Added file: http://bugs.python.org/file13107/mmapmodule.patch
___
Python tracker
<http://bugs.python.org/issue2
Tim Golden added the comment:
OK, I can see why this is happening and in fact there are two levels of
problem. The trouble is that, in my ignorance, I can't work out exactly
why the existing code is doing what it's doing.
(References to mmapmodule.c at r69666)
Problem 1: At lin
Changes by Tim Golden :
--
type: -> crash
___
Python tracker
<http://bugs.python.org/issue5261>
___
___
Python-bugs-list mailing list
Unsubscri
Tim Golden added the comment:
Problem seems to be in Modules/_multiprocessing/semaphore.c
line 549 where "__enter__" is defined as an alias for
semlock_acquire, as is "acquire" a few lines above. However,
while "acquire" specifies METH_VARARGS | METH_
Tim Golden added the comment:
Reproduced on trunk r69621
--
nosy: +tim.golden
versions: +Python 2.7
___
Python tracker
<http://bugs.python.org/issue5261>
___
___
Tim Golden added the comment:
You need to use raw strings or to use forward-slashes in your pathnames:
r"c:\downloads\hacking\0812logcompress"
or
"c:/downloads/hacking/0812logcompress"
The sequence \0 has a special meaning in strings, introducing an octal
escape, I thi
Tim Golden <[EMAIL PROTECTED]> added the comment:
What would be helpful would be a specific suggestion from the OP (or, better
still, a doc patch) as to exactly what should change or at least what kind of
words should go where. As it stands, this report highlights a real but quite
g
Tim Golden <[EMAIL PROTECTED]> added the comment:
The comment just before _ctypes.c:309 indicates that when a NULL is
returned at that point, an error condition should already obtain.
___
Python tracker <[EMAIL PROTECTED]>
<http://bugs.pytho
New submission from Tim Golden <[EMAIL PROTECTED]>:
The following code raises an Assertion Failure under debug in r64518
running on Windows XP SP2:
import ctypes
class X (ctypes.Structure): pass
ctypes.POINTER (X)
Assertion failed: PyErr_Occurred(), file ..\Modules\_ctypes\_ctypes.c
Tim Golden <[EMAIL PROTECTED]> added the comment:
The attached file sp-3.py simulates what I think is happening within the
subprocess module. Note that the OS handle is duplicated to allow
inheritance and then left unclosed on failure. If it is explicitly
closed, the file can be removed.
Tim Golden <[EMAIL PROTECTED]> added the comment:
Giampaolo Rodola' wrote:
> New submission from Giampaolo Rodola' <[EMAIL PROTECTED]>:
>
> By using:
> http://www.python.org/dev/daily-msi/python-2.6.14041.msi
>
>
> C:\>C:\python26\python
>
Tim Golden <[EMAIL PROTECTED]> added the comment:
Trent's supplied me with enough info to patch the project files
manually. The attached patch against r64120 results in the
_multiprocessing module building. I'm running the tests now but I'll
upload the patch in any ca
Tim Golden <[EMAIL PROTECTED]> added the comment:
I'm sorry; I've had a look and there's no chance of my updating the
solution and project files by hand; and I can't run Visual Studio at the
moment. I'll try emailing Trent or Christian in the hope that o
Tim Golden <[EMAIL PROTECTED]> added the comment:
Benjamin Peterson wrote:
> Benjamin Peterson <[EMAIL PROTECTED]> added the comment:
>
> On Wed, Jun 11, 2008 at 9:20 AM, Tim Golden <[EMAIL PROTECTED]> wrote:
>> Tim Golden <[EMAIL PROTECTED]> added the com
Tim Golden <[EMAIL PROTECTED]> added the comment:
The _multiprocessing module is not building under Windows at the moment.
Attempting to import multiprocessing (from an .exe build from the
current svn) gives "ImportError: No module named _multiprocessing" and
the test suite sk
New submission from Tim Golden <[EMAIL PROTECTED]>:
The docs for _winreg refer to RegLoadKey, which is the underlying API.
The function is actually exposed as LoadKey. Patch attached.
--
assignee: georg.brandl
components: Documentation
files: _winreg.patch
keywords: patch
me
New submission from Tim Golden <[EMAIL PROTECTED]>:
At present, os.access under Windows simply calls GetFileAttributes to
determine the readonly attribute (ignoring directories). The patch
attached combines this with the use of the AccessCheck API to compare
the user's permissions
New submission from Tim Golden <[EMAIL PROTECTED]>:
doc\make.bat, used to build the docs under Windows, retains the
hardcoded pydoc.hhp name when building htmlhelp. Now that the help files
are built as .chm this file no longer exists and the build fails.
The attached patch to make.bat
Tim Golden <[EMAIL PROTECTED]> added the comment:
Updated patch against r61514. Test code now PEP8-compliant (I hope). New
tests cover spaces in command and parameter with and without shell=True,
both as simple command string and as list of command/args.
Added file: http://bugs.pyth
Tim Golden <[EMAIL PROTECTED]> added the comment:
Gabriel Genellina wrote:
> Gabriel Genellina <[EMAIL PROTECTED]> added the comment:
>
> You aren't testing the modified code, the Popen call should say
> shell=True.
>
> I think that a more PEP8-complian
Changes by Tim Golden <[EMAIL PROTECTED]>:
--
title: subprocess under windows fails to quote properly under Windows when
shell=True -> subprocess under windows fails to quote properly when shell=True
__
Tracker <[EMAIL PROTE
New submission from Tim Golden <[EMAIL PROTECTED]>:
The subprocess.Popen function reorganises the command line for process
creation when shell=True is passed in under Windows. It runs the
existing executable & arguments as arguments to %COMSPEC% /c. However
this fails when a second
Tim Golden added the comment:
Alexander Belopolsky wrote:
> Alexander Belopolsky added the comment:
>
>> Feel free to propose an alternative wording for the patch
>
> I thought I already did in my first post. The complete sentence should
> read:
>
> "&q
Tim Golden added the comment:
Alexander Belopolsky wrote:
> Alexander Belopolsky added the comment:
>
>> .. but I have made the doc reference a link to the os.pathsep
>
> I knew you would say that :-). I was making my comment out of real life
> experience: sysadmi
Tim Golden added the comment:
Alexander Belopolsky wrote:
> Alexander Belopolsky added the comment:
>
> PYTHONPATH variable is likely to be defined by sysadmins who may not know
> what os.pathsep is. Maybe it is better to say "OS-dependent separator
> (';' on W
New submission from Tim Golden:
The docs for the PYTHONPATH var indicate that its items are separated by
colons. In fact they're separated by whatever's customary for the O/S.
Patch attached.
--
components: Documentation
files: doc-using-cmdline-r61249.patch
keywords: patc
Tim Golden added the comment:
Corrected patch which replaces the @echo off at the top of the file.
Added file: http://bugs.python.org/file9573/doc-make-r61125.patch
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.python.org/
New submission from Tim Golden:
The doc/make.bat file for building the docs under Windows assumes the
standard location for the HTML Help Workshop. The attached patch looks
for an env var called HTMLHELP and uses that if it's set, falling back
to the standard location.
--
compo
New submission from Tim Golden:
When pygments is not available to the sphinx build environment, the
PygmentsBridge in the highlighting.py module raises an exception in the
unhighlighted function. This function attempts to use the .dest
attribute which isn't set in the __init__ if the pyg
New submission from Tim Golden:
A patch against r61085 of /doc to correct some very minor typos in the docs
--
components: Documentation
files: python-doc-r61085.patch
keywords: patch
messages: 63048
nosy: tim.golden
severity: minor
status: open
title: Tiny patch to docs
versions
New submission from Tim Golden:
My previous patch to the print stylesheet used by Sphinx was a little
overenthusiastic and resulted in the right edge of the text truncating
on some printers. This version reverts a part of that and gives a useful
result on the printers I've tried.
Patch att
New submission from Tim Golden:
The print-media stylesheet in the sphinx docs did not completely
eliminate the on-screen layout. The attached patch is against r59327 of
sphinx/style/default.css and has been tested against html, htmlhelp and
web under native Win32.
--
components
Tim Golden added the comment:
Joseph Armbruster wrote:
> The bad news [should a different bug be created for this?]
>
> See cmdline.rst:
> .. cmdoption:: -c
> .. cmdoption:: -m
> .. describe::
Changes by Tim Golden:
--
severity: normal -> minor
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.python.org/issue1548>
__
___
Python-bugs-list mailing list
Uns
Changes by Tim Golden:
--
severity: normal -> minor
__
Tracker <[EMAIL PROTECTED]>
<http://bugs.python.org/issue1547>
__
___
Python-bugs-list mailing list
Uns
New submission from Tim Golden:
The tiniest of punctuation typos in using/cmdline.rst. Patch is against
59286 of doc/using/cmdline.rst
--
components: Documentation
files: doc-using-cmdline-r59286.patch
messages: 58119
nosy: tim.golden
severity: normal
status: open
title: Tiny typo in
New submission from Tim Golden:
One typo refers to "from __future__ import generators" in the context of
the with statement.
Later what appears to be an incomplete sentence giving an example of the
__dir__ method.
The patch attached is against r59286 of doc/whatsn
Tim Golden added the comment:
The attached patch against r59286 tries to tease apart the uses of
filename by adding "webify_filepath" and "unwebify_filepath" functions
to sphinx.utils which are then used throughout the app to convert from
filesystem-separated to web-separat
501 - 600 of 607 matches
Mail list logo