[issue1136] Bdb documentation

2007-09-12 Thread Tim Golden

Tim Golden added the comment:

I've reviewed the docs for English and general readability. As
mentioned, I've no idea of the tech involved. I did look through the
bdb.py source and the existing docs for pdb to get some idea of the
terminology used. Ultimately I've changed very little; in a couple of
places the English was just wrong and in a couple of others I felt a
small change improved things.

One thing I couldn't work out is that arklad, the author, used canonic
throughout which I've never seen used. I would simply have changed it to
canonical except that the code itself uses canonic as a function
name! In addition, there's no explanation of what canonic(al) means in
the context. I've left ?TJG? marks around all the instances.

--
nosy: +tim.golden

__
Tracker [EMAIL PROTECTED]
http://bugs.python.org/issue1136
__

bdb.rst
Description: Binary data
___
Python-bugs-list mailing list 
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue10618] regression in subprocess.call() command quoting

2010-12-04 Thread Tim Golden

Tim Golden m...@timgolden.me.uk added the comment:

I'm not quite sure how anyone's supposed to determine
which bugs are likely to have been worked around and
which haven't :) I'm also unsure why a clear bugfix
shouldn't make it into a minor version release. Surely
this isn't the only one to do so...

I'm happy to repatch/test to strip quotes before adding,
but I see that Benjamin prefers to leave it alone.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue10618
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue1602] windows console doesn't print utf8 (Py30a2)

2011-01-10 Thread Tim Golden

Tim Golden m...@timgolden.me.uk added the comment:

Reopening as there seems to be some possibility of progress

--
nosy:  -BreamoreBoy
resolution: invalid - 
status: closed - open

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1602
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue1602] windows console doesn't print utf8 (Py30a2)

2011-01-10 Thread Tim Golden

Changes by Tim Golden m...@timgolden.me.uk:


--
versions: +Python 3.3 -Python 3.1, Python 3.2

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1602
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue1255] Strange Python hangup

2007-10-10 Thread Tim Golden

Tim Golden added the comment:

Do you realise that the code at the bottom of bb.py is executed when you
import it from aa.py? In other words, when you run aa.py, the whole of
your significant code is running within an import statement. I don't
know if it's the cause of the problem (although I remember past
strictures on the messiness of threads and imports) but I doubt if it's
a fantastic design choice.

--
nosy: +tim.golden

__
Tracker [EMAIL PROTECTED]
http://bugs.python.org/issue1255
__
___
Python-bugs-list mailing list 
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue1279] os.system() oddity under Windows XP SP2

2007-10-15 Thread Tim Golden

Tim Golden added the comment:

Not, apparently, on my (XP SP2) box:

dump
Python 2.5.1 (r251:54863, Apr 18 2007, 08:51:08) [MSC v.1310 32 bit
(Intel)] on win32
Type help, copyright, credits or license for more information.
 import os
 os.system (python -c f=open
('temp.txt','w');f.write('hello');f.close ()); print open
(temp.txt).read ()
0
hello


/dump

Perhaps you could provide a runnable piece of code which demonstrates
the problem?

--
nosy: +tim.golden

__
Tracker [EMAIL PROTECTED]
http://bugs.python.org/issue1279
__
___
Python-bugs-list mailing list 
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue1520] 'without make' documentation build anomaly

2007-12-02 Thread Tim Golden

Tim Golden added the comment:

Looks like it's the backslash as Windows path separator confusing the
sphinx.util.relative_uri function. I'll try to put a patch together,
although I'm not sure if relative_uri should use os.sep or whether the
paths should be canonicalised before getting there. I'll try with os.sep
to start with.

--
nosy: +tim.golden

__
Tracker [EMAIL PROTECTED]
http://bugs.python.org/issue1520
__
___
Python-bugs-list mailing list 
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue1520] 'without make' documentation build anomaly

2007-12-02 Thread Tim Golden

Tim Golden added the comment:

Patch against sphinx r59269. Split on / and local os.sep. Causes
problems with sphinx-web under native Win32.

Added file: http://bugs.python.org/file8853/sphinx-r59269.patch

__
Tracker [EMAIL PROTECTED]
http://bugs.python.org/issue1520
__

sphinx-r59269.patch
Description: Binary data
___
Python-bugs-list mailing list 
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue1520] 'without make' documentation build anomaly

2007-12-02 Thread Tim Golden

Tim Golden added the comment:

I'm a little flummoxed. Patching relative_uri in the
sphinx/util/__init__.py to re.split on (/ + os.sep) sorts out the
static generation. But it seems to introduce a couple of problems with
the web-server version of the docs. One is that links end up without a
category: http://localhost:3000/windows/ instead of
http://localhost:3000/using/windows/. The other is that, even if you put
the URLs in by hand, they don't work whereas /using%5cwindows *does*
work. Which suggests that the windows-y path is being stored somewhere
in the pickle files.

So now we're stretched two ways: the HTML needs /-delimited names to
work; the pickled files need \-delimited. I'm still looking at it, but
if anyone has more knowledge that I have (couldn't exactly have less)
then please feel free to chip in.

__
Tracker [EMAIL PROTECTED]
http://bugs.python.org/issue1520
__
___
Python-bugs-list mailing list 
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue1520] 'without make' documentation build anomaly

2007-12-02 Thread Tim Golden

Tim Golden added the comment:

OK, hacking away a bit further, I think I've found a solution, but I'll
need to tidy it up a bit. In essence, the problem is that the filename
is trying to be two things: the pointer for the local filesystem, and
the uri for the web server. On *nix, this will pretty much work. On
Windows -- and elsewhere, it won't.

My approach is to treat the filename a little like unicode: decode to
web-format on the way in; encode to local-format on the way out. In
practice, this resolves to a bunch of .replace(path.sep, /) calls in
one place and .replace(/, path.sep) in the comparatively rare cases we
actually need to read the local filesystem.

I realise that, on Windows, I could get a bit hacky and just assume that
/ will work throughout. But that obviously fails now for things like
VMS and in the future for the hypothetical filesystem which uses . as
its separator etc.

I'm still working it through. The web and html versions are certainly
working but I need to check out htmlhelp which has its own special needs.

