[issue1731717] race condition in subprocess module

[Gabriel]

Gabriel  added the comment:

I had this happen to me today, I used Popen and os.waitpid(p.pid, 0) and
I got an "OSError: [Errno 10] No child processes". I haven't tried
adding a sleep, since it's unreliable. I'm using python 2.5.2, Windows
XP. I haven't tried this on linux yet as the problem arose while testing
my app for windows.

--
nosy: +grossetti

___
Python tracker 

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

[issue5358] Unicode control characters are not allowed as identifiers

[Baiju M]

New submission from Baiju M :

I tried to use Zero-width joiner (U+200D) as part of an identifier.
It produce an exception like this:

SyntaxError: invalid character in identifier

I have attached the Python file which produce this error.

Zero-width joiner (U+200D) is a Unicode control character:
http://en.wikipedia.org/wiki/Unicode_control_characters

--
components: Unicode
files: identifier.py
messages: 82664
nosy: baijum
severity: normal
status: open
title: Unicode control characters are not allowed as identifiers
type: behavior
versions: Python 3.0, Python 3.1
Added file: http://bugs.python.org/file13162/identifier.py

___
Python tracker 

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

[issue5359] _dbm extension only built using gdbm on linux

[Matthias Klose]

New submission from Matthias Klose :

in 3.x the _dbm extension can only be built with gdbm as the backend
library. gdbm is licensed under the GPL, so some people/projects like
Debian may have objections to build _dbm with the gdbm backend. It gets
difficult to determine, which code then uses _dbm/anydbm/... and then
check if it is compatible with the GPL.

Unless there is an alternative like dbm.sqlite (issue 3783), I would
like to re-add the detection of the BerkleyDB headers and libs, such
that _dbm can be built again without depending on gdbm. Adding a
configure flag to explicitely select one implementation (issue 4587)
would be helpful.

Ok to go ahead?

--
components: Build
messages: 82665
nosy: doko
severity: normal
status: open
title: _dbm extension only built using gdbm on linux
versions: Python 3.0, Python 3.1

___
Python tracker 

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

[issue5358] Unicode control characters are not allowed as identifiers

[Martin v. Löwis]

Martin v. Löwis  added the comment:

Why do you think this is a bug?

--
nosy: +loewis

___
Python tracker 

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

[issue5335] mmap can crash with tagname (on windows)

[Hirokazu Yamamoto]

Hirokazu Yamamoto  added the comment:

This is duplicated issue of issue1733986. So I'll closing...

--
resolution:  -> duplicate
status: open -> closed
superseder:  -> mmap.mmap can overrun buffer

___
Python tracker 

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

[issue1733986] mmap.mmap can overrun buffer

[Hirokazu Yamamoto]

Hirokazu Yamamoto  added the comment:

I came from issue5335. I think attached patch will fix this issue.

--
keywords: +patch
nosy: +ocean-city
priority: normal -> high
type:  -> crash
versions: +Python 2.6, Python 2.7, Python 3.0, Python 3.1
Added file: http://bugs.python.org/file13163/fix_tagname_crash.patch

___
Python tracker 

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

[issue5358] Unicode control characters are not allowed as identifiers

[Ezio Melotti]

Changes by Ezio Melotti :


--
nosy: +ezio.melotti

___
Python tracker 

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

[issue4258] Use 30-bit digits instead of 15-bit digits for Python integers.

[Mark Dickinson]

Mark Dickinson  added the comment:

Okay, let's abandon 30-bit digits on 32-bit machines:  it's still 
unclear whether there's any real performance gain, and it's trivial to 
re-enable 30-bit digits by default later.  I'm also going to abandon the 
optimizations for now;  it'll be much easier to work on them once the base 
patch is in.