__
Tracker [EMAIL PROTECTED]
http://bugs.python.org/issue1520
__
___
Python-bugs-list mailing list 
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue1520] 'without make' documentation build anomaly

2007-12-03 Thread Tim Golden

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-separated paths and back. A WEB_SEP constant
has been defined principally to make its purpose clear in case a
constant / was thought to have been a hangover from the code's *nix
origins.

The result has been tested by building manually, ie by invoking python
tools\sphinx-build.py with appropriate params under Win32 native only.
The html, web and htmlhelp versions all style up and function correctly.
It would obviously benefit from testing under Cygwin and *nix. I'm
particularly keen to see it it works on a filesystem which doesn't allow
/ as a separator.

Added file: http://bugs.python.org/file8860/sphinx-r59286.patch

__
Tracker [EMAIL PROTECTED]
http://bugs.python.org/issue1520
__Index: builder.py
===
--- builder.py	(revision 59286)
+++ builder.py	(working copy)
@@ -27,7 +27,8 @@
 from docutils.frontend import OptionParser
 
 from .util import (get_matching_files, attrdict, status_iterator,
-   ensuredir, get_category, relative_uri)
+   ensuredir, get_category, relative_uri, webify_filepath,
+   unwebify_filepath)
 from .writer import HTMLWriter
 from .util.console import bold, purple, green
 from .htmlhelp import build_hhx
@@ -483,12 +484,12 @@
 self.srcdir, '*.rst', exclude=set(self.config.get('unused_files', (:
 try:
 targetmtime = path.getmtime(path.join(self.outdir,
-  filename[:-4] + '.html'))
+  unwebify_filepath(filename)[:-4] + '.html'))
 except:
 targetmtime = 0
 if filename not in self.env.all_files:
 yield filename
-elif path.getmtime(path.join(self.srcdir, filename))  targetmtime:
+elif path.getmtime(path.join(self.srcdir, unwebify_filepath(filename)))  targetmtime:
 yield filename
 
 
@@ -513,7 +514,7 @@
 ctx = self.globalcontext.copy()
 ctx.update(context)
 output = self.templates[templatename].render(ctx)
-outfilename = path.join(self.outdir, filename[:-4] + '.html')
+outfilename = path.join(self.outdir, unwebify_filepath(filename)[:-4] + '.html')
 ensuredir(path.dirname(outfilename)) # normally different from self.outdir
 try:
 with codecs.open(outfilename, 'w', 'utf-8') as fp:
@@ -522,7 +523,7 @@
 print self.warning_stream, Error writing file %s: %s % (outfilename, err)
 if self.copysource and context.get('sourcename'):
 # copy the source file for the show source link
-shutil.copyfile(path.join(self.srcdir, filename),
+shutil.copyfile(path.join(self.srcdir, unwebify_filepath(filename)),
 path.join(self.outdir, context['sourcename']))
 
 def handle_finish(self):
@@ -547,10 +548,10 @@
 self.srcdir, '*.rst', exclude=set(self.config.get('unused_files', (:
 try:
 targetmtime = path.getmtime(path.join(self.outdir,
-  filename[:-4] + '.fpickle'))
+  unwebify_filepath(filename)[:-4] + '.fpickle'))
 except:
 targetmtime = 0
-if path.getmtime(path.join(self.srcdir, filename))  targetmtime:
+if path.getmtime(path.join(self.srcdir, unwebify_filepath(filename)))  targetmtime:
 yield filename
 
 def get_target_uri(self, source_filename):
@@ -577,7 +578,7 @@
 self.indexer.feed(filename, category, title, doctree)
 
 def handle_file(self, filename, context, templatename='page'):
-outfilename = path.join(self.outdir, filename[:-4] + '.fpickle')
+outfilename = path.join(self.outdir, unwebify_filepath(filename)[:-4] + '.fpickle')
 ensuredir(path.dirname(outfilename))
 context.pop('pathto', None) # can't be pickled
 with file(outfilename, 'wb') as fp:
@@ -587,7 +588,7 @@
 if context.get('sourcename'):
 source_name = path.join(self.outdir, 'sources', context['sourcename'])
 ensuredir(path.dirname(source_name))
-shutil.copyfile(path.join(self.srcdir, filename), source_name)
+shutil.copyfile(path.join(self.srcdir, unwebify_filepath(filename)), source_name)
 
 def handle_finish(self):
 # dump the global context
Index: directives.py
===
--- directives.py	(revision 59286

[issue1547] Minor typos in whatsnew26

2007-12-03 Thread Tim Golden

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/whatsnew/2.6.rst

--
components: Documentation
files: whatsnew-r59286.patch
messages: 58118
nosy: tim.golden
severity: normal
status: open
title: Minor typos in whatsnew26
versions: Python 2.6, Python 3.0
Added file: http://bugs.python.org/file8861/whatsnew-r59286.patch

__
Tracker [EMAIL PROTECTED]
http://bugs.python.org/issue1547
__Index: 2.6.rst
===
--- 2.6.rst	(revision 59286)
+++ 2.6.rst	(working copy)
@@ -108,7 +108,7 @@
 
 The previous version, Python 2.5, added the ':keyword:`with`'
 statement an optional feature, to be enabled by a ``from __future__
-import generators`` directive.  In 2.6 the statement no longer need to
+import with_statement`` directive.  In 2.6 the statement no longer need to
 be specially enabled; this means that :keyword:`with` is now always a
 keyword.  The rest of this section is a copy of the corresponding 
 section from What's New in Python 2.5 document; if you read
@@ -481,7 +481,7 @@
   of strings containing the names of valid attributes for the object,
   and lets the object control the value that :func:`dir` produces.
   Objects that have :meth:`__getattr__` or :meth:`__getattribute__` 
-  methods.
+  methods can use this to advertise pseudo-attributes they will honour.
 
   .. % Patch 1591665
 
___
Python-bugs-list mailing list 
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue1548] Tiny typo in doc\using\cmdline.rst

2007-12-03 Thread Tim Golden

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 doc\using\cmdline.rst
versions: Python 2.6
Added file: http://bugs.python.org/file8862/doc-using-cmdline-r59286.patch

__
Tracker [EMAIL PROTECTED]
http://bugs.python.org/issue1548
__Index: cmdline.rst
===
--- cmdline.rst	(revision 59286)
+++ cmdline.rst	(working copy)
@@ -10,7 +10,7 @@
 
 .. note:: 

-   Other implementation's command line schemes may differ.  See
+   Other implementations' command line schemes may differ.  See
:ref:`implementations` for further resources.
 
 
___
Python-bugs-list mailing list 
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue1547] Minor typos in whatsnew26

2007-12-03 Thread Tim Golden

Changes by Tim Golden:


--
severity: normal - minor

__
Tracker [EMAIL PROTECTED]
http://bugs.python.org/issue1547
__
___
Python-bugs-list mailing list 
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue1548] Tiny typo in doc\using\cmdline.rst

2007-12-03 Thread Tim Golden

Changes by Tim Golden:


--
severity: normal - minor

__
Tracker [EMAIL PROTECTED]
http://bugs.python.org/issue1548
__
___
Python-bugs-list mailing list 
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue1520] 'without make' documentation build anomaly