Here's a 'release-candidate' version of the patch:  it's exactly the 
same as before, except:

  - I removed all x_divrem changes (though I've left the extra division
tests in, since they're just as relevant to the old x_divrem).
I'll open a separate issue for these optimizations.

  - enable 30-bit digits by default only on 64-bit platforms, where
for the purposes of this patch a 64-bit platform is one with all
the necessary integer types available (signed and unsigned 32-
and 64-bit integer types) and SIZEOF_VOID_P >= 8.

I've also updated the version uploaded to Rietveld:  the patchset there 
should exactly match 30bit_longdigit20.patch.  See:

  http://codereview.appspot.com/14105


Martin, thank you for all your help with reviewing the previous patch.  
Is it okay with you to check this version in?  It's the same as the 
version that you reviewed, except with some extra tests for correct 
division, and with 30-bit digits disabled by default on 32-bit 
platforms.

Gregory, if you have time, please could you double check that the 
configure stuff is working okay for you with this patch?  "configure --
enable-big-digits" should produce 30-bit digits; "configure --disable-
big-digits" should produce 15-bit digits, and a plain "configure" should 
give 15-bit or 30-bit depending on your platform.

Anyone else have any objections to this going in exactly as it is?  I'd 
quite like to resolve this issue one way or the other and move on.

--
assignee:  -> marketdickinson
Added file: http://bugs.python.org/file13164/30bit_longdigit20.patch

___
Python tracker 

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

[issue5360] RO (shorthand for READONLY) gone, not in documentation

[Rémi Koenig]

New submission from Rémi Koenig :

The Defining New Types document for python3
(http://docs.python.org/3.0/extending/newtypes.html) cites the RO macro
(which existed in versions < 3). However, it has been removed in py3k
(cf http://wiki.python.org/moin/Py3kExtensionModules or
python3.0/structmember.h).

--
assignee: georg.brandl
components: Documentation
messages: 82670
nosy: Rémi, georg.brandl
severity: normal
status: open
title: RO (shorthand for READONLY) gone, not in documentation
type: compile error
versions: Python 3.0

___
Python tracker 

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

[issue4565] Rewrite the IO stack in C

[Benjamin Peterson]

Benjamin Peterson  added the comment:

We also have to figure out how to make the C IOBase a ABC, so people can
implement it.

___
Python tracker 

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

[issue5360] RO (shorthand for READONLY) gone, not in documentation

[Rémi Koenig]

Rémi Koenig  added the comment:

Sorry to bother, but I just noticed that in this same document
(http://docs.python.org/3.0/extending/newtypes.html),
PyObject_HEAD_INIT(NULL) has not been replaced by
PyVarObject_HEAD_INIT(NULL, 0).

___
Python tracker 

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

[issue2636] Regexp 2.7 (modifications to current re 2.2.2)

[Matthew Barnett]

Matthew Barnett  added the comment:

issue2636-features-3.diff is based on the 2.x trunk.

Added comments.
Restricted line lengths to no more than 80 characters
Added common POSIX character classes like [[:alpha:]].
Added further checks to reduce unnecessary backtracking.

I've decided to remove \onnn and \x{n} because they aren't supported
elsewhere in the language.

Added file: http://bugs.python.org/file13165/issue2636-features-3.diff

___
Python tracker 

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

[issue4474] PyUnicode_FromWideChar incorrect for characters outside the BMP (unix only)

[Mark Dickinson]

Mark Dickinson  added the comment:

Updated Victor's patch:

  - applies cleanly against newly whitespace-normalized unicodeobject.c
  - renamed USE_WCHAR_SURROGATE to CONVERT_WCHAR_TO_SURROGATES
  - add defined(SIZEOF_WCHAR_T) check

I find the patched version of PyUnicode_FromWideChar quite hard to follow 
with all the #ifdefery.  I wonder whether it might be better to define a 
_PyUnicode16_FromWideChar32 helper function instead.

Added file: 
http://bugs.python.org/file13166/unicode_fromwidechar_surrogate-5.patch

___
Python tracker 

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

[issue4474] PyUnicode_FromWideChar incorrect for characters outside the BMP (unix only)

[Marc-Andre Lemburg]

Marc-Andre Lemburg  added the comment:

On 2009-02-24 20:39, Mark Dickinson wrote:
> Mark Dickinson  added the comment:
> 
> Updated Victor's patch:
> 
>   - applies cleanly against newly whitespace-normalized unicodeobject.c
>   - renamed USE_WCHAR_SURROGATE to CONVERT_WCHAR_TO_SURROGATES
>   - add defined(SIZEOF_WCHAR_T) check
> 
> I find the patched version of PyUnicode_FromWideChar quite hard to follow 
> with all the #ifdefery.  I wonder whether it might be better to define a 
> _PyUnicode16_FromWideChar32 helper function instead.

Same here. It would be better to have a single #ifdef #else #endif
block with one branch for the CONVERT_WCHAR_TO_SURROGATES case and the
other for the normal operation.

No need for a new helper function.

___
Python tracker 

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

[issue4474] PyUnicode_FromWideChar incorrect for characters outside the BMP (unix only)

[Mark Dickinson]

Mark Dickinson  added the comment:

> It would be better to have a single #ifdef #else #endif

Yes, of course it would. :)

___
Python tracker 

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

[issue4565] Rewrite the IO stack in C

[Antoine Pitrou]

Antoine Pitrou  added the comment:

> We also have to figure out how to make the C IOBase a ABC, so people can
> implement it.

Mmmh, I know absolutely nothing about the ABC implementation.

___
Python tracker 

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

[issue1578269] Add os.link() and os.symlink() support for Windows

[Giampaolo Rodola']

Changes by Giampaolo Rodola' :


--
nosy: +giampaolo.rodola

___
Python tracker 

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

[issue4474] PyUnicode_FromWideChar incorrect for characters outside the BMP (unix only)

[Mark Dickinson]

Mark Dickinson  added the comment:

New patch, with two separate versions of PyUnicode_FromWideChar.

Added file: 
http://bugs.python.org/file13167/unicode_fromwidechar_surrogate-6.patch

___
Python tracker 

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

[issue4474] PyUnicode_FromWideChar incorrect for characters outside the BMP (unix only)

[Marc-Andre Lemburg]

Marc-Andre Lemburg  added the comment:

On 2009-02-24 21:50, Mark Dickinson wrote:
> Mark Dickinson  added the comment:
> 
> New patch, with two separate versions of PyUnicode_FromWideChar.

Thanks, much better :-)

___
Python tracker 

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

[issue5361] Obsolete mispelled in string formatting docs

[Steven D'Aprano]

New submission from Steven D'Aprano :

The table of string formatting conversions has a mispelling:

http://docs.python.org/library/stdtypes.html#string-formatting

'u'  Obselete type – it is identical to 'd'.

Should be "Obsolete".

--
assignee: georg.brandl
components: Documentation
messages: 82680
nosy: georg.brandl, stevenjd
severity: normal
status: open
title: Obsolete mispelled in string formatting docs

___
Python tracker 

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

[issue1006238] cross compile patch

[Garrett Cooper]

Garrett Cooper  added the comment:

I can definitely chime in on this issue.

A (proper) testcase would be to do something like the following:

# Example item -- this isn't what you'll be using...
CROSS_COMPILE_PREFIX=binos_c3.4.3-p1.mips64-octeon-linux-

AS="${CROSS_COMPILE}as" \
CC="${CROSS_COMPILE}gcc" \
CC="${CROSS_COMPILE}c++" \
LD="${CROSS_COMPILE}ld" \
NM="${CROSS_COMPILE}nm" \
./configure --prefix=/usr/local \
BUILD_PYTHON=/path/to/native/python \
DESTDIR=/where/i/want/to/install/python \
--build=`"${CROSS_COMPILE}gcc" -dumpmachine` \
--host=`arch`

NOTES:
- arch(1) is a GNU specific utility, so it won't work outside of Linux!
- I'll leave it as an exercise to the reader to specify the correct
CFLAGS, CPPFLAGS, LDFLAGS, and LDLIBS.

Some testcases during the compile that fail if cross-compiling are (with
2.6.1):
- chflags
- lchflags
- printf with %zd format support.

Thanks!

--
nosy: +yaneurabeya

___
Python tracker 

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

[issue1006238] cross compile patch

[Garrett Cooper]

Garrett Cooper  added the comment:

Am I correct in this understanding, or is Pgen unneeded after the
grammar file is generated?

___
Python tracker 

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

[issue5362] Add configure option to disable Py3k warnings

[Collin Winter]

New submission from Collin Winter :

The attached patch adds a --with-py3k-warnings option to configure.
Passing --without-py3k-warnings disables all Py3k compatibility warnings
(the default is to keep the warnings). For production deployments where
performance is more important than warnings no-one will see, this can
provide a useful performance improvement:

2to3 (translate 2to3's source tree five times):
Min: 22.406 -> 22.114: 1.32% faster
Avg: 22.439 -> 22.158: 1.27% faster

Django (render a 150x150 table 100 times):
Min: 0.595 -> 0.586: 1.58% faster
Avg: 0.598 -> 0.588: 1.76% faster

Spitfire (render a 1000x1000 table 100 times):
Min: 0.752 -> 0.743: 1.29% faster
Avg: 0.754 -> 0.745: 1.25% faster

Unpickle (unpickling a list of 8000 dicts 100 times):
Min: 0.305 -> 0.301: 1.41% faster
Avg: 0.307 -> 0.302: 1.49% faster

Build env: gcc 4.3.1 x86_64 on Linux 2.6.18 (Core2 Duo)

--
assignee: jyasskin
components: Interpreter Core
files: no_py3k_warning.patch
keywords: needs review, patch, patch
messages: 82683
nosy: brett.cannon, collinwinter, jyasskin, pitrou
severity: normal
status: open
title: Add configure option to disable Py3k warnings
type: performance
versions: Python 2.7
Added file: http://bugs.python.org/file13168/no_py3k_warning.patch

___
Python tracker 

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

[issue3959] Add Google's ipaddr.py to the stdlib

[pmoody]

pmoody  added the comment:

Are there any committers who'd be able to help get this integrated?
Martin is booked solid until April and that would apparently
jeopardize the inclusion of ipaddr in python 3.1.

Cheers,
/peter

On Wed, Feb 4, 2009 at 11:27 AM, Martin v. Löwis  wrote:
>
> Martin v. Löwis  added the comment:
>
>> in light of david's email, would you agree that the bar for buy-in has
>> been met or should I now start writing a PEP and preparing offerings
>> for the BDFL?
>
> Yes, I think it can be integrated now. I'll look into it, unless
> somebody is faster (it could well take some weeks until I find some
> time)
>
> ___
> Python tracker 
> 
> ___
>

___
Python tracker 

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

[issue5362] Add configure option to disable Py3k warnings

[Martin v. Löwis]

Martin v. Löwis  added the comment:

I would like to understand the problem better first. I find it hard to
believe that mere access to a global (even if frequent) costs so much. I
think we should strive to make the warnings check faster if it indeed
produces significant runtime costs, rather than offering to remove it.

--
nosy: +loewis

___
Python tracker 

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

[issue5084] unpickling does not intern attribute names

[Jake McGuire]

Jake McGuire  added the comment:

The fromstring/asstring dance was due to my incomplete understanding of 
refcounting.  PyDict_Next returns a borrowed reference but 
PyString_InternInPlace expects an owned reference.  Thanks to Kirk 
McDonald, I have a new patch that does the refcounting correctly.

What sort of test did you have in mind?

Added file: http://bugs.python.org/file13169/cPickle.c.diff

___
Python tracker 

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

[issue4715] optimize bytecode for conditional branches

[Jeffrey Yasskin]

Jeffrey Yasskin  added the comment:

I've updated Antoine's patch to incorporate my comments. This interacts
with issue 2459, but I haven't yet looked at its patch to figure out
how. As a first cut, I'll propose committing this patch, backporting it
to trunk, syncing it into the issue 2459 patch, and then committing that
patch. Let me know if that's crazy.

http://codereview.appspot.com/20064

Added file: http://bugs.python.org/file13170/condbranches-plus2.patch

___
Python tracker 

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

[issue3959] Add Google's ipaddr.py to the stdlib

[Antoine Pitrou]

Antoine Pitrou  added the comment:

You should post a message on python-dev and put your patch on Rietveld,
this will give you more chances to find an interested developer.

___
Python tracker 

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

[issue4715] optimize bytecode for conditional branches

[Antoine Pitrou]

Antoine Pitrou  added the comment:

Le mercredi 25 février 2009 à 00:51 +, Jeffrey Yasskin a écrit :
> I've updated Antoine's patch to incorporate my comments. This interacts
> with issue 2459, but I haven't yet looked at its patch to figure out
> how. As a first cut, I'll propose committing this patch, backporting it
> to trunk, syncing it into the issue 2459 patch, and then committing that
> patch. Let me know if that's crazy.

Doesn't sound crazier than doing it in another order :-))
There'll be a bit of work to reconcile both patches anyway (and also to
adapt the 2459 in order to apply cleanly against current trunk).

2459 defines its own JUMP_ABS_IF_TRUE / JUMP_ABS_IF_FALSE (which are the
same as this patch's POP_JUMP_IF_TRUE / POP_JUMP_IF_FALSE), but it also
keeps the old relative non-popping conditional jumps, which as this
issue shows is sub-optimal.

Thank you for taking this, Jeffrey, my own priority right now being the
io-c branch.

___
Python tracker 

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

[issue5084] unpickling does not intern attribute names

[Antoine Pitrou]

Antoine Pitrou  added the comment:

The test should check that the unpickled strings have been interned.

___
Python tracker 

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

[issue5084] unpickling does not intern attribute names

[Antoine Pitrou]

Antoine Pitrou  added the comment:

In your patch, I'm not sure where the `name` variable is coming from.
Have you checked it works?

___
Python tracker 

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

[issue5084] unpickling does not intern attribute names

[Antoine Pitrou]

Antoine Pitrou  added the comment:

To give an example of what the test could check:

>>> class C(object):
...  def __init__(self):
...self.some_long_attribute_name = 5
... 
>>> c = C()
>>> c.__dict__
{'some_long_attribute_name': 5}
>>> sorted(map(id, c.__dict__))
[140371243499696]
>>> import pickle
>>> d = pickle.loads(pickle.dumps(c, -1))
>>> d
<__main__.C object at 0x7faaba1b0390>
>>> d.__dict__
{'some_long_attribute_name': 5}
>>> sorted(map(id, d.__dict__))
[140371243501232]

The `sorted(map(id, d.__dict__))` should have been the same before and
after pickling.

___
Python tracker 

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

[issue5362] Add configure option to disable Py3k warnings

[Jeffrey Yasskin]

Jeffrey Yasskin  added the comment:

s/Leaving/Turning/ in configure.in.

It looks like the convention for other --with flags that default to
enabled is to document them as --with(out)-xxx. (except tsc...) I guess
it's probably even better just to say what the default is in the
documentation.

"Trade away speed for Py3k compat warnings" is a bit harsh for a 1.5%
penalty. How about "Define to get Py3k compatibility warnings at the
cost of a little speed"?

___
Python tracker 

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

[issue4715] optimize bytecode for conditional branches

[Jeffrey Yasskin]

Jeffrey Yasskin  added the comment:

Committed as r69961. I'll post the backport to trunk here at least a day
before I commit it.

--
type: resource usage -> performance
versions: +Python 2.7

___
Python tracker 

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

[issue1676] Fork/exec issues with Tk 8.5/Python 2.5.1 on OS X

[Lars Yencken]

Changes by Lars Yencken :


--
nosy: +lars512

___
Python tracker 

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

[issue4609] Allow use of > 256 FD's on solaris in 32 bit mode

[Ross Hayden]

Changes by Ross Hayden :


--
nosy: +ross

___
Python tracker 

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

[issue3959] Add Google's ipaddr.py to the stdlib

[Gregory P. Smith]

Gregory P. Smith  added the comment:

I'll take care of this.  My goal is to do it before PyCon.

--
assignee:  -> gregory.p.smith

___
Python tracker 

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

[issue5363] Documentation of filecmp.compfiles missing word & possible explanation

[Mitchell Model]

New submission from Mitchell Model :

"contains the list of files match in both directories," should have 
"that" before "match"

In addition I couldn't figure out from the documentation what "common" 
was supposed to be doing -- it sounded more like something that should 
be part of the result not a parameter. I had to look at the code to 
realize that this function ONLY compares the files listed in common -- 
that it doesn't compare two directories completely, but rather dir1 and 
dir2 are merely prefixes for the files listed in common that are to be 
compared.

One thing that would help would be to move the paragraph about common 
first. Another would be to "a list of file names found in both 
directories" to "a list of the names of files to be compared, which must 
be in both directories" or something like that.

--
assignee: tarek
components: Distutils
messages: 82696
nosy: MLModel, tarek
severity: normal
status: open
title: Documentation of filecmp.compfiles missing word & possible explanation
versions: Python 2.6, Python 2.7, Python 3.0, Python 3.1

___
Python tracker 

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

[issue5363] Documentation of filecmp.compfiles missing word & possible explanation

[Mitchell Model]

Changes by Mitchell Model :


--
components: +Documentation -Distutils

___
Python tracker 

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

[issue2054] add ftp-tls support to ftplib - RFC 4217

[Jeff Oyama]

Jeff Oyama  added the comment:

Actually I have encountered a possible bug. the close() method doesn't seem
to actually close the connection...

On Mon, Feb 23, 2009 at 11:56 PM, Jeff Oyama  wrote:

>
> Jeff Oyama  added the comment:
>
> Thank you Giampaolo, it works just as I was hoping, =] I tested it on
> glftpd
> using python 2.6.1.
>
> Added file: http://bugs.python.org/file13161/unnamed
>
> ___
> Python tracker 
> 
> ___
>

Added file: http://bugs.python.org/file13171/unnamed

___
Python tracker 

___Actually I have encountered a possible bug. the close() method doesn't seem 
to actually close the connection...On Mon, Feb 
23, 2009 at 11:56 PM, Jeff Oyama rep...@bugs.python.org> 
wrote:

Jeff Oyama j...@oyama.org> added the 
comment:

Thank you Giampaolo, it works just as I was hoping, 
=] I tested it on glftpd
using python 2.6.1.

Added file: http://bugs.python.org/file13161/unnamed"; 
target="_blank">http://bugs.python.org/file13161/unnamed

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