2007-12-03 Thread Tim Golden

Tim Golden added the comment:

Joseph Armbruster wrote:
 The bad news [should a different bug be created for this?]
 
 See cmdline.rst:
 .. cmdoption:: -c command
 .. cmdoption:: -m module-name
 .. describe:: script
 
 Looks like genindex.html is being generated with some iffy markup:
 dt-c command/dt

I'm not sure which way round you're say this. On my system,
the markup is:

dl class=cmdoption
dt id=cmdoption-c
tt class=descname-c/ class=descclassname 
lt;commandgt;/tta class=headerlink href=#cmdoption-c 
title=Permalink to this definition¶/a/dt
ddpExecute .../p/dd
/dl

Is this right? Or wrong? It doesn't look disastrous to me.
In either case, I doubt it's related to the path issues we've
been addressing in this Issue. Probably best to raise a new
Issue if you think there's a problem.

TJG

__
Tracker [EMAIL PROTECTED]
http://bugs.python.org/issue1520
__
___
Python-bugs-list mailing list 
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue1555] Print-media stylesheet for sphinx docs incomplete

2007-12-04 Thread Tim Golden

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: Documentation tools (Sphinx)
files: defaults-r59327.patch
messages: 58193
nosy: tim.golden
severity: minor
status: open
title: Print-media stylesheet for sphinx docs incomplete
versions: Python 3.0
Added file: http://bugs.python.org/file8874/defaults-r59327.patch

__
Tracker [EMAIL PROTECTED]
http://bugs.python.org/issue1555
__Index: style/default.css
===
--- style/default.css	(revision 59327)
+++ style/default.css	(working copy)
@@ -782,9 +782,13 @@
 div.documentwrapper {
 width: 100%;
 }
-
-div.body {
-margin: 0;
+
+	div.document, 
+	div.documentwrapper, 
+	div.bodywrapper, 
+	div.body {
+margin: 0;
+		width : 100%;
 }
 
 div.sidebar,
___
Python-bugs-list mailing list 
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue1964] Slight adjustment to sphinx print-media stylesheet

2008-01-29 Thread Tim Golden

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 attached against r60407 of Sphinx

--
components: Documentation tools (Sphinx)
files: sphinx-r60407.patch
messages: 61821
nosy: tim.golden
severity: minor
status: open
title: Slight adjustment to sphinx print-media stylesheet
versions: Python 2.6, Python 3.0
Added file: http://bugs.python.org/file9317/sphinx-r60407.patch

__
Tracker [EMAIL PROTECTED]
http://bugs.python.org/issue1964
__
___
Python-bugs-list mailing list 
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue2207] Bug in Sphinx highlighting when pygments not available

2008-02-29 Thread Tim Golden

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 pygments import has failed.

The attached patch moves the .dest setting up so it is always set
regardless of the presence of pygments. I don't know if it's a complete
solution, but the help, htmlhelp and web all build.

--
components: Documentation tools (Sphinx)
files: sphinx-highlighting-r61125.patch
keywords: patch
messages: 63130
nosy: tim.golden
severity: normal
status: open
title: Bug in Sphinx highlighting when pygments not available
versions: Python 2.6, Python 3.0
Added file: http://bugs.python.org/file9571/sphinx-highlighting-r61125.patch

__
Tracker [EMAIL PROTECTED]
http://bugs.python.org/issue2207
__
___
Python-bugs-list mailing list 
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue2208] Patch to doc/make.bat to allow non-standard HTML Help location

2008-02-29 Thread Tim Golden

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.

--
components: Documentation tools (Sphinx)
files: doc-make-r61125.patch
keywords: patch
messages: 63131
nosy: tim.golden
severity: normal
status: open
title: Patch to doc/make.bat to allow non-standard HTML Help location
versions: Python 2.6
Added file: http://bugs.python.org/file9572/doc-make-r61125.patch

__
Tracker [EMAIL PROTECTED]
http://bugs.python.org/issue2208
__
___
Python-bugs-list mailing list 
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue2208] Patch to doc/make.bat to allow non-standard HTML Help location

2008-02-29 Thread Tim Golden

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/issue2208
__
___
Python-bugs-list mailing list 
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue2239] Tiny patch to cmdline docs

2008-03-05 Thread Tim Golden

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: patch
messages: 63276
nosy: georg.brandl, tim.golden
severity: normal
status: open
title: Tiny patch to cmdline docs
versions: Python 2.6
Added file: http://bugs.python.org/file9609/doc-using-cmdline-r61249.patch

__
Tracker [EMAIL PROTECTED]
http://bugs.python.org/issue2239
__
___
Python-bugs-list mailing list 
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue2239] Tiny patch to cmdline docs

2008-03-05 Thread Tim Golden

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 Windows, ':' on Linux and other UNIX-like OSes or ',' on some 
 less-known systems.)

Maybe, but I have made the doc reference a link to the os.pathsep
doc which basically says what you just did.

__
Tracker [EMAIL PROTECTED]
http://bugs.python.org/issue2239
__
___
Python-bugs-list mailing list 
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue2239] Tiny patch to cmdline docs

2008-03-05 Thread Tim Golden

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: sysadmins rarely know python language, but can do good job 
 administering python installations.  For better or worse, the target 
 audience for PYTHONPATH documentation is likely not to be interested in 
 being educated in the python programming language.  Having to lookup the 
 meaning of os.pathsep just to understand the format of PYTHONPATH is a 
 distraction. Another problem with the link is that cmdline page is 
 likely to be printed out for a quick reference and the link will 
 obviously not work in the printout.

I don't feel strongly about it. Feel free to propose an alternative
wording for the patch :)

__
Tracker [EMAIL PROTECTED]
http://bugs.python.org/issue2239
__
___
Python-bugs-list mailing list 
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue2239] Tiny patch to cmdline docs

2008-03-05 Thread Tim Golden

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:
 
 
 The format is the same as the shell's :envvar:`PATH`: one or more 
 directory pathnames separated by an OS-dependent character (';' on 
 Windows, ':' on Linux and other UNIX-like OSes or ',' on some 
 less-known systems.)
 

Ah. Lazily, I meant: feel free to upload a patch containing the
wording you think good.

TJG

__
Tracker [EMAIL PROTECTED]
http://bugs.python.org/issue2239
__
___
Python-bugs-list mailing list 
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue2304] subprocess under windows fails to quote properly under Windows when shell=True

2008-03-16 Thread Tim Golden

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 parameter (typically a filename) has an
embedded space. The patch attached adds extra parameters in this case
and adds appropriate unit tests.

Frankly I'm new to unittests and I more-or-less cloned an existing one
to make this work. Happy to be corrected if there's things done wrong c.

--
components: Library (Lib)
files: subprocess-r61420.patch
keywords: patch
messages: 63583
nosy: tim.golden
severity: normal
status: open
title: subprocess under windows fails to quote properly under Windows when 
shell=True
type: behavior
versions: Python 2.6
Added file: http://bugs.python.org/file9682/subprocess-r61420.patch

__
Tracker [EMAIL PROTECTED]
http://bugs.python.org/issue2304
__
___
Python-bugs-list mailing list 
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue2304] subprocess under windows fails to quote properly when shell=True

2008-03-16 Thread Tim Golden

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 PROTECTED]
http://bugs.python.org/issue2304
__
___
Python-bugs-list mailing list 
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue2304] subprocess under windows fails to quote properly when shell=True

2008-03-17 Thread Tim Golden

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-compliant style would be nice (removing the 
 spaces after open and read, and using consistent indentation)

D'oh. Thanks, Gabriel. I'll rework the test and tidy
up the patch.

--
title: subprocess under windows fails to quote properly when shell=True - 
subprocess under windows fails to quote properly whenshell=True

__
Tracker [EMAIL PROTECTED]
http://bugs.python.org/issue2304
__
___
Python-bugs-list mailing list 
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue2304] subprocess under windows fails to quote properly when shell=True

2008-03-18 Thread Tim Golden

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.python.org/file9725/subprocess-r61514.patch

__
Tracker [EMAIL PROTECTED]
http://bugs.python.org/issue2304
__
___
Python-bugs-list mailing list 
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue2421] doc\make.bat fails for htmlhelp because of hardcoded filename

2008-03-19 Thread Tim Golden

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 release.chm this file no longer exists and the build fails.

The attached patch to make.bat builds the htmlhelp from within a
temporary Python session which picks the correct filename up from conf.py.

--
assignee: georg.brandl
components: Documentation tools (Sphinx)
files: doc-make-r61620.patch
keywords: patch
messages: 64051
nosy: georg.brandl, tim.golden
severity: normal
status: open
title: doc\make.bat fails for htmlhelp because of hardcoded filename
type: behavior
versions: Python 2.6
Added file: http://bugs.python.org/file9764/doc-make-r61620.patch

__
Tracker [EMAIL PROTECTED]
http://bugs.python.org/issue2421
__
___
Python-bugs-list mailing list 
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue8869] execfile does not work with UNC paths

2010-06-01 Thread Tim Golden

Tim Golden m...@timgolden.me.uk 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 nearing its end-of-life,
I doubt this issue will garner much sympathy.

I haven't actually looked at the code to discover just *why* execfile
doesn't support that style of filename. But is there any reason you
can't use exec (open (...))?

--
nosy: +tim.golden

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue8869
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue850997] mbcs encoding ignores errors

2010-06-17 Thread Tim Golden

Tim Golden m...@timgolden.me.uk 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 Victorvictor.stin...@haypocalc.com  added the comment:

 Tim: are you interested in testing this patch?

 --
 nosy: +tim.golden

 ___
 Python trackerrep...@bugs.python.org
 http://bugs.python.org/issue850997
 ___

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue850997
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue9024] PyDateTime_IMPORT macro incorrectly marked up

2010-06-18 Thread Tim Golden

New submission from Tim Golden m...@timgolden.me.uk:

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: pydatetime_import.patch
keywords: patch
messages: 108086
nosy: tim.golden
priority: normal
severity: normal
status: open
title: PyDateTime_IMPORT macro incorrectly marked up
Added file: http://bugs.python.org/file17703/pydatetime_import.patch

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9024
___Index: c-api/datetime.rst
===
--- c-api/datetime.rst  (revision 82013)
+++ c-api/datetime.rst  (working copy)
@@ -8,9 +8,9 @@
 Various date and time objects are supplied by the :mod:`datetime` module.
 Before using any of these functions, the header file :file:`datetime.h` must be
 included in your source (note that this is not included by :file:`Python.h`),
-and the macro :cfunc:`PyDateTime_IMPORT` must be invoked.  The macro puts a
-pointer to a C structure into a static variable,  ``PyDateTimeAPI``, that is
-used by the following macros.
+and the macro :cmacro:`PyDateTime_IMPORT` must be invoked, usually as part of
+the module initialisation function.  The macro puts a pointer to a C structure
+into a static variable,  ``PyDateTimeAPI``, that is used by the following 
macros.
 
 Type-check macros:
 
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue9024] PyDateTime_IMPORT macro incorrectly marked up

2010-06-18 Thread Tim Golden

Changes by Tim Golden m...@timgolden.me.uk:


--
assignee:  - d...@python
components: +Documentation
nosy: +d...@python
stage:  - patch review
versions: +Python 3.2

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9024
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue9035] os.path.ismount on windows doesn't support windows mount points

2010-06-20 Thread Tim Golden

Tim Golden m...@timgolden.me.uk 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, posixmodule.c?

--
assignee:  - tim.golden
nosy: +tim.golden
stage:  - needs patch
type:  - behavior
versions: +Python 3.2 -Python 2.6, Python 2.7

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9035
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue9035] os.path.ismount on windows doesn't support windows mount points

2010-06-20 Thread Tim Golden

Tim Golden m...@timgolden.me.uk 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 exposes the full file attributes on Windows, and its
posix equivalent if there is one? This could then be used
in the ismount function currently implemented in ntpath.py.

--
title: os.path.ismount on windows doesn't support windows mount points - 
os.path.ismount on windows doesn't support windows mount  points

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9035
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue9035] os.path.ismount on windows doesn't support windows mount points

2010-06-20 Thread Tim Golden

Tim Golden m...@timgolden.me.uk 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 rep...@bugs.python.org
http://bugs.python.org/issue9035
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue8869] execfile does not work with UNC paths

2010-06-22 Thread Tim Golden

Changes by Tim Golden m...@timgolden.me.uk:


--
resolution:  - out of date
status: open - closed

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue8869
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue8440] test_heapq interfering with test_import on py3k

2010-06-22 Thread Tim Golden

Tim Golden m...@timgolden.me.uk added the comment:

Error no longer occurs

--
resolution:  - fixed
status: open - closed

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue8440
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue9035] os.path.ismount on windows doesn't support windows mount points

2010-06-22 Thread Tim Golden

Tim Golden m...@timgolden.me.uk 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 under Windows. The current solution is correct for a subset
of cases. Arguably a bug, although I doubt I'd get that past the
release manager!

The wider issue of exposing GetFileAttributesW, eg under one of the
unused stat fields, should be explored elsewhere.

On 22/06/2010 11:46, Senthil Kumaran wrote:

 Senthil Kumaranorsent...@gmail.com  added the comment:

 I see that ismount like function on windows is provide by the various
 Win32 extensions.

 If Windows supported is added to ismount function itself, then it might be
 a good idea to have attributes or list_attributes function as well.

 But for posix, how will it be different from details provided by stat?
 Would not it add redundancy?

 Or would it be better to provide file attributes as part of stat
 itself (if some are missing in Windows).

 --
 nosy: +orsenthil

 ___
 Python trackerrep...@bugs.python.org
 http://bugs.python.org/issue9035
 ___
 ___
 Python-bugs-list mailing list
 Unsubscribe: 
 http://mail.python.org/mailman/options/python-bugs-list/mail%40timgolden.me.uk

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9035
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue8056] Piped parent's multiprocessing.Process children cannot write to stdout

2010-06-30 Thread Tim Golden

Tim Golden m...@timgolden.me.uk 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 Friedvojtech.fr...@gmail.com  added the comment:

 I have the same problem. Not only for piping but also when using redirection, 
 e.g. 'stdout.py  log.txt'.
 Interesting thing is that when I run it like 'python.exe stdout.py  log.txt' 
 it works. I don't see why this should matter but apparently it does. .py 
 files are registered to be open with 'C:\Python26\python.exe %1 %*'.
 My system: WinXp SP3, Python 2.6.5

 --
 nosy: +Vojtech.Fried

 ___
 Python trackerrep...@bugs.python.org
 http://bugs.python.org/issue8056
 ___
 ___
 Python-bugs-list mailing list
 Unsubscribe: 
 http://mail.python.org/mailman/options/python-bugs-list/mail%40timgolden.me.uk

--
nosy: +tim.golden

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue8056
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue2528] Change os.access to check ACLs under Windows

2010-07-10 Thread Tim Golden

Tim Golden m...@timgolden.me.uk 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 something which at least uses current security mechanisms
to determine its answer.

However, it's just one function among the many Posixesque
functions which don't quite map to Windows. And one which
you might well be advised to avoid in any case since there's
an race condition inherent in checking for a file's accessibility
and then making use of that fact.  Better to try and fall back.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2528
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue9362] Make exit/quit hint more novice friendly

2010-07-28 Thread Tim Golden

Tim Golden m...@timgolden.me.uk 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 package on Windows

--
nosy: +tim.golden

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9362
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue6040] bdist_msi does not deal with pre-release version

2010-08-04 Thread Tim Golden

Changes by Tim Golden m...@timgolden.me.uk:


--
nosy: +tim.golden

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6040
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue4453] MSI installer shows error message if Compile .py files to bytecode option is selected

2010-08-04 Thread Tim Golden

Changes by Tim Golden m...@timgolden.me.uk:


--
nosy: +tim.golden

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4453
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue9503] print statement hangs Windows service

2010-08-04 Thread Tim Golden

Changes by Tim Golden m...@timgolden.me.uk:


--
nosy: +tim.golden

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9503
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue3210] subprocess.Popen does not release process handles if process cannot be started

2010-08-05 Thread Tim Golden

Tim Golden m...@timgolden.me.uk 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 rep...@bugs.python.org
http://bugs.python.org/issue3210
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue3210] subprocess.Popen does not release process handles if process cannot be started

2010-08-05 Thread Tim Golden

Tim Golden m...@timgolden.me.uk added the comment:

Patch added for 31 branch

--
Added file: http://bugs.python.org/file18399/3210.release31-maint.patch

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue3210
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue3210] subprocess.Popen does not release process handles if process cannot be started

2010-08-05 Thread Tim Golden

Tim Golden m...@timgolden.me.uk added the comment:

Patch added for 27 branch

--
Added file: http://bugs.python.org/file18400/3210.release27-maint.patch

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue3210
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue3210] subprocess.Popen does not release process handles if process cannot be started

2010-08-05 Thread Tim Golden

Tim Golden m...@timgolden.me.uk added the comment:

Brian, I'm not sure that the test as rewritten will exercise the error.

The key is that the traceback object will prevent the handles
from being finalised until it is itself finalised. After your
change I expect the handles to release anyway since the traceback
is already finalised.

In other words, I'm not testing that an exception is raised (which
is what the with assertRaises would suggest); I'm testing that
*within the lifetime of the exception* it's still possible to
remove the files which were passed as stdin/out/err where it wasn't
previously.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue3210
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue3210] subprocess.Popen does not release process handles if process cannot be started

2010-08-06 Thread Tim Golden

Tim Golden m...@timgolden.me.uk added the comment:

Committed in r83759 r83760 r83761

--
resolution:  - fixed
status: open - closed

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue3210
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue7443] test.support.unlink issue on Windows platform

2010-08-06 Thread Tim Golden

Changes by Tim Golden m...@timgolden.me.uk:


--
assignee:  - tim.golden

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue7443
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue4708] os.pipe should return inheritable descriptors (Windows)

2010-08-06 Thread Tim Golden

Changes by Tim Golden m...@timgolden.me.uk:


--
assignee:  - tim.golden
nosy: +tim.golden

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4708
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue1714451] subprocess.py problems errors when calling cmd.exe

2010-08-06 Thread Tim Golden

Changes by Tim Golden m...@timgolden.me.uk:


--
assignee:  - tim.golden
nosy: +tim.golden

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1714451
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue1681974] mkdtemp fails on Windows if username has non-ASCII character

2010-08-06 Thread Tim Golden

Changes by Tim Golden m...@timgolden.me.uk:


--
nosy: +tim.golden

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1681974
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue1672853] Error reading files larger than 4GB

2010-08-06 Thread Tim Golden

Changes by Tim Golden m...@timgolden.me.uk:


--
assignee:  - tim.golden
nosy: +tim.golden

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1672853
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue1707753] get status output fix for Win32

2010-08-06 Thread Tim Golden

Changes by Tim Golden m...@timgolden.me.uk:


--
assignee:  - tim.golden
nosy: +tim.golden

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1707753
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue1142] code sample showing errors reading large files with py 2.5/3.0

2010-08-06 Thread Tim Golden

Changes by Tim Golden m...@timgolden.me.uk:


--
nosy: +tim.golden

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1142
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue6609] zipfile: WindowsError [267] The directory name is invalid

2010-08-06 Thread Tim Golden

Changes by Tim Golden m...@timgolden.me.uk:


--
nosy: +tim.golden

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6609
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue6839] zipfile can't extract file

2010-08-06 Thread Tim Golden

Changes by Tim Golden m...@timgolden.me.uk:


--
assignee:  - tim.golden
nosy: +tim.golden

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6839
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue8006] os.popen in Python 3.1

2010-08-06 Thread Tim Golden

Changes by Tim Golden m...@timgolden.me.uk:


--
assignee:  - tim.golden
nosy: +tim.golden

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue8006
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue2304] subprocess under windows fails to quote properly when shell=True

2010-08-06 Thread Tim Golden

Changes by Tim Golden m...@timgolden.me.uk:


--
assignee:  - tim.golden
components: +Windows

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2304
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue3210] subprocess.Popen does not release process handles if process cannot be started

2010-08-06 Thread Tim Golden

Tim Golden m...@timgolden.me.uk added the comment:

Blast. Thanks; I'll have to rework those patches then.

--
status: closed - open

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue3210
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue1566260] Better order in file type descriptions

2010-08-06 Thread Tim Golden

Tim Golden m...@timgolden.me.uk added the comment:

If individual users want to change these around, it's easy enough to do so 
post-hoc. Closing as won't fix.

--
nosy: +tim.golden
resolution:  - wont fix

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1566260
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue1566260] Better order in file type descriptions

2010-08-06 Thread Tim Golden

Changes by Tim Golden m...@timgolden.me.uk:


--
stage:  - committed/rejected
status: open - closed

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1566260
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue3210] subprocess.Popen does not release process handles if process cannot be started

2010-08-08 Thread Tim Golden

Tim Golden m...@timgolden.me.uk added the comment:

OK, the issue identified by Hirokazu Yamamoto in msg113106 only actually 
affects the 2.x series, because of the awkwardly multiple-level interaction 
between file handles. The io rewrite in 3.x seems not to suffer the same way. 
Nonetheless, the proposed adhok.patch (slightly updated to match the current 
codebase) seems to cover all the cases, and is rather more elegant. So I'll 
revert the C module changes and apply that instead to all branches.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue3210
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue3210] subprocess.Popen does not release process handles if process cannot be started

2010-08-08 Thread Tim Golden

Tim Golden m...@timgolden.me.uk added the comment:

Committed in r83815, r83816, r83817

--
stage: patch review - committed/rejected
status: open - closed

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue3210
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue2304] subprocess under windows fails to quote properly when shell=True

2010-08-08 Thread Tim Golden

Tim Golden m...@timgolden.me.uk added the comment:

Committed as r83830, r83831, r83832

--
resolution:  - fixed
stage: patch review - committed/rejected
status: open - closed

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2304
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue1714451] subprocess.py problems errors when calling cmd.exe

2010-08-08 Thread Tim Golden

Tim Golden m...@timgolden.me.uk added the comment:

Duplicate of #2304

--
resolution:  - duplicate
status: open - closed

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1714451
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue8631] subprocess.Popen.communicate(...) hangs on Windows

2010-08-08 Thread Tim Golden

Tim Golden m...@timgolden.me.uk added the comment:

I can't reproduce on W7. Strangely, though, although my banner suggests that I 
have exactly the same build as you, I get a \r\n at the end of the 
communicate bytestream, not a simple \n as you're getting. Do you have any 
environment variables or Python command-line switches which might be affecting 
anything?

Could you try with python31/26 -E -S just in case, please?

--
assignee:  - tim.golden

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue8631
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue8006] os.popen in Python 3.1

2010-08-08 Thread Tim Golden

Tim Golden m...@timgolden.me.uk added the comment:

I've just run this:

code
import os
fd = os.popen(cat.exe, w)

for i in range (100):
  _ = fd.write (%d\n % i)

fd.flush ()

/code

and seen the expected list of numbers on the screen. cat.exe is from the 
gnuwin32 tools but I'm assuming that any equivalent .exe would do as well.

I suspect that the OP's issue may be with buffering, but I'd like to hear back. 
Switching to pending before closing as invalid. If the OP wishes to come back 
with a repeatable failing case I'm quite happy to look at it.

--
status: open - pending

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue8006
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue1759845] subprocess.call fails with unicode strings in command line

2010-08-08 Thread Tim Golden

Tim Golden m...@timgolden.me.uk added the comment:

To confirm the situation on 3.x: a unicode string with non-ascii-encodable 
characters is fine. The easy test here in the uk is a pound sign:

code
import subprocess

FILENAME = abc£.bat
FILENAME.encode (ascii)
#
# UnicodeEncodeError
#
with open (FILENAME, w) as f:
  f.write (echo hello\n)

subprocess.call ([FILENAME])
#
# hello output as expected
#

/code

So no action for 3.x. I'm sympathetic (in principle) to making a change to 2.7 
but I haven't looked over the competing patches and assessed the ins-and-outs.

--
assignee:  - tim.golden
nosy: +tim.golden

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1759845
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue1707753] get status output fix for Win32

2010-08-08 Thread Tim Golden

Tim Golden m...@timgolden.me.uk added the comment:

OK, I'm going to close this one:

* commands is out of 3.x
* it's a convenience for 2.x anyway, not a showstopper

If anyone feels keen enough to reopen the request, I'm willing to commit a 
suitable patch against the release27-maint branch, but I'm not interested in 
running it up myself.

--
resolution:  - out of date
stage: unit test needed - committed/rejected
status: open - closed

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1707753
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue5673] Add timeout option to subprocess.Popen

2010-08-09 Thread Tim Golden

Changes by Tim Golden m...@timgolden.me.uk:


--
nosy: +tim.golden

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue5673
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue949667] setblocking() method on file objects

2010-08-09 Thread Tim Golden

Tim Golden m...@timgolden.me.uk added the comment:

There are at least two ways to do non-blocking file IO on Windows:

Overlapped I/O
I/O Completion ports

Don't know what's best here, but happy to see what might be achieved
if it was thought worth pursuing.

--
nosy: +tim.golden

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue949667
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue9055] test_issue_8959_b fails when run from a service

2010-08-11 Thread Tim Golden

Tim Golden m...@timgolden.me.uk added the comment:

In the interests of moving this forward, I've committed the one-line removal of 
the assertion in r83948. Hopefully that will bring this buildbot back to life.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9055
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue9575] os.listdir() crashes on some long and deep paths in Windows 7

2010-08-12 Thread Tim Golden

Changes by Tim Golden m...@timgolden.me.uk:


--
assignee:  - tim.golden
nosy: +tim.golden
versions:  -Python 2.5

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9575
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue9055] test_issue_8959_b fails when run from a service

2010-08-12 Thread Tim Golden

Tim Golden m...@timgolden.me.uk added the comment:

Fudge-fix committed as r83948, r83958. Not sure what status to set

--
status: open - pending

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9055
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue2304] subprocess under windows fails to quote properly when shell=True

2010-08-12 Thread Tim Golden

Tim Golden m...@timgolden.me.uk added the comment:

ReComitted as r83947, r83956, r83957 and this time the buildbots look happy. 
(At least as regards this change).

--
resolution:  - fixed
status: open - closed

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2304
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue9575] os.listdir() crashes on some long and deep paths in Windows 7

2010-08-12 Thread Tim Golden

Tim Golden m...@timgolden.me.uk added the comment:

I can't get it crash on a path that short. I can produce an error message if I 
push it beyond the 254 limit, but you can work around that by applying the 
filesystem namespace prefix:

code
import os

path = c:\\ + \\.join (130 * ['xx'])
os.makedirs (path)
#
# will fail more or less violently
#
path = ?\\ + path
os.makedirs (path)
os.listdir (path)

/code

Probably related issues:

http://bugs.python.org/issue9246
http://bugs.python.org/issue4071

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9575
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue9575] os.listdir() crashes on some long and deep paths in Windows 7

2010-08-12 Thread Tim Golden

Tim Golden m...@timgolden.me.uk added the comment:

See: http://msdn.microsoft.com/en-us/library/aa365247%28VS.85%29.aspx

I tried first with your exact path and it caused no issues on
my Win7 box. FWIW you could easily roll your own os.walk
(starting by copying the code that's there) if you needed
to roll in special-cases.

Could you provide a batch file / Python script which exactly
reproduces the issue and a cut-and-paste of the resulting
traceback. (Unless crash really does mean crash).

Would you be able to narrow the issue down any further: see
if any other paths cause the same issue, or if a longer or
a shorter version of the path is problematic.

TJG

--
title: os.listdir() crashes on some long and deep paths in Windows 7 - 
os.listdir() crashes on some long and deep paths in Windows 7

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9575
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue1475] test_popen fails when the directory contains a space

2010-08-12 Thread Tim Golden

Tim Golden m...@timgolden.me.uk added the comment:

Fixed by issue2304

--
components:  -Library (Lib), Tests, Windows
nosy: +tim.golden
resolution:  - duplicate
stage:  - committed/rejected
status: open - closed
type:  - behavior

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1475
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue5484] subprocess.call() fails for .bat files on Windows, if executable path contains parenthesis

2010-08-12 Thread Tim Golden

Tim Golden m...@timgolden.me.uk added the comment:

OK; issue2304 is now fixed and closed. But it doesn't seem to make any 
difference to this behaviour. I've just retested and I see on py3k the 
behaviour the OP saw. CreateProcess does the same thing and I couldn't even get 
it to work with multiply-nested quotes. So I would maintain it's not a Python 
issue as such.

I'm closing as won't fix and leave it to the OP to reopen if he feels there's 
something which could be done.

--
assignee:  - tim.golden
resolution:  - wont fix
stage: unit test needed - committed/rejected
status: open - closed

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue5484
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue9575] os.listdir() crashes on some long and deep paths in Windows 7

2010-08-13 Thread Tim Golden

Tim Golden m...@timgolden.me.uk added the comment:

Thanks for the feedback. I'll close this for now as works for me. Feel free 
to reopen if you can come up with anything fresh.

--
resolution:  - works for me
stage:  - committed/rejected
status: open - closed

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9575
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue6609] zipfile: WindowsError [267] The directory name is invalid

2010-08-13 Thread Tim Golden

Tim Golden m...@timgolden.me.uk added the comment:

Closing as won't fix: this would be the same if you had any code which tried 
to do os.mkdir (aux). By way of comparison, Info-ZIP's UnZip returns 
checkdir error: aux exists but is not directory which actually seems less 
useful!

I don't believe it's the responsibility of the zipfile module (or tarfile or 
bz2 or any other) to silently reinterpret the names of files -- how would any 
code in the package expecting aux/code.c cope with the name change?

--
assignee:  - tim.golden
resolution:  - wont fix
stage:  - committed/rejected
status: open - closed

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6609
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue9584] Allow curly braces in fnmatch

2010-08-13 Thread Tim Golden

Tim Golden m...@timgolden.me.uk added the comment:

I don't see any reason to turn this down except, perhaps, for keeping something 
simple. 

Certainly I don't believe that Windows users will be confused by the fact that 
there are wildcards other than * and ?. fnmatch already implements [] and 
[!] which are not supported on Windows.

--
nosy: +tim.golden

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9584
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue9588] Skip subprocess shell tests on Windows per file association setup

2010-08-13 Thread Tim Golden

Tim Golden m...@timgolden.me.uk added the comment:

Assuming I understand you correctly, could I propose this rather less involved 
patch which simply specifies the sys.executable as part of the command line. 
The test doesn't propose to test file associations and indeed two of the test 
already fill in the executable

--
Added file: http://bugs.python.org/file18511/9588.exec.patch

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9588
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue1102] Add support for _msi.Record.GetString() and _msi.Record.GetInteger()

2010-08-16 Thread Tim Golden

Changes by Tim Golden m...@timgolden.me.uk:


--
components: +Windows
nosy: +tim.golden

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1102
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue9627] Regrtest failed to clean up temporary directory

2010-08-17 Thread Tim Golden

Tim Golden m...@timgolden.me.uk added the comment:

This is usually because the bug mentioned in issue7443 (although it 
could be something else, obviously). It should sort itself out on the 
next run. I'll rerun on my local checkout to see if there real WinXP issues.

--
nosy: +tim.golden

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9627
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue9433] regrtest.py -j 2 doesn't work on Windows: remove close_fds=True on Windows

2010-08-18 Thread Tim Golden

Tim Golden m...@timgolden.me.uk added the comment:

I can confirm that the patched regrtest runs ok on WinXP.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9433
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue1005895] curses for win32

2010-08-19 Thread Tim Golden

Tim Golden m...@timgolden.me.uk added the comment:

It looks as though issue2889 has a better chance of getting into the VS build 
than this one, which appears to be MingW-based (at a quick glance). I'm loosely 
keen to see it in, although I have no knowledge of curses as such. I'll assign 
it to myself so it appears on my to-do queue but I'm not promising to do 
anything with it immediately. Anyone who feels up to the task, feel free to 
take it off me.

Mark: why do you consider issue1005895 the master, so to speak? I can see more 
mileage in pursuing the other?

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1005895
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue2889] curses for windows (alternative patch)

2010-08-19 Thread Tim Golden

Tim Golden m...@timgolden.me.uk added the comment:

I'll pick it up for the moment to shepherd it along because I'm reasonably keen 
to see a Windows curses in the stdlib. However, I'm no expert in curses and I 
don't promise to do anything immediate with it.

--
assignee:  - tim.golden

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2889
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue798876] windows sys.path contains nonexistant directory

2010-08-20 Thread Tim Golden

Changes by Tim Golden m...@timgolden.me.uk:


--
assignee:  - tim.golden
components:  -Interpreter Core
nosy: +tim.golden
versions: +Python 3.1, Python 3.2

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue798876
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue9654] merge PC/getpathp.c into Modules/getpath.c

2010-08-23 Thread Tim Golden

Tim Golden m...@timgolden.me.uk added the comment:

+1 in principle

--
nosy: +tim.golden

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9654
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue2889] curses for windows (alternative patch)

2010-08-24 Thread Tim Golden

Tim Golden m...@timgolden.me.uk added the comment:

I have no strong opinion, Roumen, (and no experience with the package) 
but why -1 from you?

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2889
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue1559298] test_popen fails on Windows if installed to Program Files

2010-08-25 Thread Tim Golden

Changes by Tim Golden m...@timgolden.me.uk:


--
assignee:  - tim.golden
nosy: +tim.golden

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1559298
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue2528] Change os.access to check ACLs under Windows

2010-08-25 Thread Tim Golden

Changes by Tim Golden m...@timgolden.me.uk:


--
assignee:  - tim.golden

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2528
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue9699] invalid call of Windows API _popen() generating The input line is too long error message

2010-08-27 Thread Tim Golden

Changes by Tim Golden m...@timgolden.me.uk:


--
assignee:  - tim.golden

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9699
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue9699] invalid call of Windows API _popen() generating The input line is too long error message

2010-08-27 Thread Tim Golden

Tim Golden m...@timgolden.me.uk added the comment:

I'll pick this up and try to decide what's best to
do. I'm away from home for a few weeks and have only
intermittent internet access. Thanks, Sorin, for
a reproducible test case.

--
title: invalid call of Windows API _popen() generating The input line is too 
long error message - invalid call of Windows API _popen() generating The input 
   line is too long error message

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9699
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue9808] Implement os.getlogin on Windows

2010-09-09 Thread Tim Golden

Tim Golden m...@timgolden.me.uk added the comment:

+1 on the idea in general; I'm away at the moment
and not in a position to review the patch. Hopefully
Brian can review/commit

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9808
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



  1   2   3   4   5   6   7   >