[issue14097] Improve the "introduction" page of the tutorial

2012-02-25 Thread Ezio Melotti

Ezio Melotti  added the comment:

> Be careful with whitespace changes.

Here I agree with you and disagree with the PEP 8 (specifically where it says 
that "hypot2 = x*x + y*y" and "c = (a+b) * (a-b)" are "wrong").

In
-   >>> 3+1j*3
+   >>> 3 + 1j * 3
I intentionally added spaces around all the operators to leave the expression 
ambiguous.  IIUC the point of that example is to show that even if 3+1j is a 
single entity, normal precedence rules still apply, so I didn't want the reader 
to make any assumptions about the precedence based on the whitespace (or even 
worse thing that the whitespace might define the precedence).

Additional comments and replies about the list:
 1) A user was confused about >>>, thought that was necessary to "define 
variables", that you had to always include it in your code.  The tutorial 
should make clear that this is used only in the interactive interpreter and not 
in regular code;
 2-3) what Eli said sounds good to me;
 5) "A value can be assigned to several variables simultaneously:" might lead 
to thing that "a = b = []" is the same as "a = []; b= []".  The term "variable" 
has a different meaning in other languages, but that doesn't mean we should 
refrain to use it -- we just have to specify what we mean with "variable" (this 
can be done later, saying that "both a and b will refer to the same list" 
should be enough here);
 6) here the idea is that is not the value to be assigned to the variable (like 
in C where the vars are like boxes and you can put values inside), but the 
variable that is used to refer to the object (so it's more like a label 
assigned to the object).  Again we can don't need to be too specific here yet, 
but we shouldn't say wrong things either.  See also 
http://python.net/crew/mwh/hacks/objectthink.html;
 11) presenting the error raised with the index first and the fact that slices 
are more "permissive" later sounds better to me;
 13) I'm not saying we should specifically tell users to put object of the same 
kind in a list, but just to avoid focusing on this aspect and use "realistic" 
examples.  Adding a note at the end saying that "lists can also contain objects 
of different types" should be enough;

--

___
Python tracker 

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



[issue12119] distutils and python -B

2012-02-25 Thread Éric Araujo

Éric Araujo  added the comment:

I thought about this and am not sure about my position.

I am 100% convinced that what packaging does is the right thing.  Modules like 
py_compile, compileall (in 3.2+) and packaging are able to compile with or 
without optimization depending on options that are given to them, independently 
from the optimization level of the calling Python.

distutils however depends on the calling Python’s -B and -O options, and I just 
don’t think there is much value in fixing this bug and causing obscure bugs for 
people who’ve been using -B to control distutils for the past months.

Antoine, you’re a pragmatist and a fixer of bugs.  Arfrever, you’re using this 
misfeature.  I’d like to know your opinion.

--
resolution:  -> fixed
status: open -> pending

___
Python tracker 

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



[issue13447] Add tests for some scripts in Tools/scripts

2012-02-25 Thread Francisco Martín Brugué

Francisco Martín Brugué  added the comment:

Hi Éric,
some questions:

1)if test_tools is going to be the test for all the Tools (at least
until it grows to much), shoudn't be be module doc something like “””Tests for 
scripts in Tools/**“””

2)is the SkipTest “reindent” specific? 

Cheers,

francis

--

___
Python tracker 

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



[issue14112] tutorial intro talks of "shallow copy" concept without explanation

2012-02-25 Thread Ezio Melotti

Ezio Melotti  added the comment:

Even if they know the meaning of "shallow" (which is not a really common word 
AFAICT), they might not know what it means in this context.

Adding an entry to glossary might be a better solution.

In this context I think the best solution would be to actually show the 
implication of having a shallow copy with another short example (hijacking the 
reader to some other page where they can find some in-depth description of what 
"shallow" might do more harm than good).

--

___
Python tracker 

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



[issue6884] Impossible to include file in sdist that starts with 'build' on Win32

2012-02-25 Thread Nadeem Vawda

Nadeem Vawda  added the comment:

There are some minor errors in your v3 patch. I've attached a v4 that fixes
them (as usual, tested on Windows and Linux):

 - s/special/sep/ in glob_to_re()
 - Add missing l(.) to filenames in test_process_template under 'graft d'


>> - test_glob_to_re() was doing two levels of escaping (r'\' -> r'')
>>   for its expected output when it should only do one (r'\' -> r'\\').
> Fix merged.  I don’t fully understand why one place needs two escapes and the 
> others just one.

The places that use one level of escaping are the ones that deal with the
regex string directly. In glob_to_re() itself, the string you're building
is a regex replacement pattern that *operates on* the regex that gets
returned, so it needs one level of escaping for when the output regex is
actually applied, and a second one for the immediate call to re.sub().
When re.sub is called, it eats one level of the escaping, and operates on
a string with one level, returning a string with one level.


>> I also took the liberty of changing the checks for whether the separator 
>> needs to be escaped
>> - it's better to escape everything except "/", just in case someone decides 
>> to port Python to
>> some platform using a weird directory separator that is neither "/" nor r"\".
> I didn’t take that part.  If someone wants to port Python with a new style of 
> os.sep or os.extsep, I’ll deal with it when they report that.

If you want to leave it as it is, that's your choice. But I do think it's
better to make this Do The Right Thing now, given how easy it is.


> On a related subject, the code in sdist that excludes VCS dirs uses r'/|\\'; 
> I thought that os.altsep was an alternate *accepted* separator, but not 
> actually produced by OS calls.  IOW, I’m asking if os.walk (used to create 
> filelist.allfiles) can ever give paths with '/' on Windows.  If not, then the 
> regex is redundant, and I can clean it in packaging.  My patch removes the 
> redundancy, just for curiosity, but I won’t actually commit that part to 
> distutils.

Well, as far as I can make out, the current implementation always uses
r'\' to join paths. But it won't convert existing '/'s in the start
path to r'\'s, so you might still end up encountering both separators
(assuming the initial path is something you get from the user somewhere
along the line).

(These remarks are not specific to os.walk, but are relevant to anything
based on os.path.join - for example, distutils.filelist.findall and
packaging.manifest._findall)

--
Added file: http://bugs.python.org/file24636/filelist-regex-v4.diff

___
Python tracker 

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



[issue13340] list.index does not accept None as start or stop

2012-02-25 Thread Éric Araujo

Changes by Éric Araujo :


--
priority: high -> normal
stage: committed/rejected -> commit review
versions:  -Python 2.7, Python 3.2

___
Python tracker 

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



[issue4256] argparse: provide a simple way to get a programmatically useful list of options

2012-02-25 Thread Zbyszek Szmek

Zbyszek Szmek  added the comment:

ZSH can just present it in a prettier way, and also includes slightly
more info (the short explanations, ordering).

 > could print out enough information for both systems.
Exactly.

ZSH can use bash completion, but then it doesn't display the extra info. 
It would be nice to keep those optional features in mind to avoid 
limiting the exported information to that useful for bash.

--

___
Python tracker 

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



[issue6884] Impossible to include file in sdist that starts with 'build' on Win32

2012-02-25 Thread Nadeem Vawda

Nadeem Vawda  added the comment:

>> - test_glob_to_re() was doing two levels of escaping (r'\' -> r'')
>>   for its expected output when it should only do one (r'\' -> r'\\').
> Fix merged.  I don’t fully understand why one place needs two escapes and the 
> others just one.

I agree that this code is confusing, though. Perhaps we should add a
comment in glob_to_re():

# Warning - manipulating a regex with a regex. Here be dragons.

;-)

--

___
Python tracker 

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



[issue6884] Impossible to include file in sdist that starts with 'build' on Win32

2012-02-25 Thread Éric Araujo

Éric Araujo  added the comment:

Thanks a bunch.  I’ll offer you cookies when we meet :)

>> Fix merged.  I don’t fully understand why one place needs two escapes and 
>> the others just one.
> The places that use one level of escaping are the ones that deal with the 
> regex string directly.
> In glob_to_re() itself, the string you're building is a regex replacement 
> pattern that *operates on*
> the regex that gets returned
Makes sense.  I’ve added a comment as you suggested (although an unfunny one, 
without dragons; I don’t like “you’re not supposed to understand this”-like 
comments).

>>> I also took the liberty of changing the checks for whether the separator 
>>> needs to be escaped
>>> - it's better to escape everything except "/", just in case someone decides 
>>> to port Python to
>>> some platform using a weird directory separator that is neither "/" nor 
>>> r"\".
>> I didn’t take that part.  If someone wants to port Python with a new style 
>> of os.sep or os.extsep,
>> I’ll deal with it when they report that.
> If you want to leave it as it is, that's your choice. But I do think it's 
> better to make this Do The
> Right Thing now, given how easy it is.
I’d be okay with some cleanup in packaging, but for distutils I want the 
minimal patch.

> Well, as far as I can make out, the current implementation always uses r'\' 
> to join paths. But it
> won't convert existing '/'s in the start path to r'\'s, so you might still 
> end up encountering both
> separators (assuming the initial path is something you get from the user 
> somewhere along the line).
Hm, it should come from the manifest template, i.e. using '/' that get 
translated to os.sep.  I’ll remove the regex redundancy in packaging, we have 
rather good tests for manifest and sdist now.

--

___
Python tracker 

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



[issue13447] Add tests for some scripts in Tools/scripts

2012-02-25 Thread Nadeem Vawda

Nadeem Vawda  added the comment:

[Éric]
> New docstring:
>
>   Tests for scripts in the Tools directory.
>
>   This file contains regression tests for some of the scripts found in the
>   Tools directory of a Python checkout or tarball, such as reindent.py.
>
> When I commit I’ll also send to python-dev a reply to the python-checkins 
> email to advertise the new file.
>
> New skip message:
>
>   test irrelevant for an installed Python
>
> Better?

Sounds good to me.

[Francisco]
> 1)if test_tools is going to be the test for all the Tools (at least
> until it grows to much), shoudn't be be module doc something like “””Tests 
> for scripts in Tools/**“””

I think Éric's suggestion in msg154197 is fine.

> 2)is the SkipTest “reindent” specific? 

No, it's common to all of the Tools scripts. When you install Python, the
Tools directory is not copied into the install destination. So if you're
using an installed copy of Python (either from running "make install", or
from a binary installer), the scripts won't exist. In this case, we
(obviously) can't test them.

--

___
Python tracker 

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



[issue13447] Add tests for some scripts in Tools/scripts

2012-02-25 Thread Éric Araujo

Éric Araujo  added the comment:

> if test_tools is going to be the test for all the Tools (at least until it 
> grows to much),
Seriously, we don’t care about the size of test files.  test_argparse.py is 
4777 lines long :)

> shoudn't be be module doc something like “””Tests for scripts in Tools/**“””
I find this wording worse than mine, sorry.

> is the SkipTest “reindent” specific? 
The SkipTest is raised at the top level of the module.  No test from test_tools 
will be able to run outside of an uninstalled build.  Maybe you are asking that 
because my message uses the singular, i.e. “test irrelevant etc.” and not 
“tests”?  In this instance, the singular “test” refers to test_tools, viewed as 
one test.

--

___
Python tracker 

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



[issue6884] Impossible to include file in sdist that starts with 'build' on Win32

2012-02-25 Thread Nadeem Vawda

Nadeem Vawda  added the comment:

>>> Fix merged.  I don’t fully understand why one place needs two escapes and 
>>> the others just one.
>> The places that use one level of escaping are the ones that deal with the 
>> regex string directly.
>> In glob_to_re() itself, the string you're building is a regex replacement 
>> pattern that *operates on*
>> the regex that gets returned
> Makes sense.  I’ve added a comment as you suggested (although an unfunny one, 
> without dragons; I don’t like “you’re not supposed to understand this”-like 
> comments).

Hmm yeah, I had just meant the dragons bit as a joke... I wouldn't want
people to get the impression that the code should be treated as magic.


>>> I didn’t take that part.  If someone wants to port Python with a new style 
>>> of os.sep or os.extsep,
>>> I’ll deal with it when they report that.
>> If you want to leave it as it is, that's your choice. But I do think it's 
>> better to make this Do The
>> Right Thing now, given how easy it is.
> I’d be okay with some cleanup in packaging, but for distutils I want the 
> minimal patch.

That's fine by me.


>> Well, as far as I can make out, the current implementation always uses r'\' 
>> to join paths. But it
>> won't convert existing '/'s in the start path to r'\'s, so you might still 
>> end up encountering both
>> separators (assuming the initial path is something you get from the user 
>> somewhere along the line).
> Hm, it should come from the manifest template, i.e. using '/' that get 
> translated to os.sep.  I’ll remove the regex redundancy in packaging, we have 
> rather good tests for manifest and sdist now.

I was thinking that maybe findall()'s 'dir' argument could be specified
by the user as a command-line flag or something. But looking through
the sdist code, it seems that it always uses the current directory. So
your change shouldn't be a problem.

--

___
Python tracker 

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



[issue13447] Add tests for some scripts in Tools/scripts

2012-02-25 Thread Francisco Martín Brugué

Francisco Martín Brugué  added the comment:

Sorry, my fault: I meant "Test for Tools" instead of """Tests for 
reindent.py.""" (Im not talking about the skip message but the test 
documentation or the first line 0,0...)

--

___
Python tracker 

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



[issue8171] bdist_wininst builds wrongly for --plat-name=win-amd64

2012-02-25 Thread Éric Araujo

Éric Araujo  added the comment:

I’ll look into this when I get to set up a Windows VM and learn more about 
wininst.

--
assignee: tarek -> eric.araujo
components: +Distutils2
nosy: +alexis, eric.araujo
stage: patch review -> test needed
versions: +3rd party, Python 3.3 -Python 3.1

___
Python tracker 

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



[issue14102] argparse: add ability to create a man page

2012-02-25 Thread Andi Albrecht

Changes by Andi Albrecht :


--
nosy: +andialbrecht

___
Python tracker 

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



[issue8170] Wrong Paths for distutils build --plat-name=win-amd64

2012-02-25 Thread Éric Araujo

Éric Araujo  added the comment:

Martin, Mark, can either of you comment on this?

--
nosy: +eric.araujo, loewis, mhammond -terry.reedy
versions: +Python 3.3 -Python 3.1

___
Python tracker 

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



[issue8171] bdist_wininst builds wrongly for --plat-name=win-amd64

2012-02-25 Thread Éric Araujo

Changes by Éric Araujo :


--
nosy:  -terry.reedy

___
Python tracker 

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



[issue14120] ARM Ubuntu 3.x buildbot failing test_dbm

2012-02-25 Thread Nadeem Vawda

New submission from Nadeem Vawda :

The ARM Ubuntu 3.x buildbot often fails test_dbm:

http://www.python.org/dev/buildbot/all/builders/ARM%20Ubuntu%203.x/builds/364/steps/test/logs/stdio
http://www.python.org/dev/buildbot/all/builders/ARM%20Ubuntu%203.x/builds/367/steps/test/logs/stdio
http://www.python.org/dev/buildbot/all/builders/ARM%20Ubuntu%203.x/builds/368/steps/test/logs/stdio
http://www.python.org/dev/buildbot/all/builders/ARM%20Ubuntu%203.x/builds/372/steps/test/logs/stdio

==
ERROR: test_anydbm_creation (test.test_dbm.TestCase-dbm.ndbm)
--
Traceback (most recent call last):
  File 
"/var/lib/buildbot/buildarea/3.x.warsaw-ubuntu-arm/build/Lib/test/test_dbm.py", 
line 70, in test_anydbm_creation
self.read_helper(f)
  File 
"/var/lib/buildbot/buildarea/3.x.warsaw-ubuntu-arm/build/Lib/test/test_dbm.py", 
line 111, in read_helper
self.assertEqual(self._dict[key], f[key.encode("ascii")])
KeyError: b'0'

==
ERROR: test_anydbm_modification (test.test_dbm.TestCase-dbm.ndbm)
--
Traceback (most recent call last):
  File 
"/var/lib/buildbot/buildarea/3.x.warsaw-ubuntu-arm/build/Lib/test/test_dbm.py", 
line 85, in test_anydbm_modification
self.read_helper(f)
  File 
"/var/lib/buildbot/buildarea/3.x.warsaw-ubuntu-arm/build/Lib/test/test_dbm.py", 
line 111, in read_helper
self.assertEqual(self._dict[key], f[key.encode("ascii")])
KeyError: b'0'

==
ERROR: test_anydbm_read (test.test_dbm.TestCase-dbm.ndbm)
--
Traceback (most recent call last):
  File 
"/var/lib/buildbot/buildarea/3.x.warsaw-ubuntu-arm/build/Lib/test/test_dbm.py", 
line 91, in test_anydbm_read
self.read_helper(f)
  File 
"/var/lib/buildbot/buildarea/3.x.warsaw-ubuntu-arm/build/Lib/test/test_dbm.py", 
line 111, in read_helper
self.assertEqual(self._dict[key], f[key.encode("ascii")])
KeyError: b'0'

The stdio logs also show possibly-related output on stderr:

test test_dbm failed
@test_27319_tmp.db: unable to flush: No such file or directory
@test_27319_tmp.db: unable to flush: No such file or directory

This is printed before the error tracebacks, but I assume it actually
happens afterwards (which is entirely possible if stderr is line-buffered
and stdout is not).

--
components: Tests
keywords: buildbot
messages: 154215
nosy: nadeem.vawda
priority: normal
severity: normal
stage: needs patch
status: open
title: ARM Ubuntu 3.x buildbot failing test_dbm
type: behavior
versions: Python 3.3

___
Python tracker 

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



[issue14076] sqlite3 module ignores placeholers in CREATE TRIGGER code

2012-02-25 Thread Christian Schilling

Changes by Christian Schilling :


--
nosy: +Christian.Schilling

___
Python tracker 

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



[issue14102] argparse: add ability to create a man page

2012-02-25 Thread Éric Araujo

Éric Araujo  added the comment:

Andi, the author of the blog post, will work on a patch.

--

___
Python tracker 

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



[issue14121] add a convenience C-API function for unpacking iterables

2012-02-25 Thread Stefan Behnel

New submission from Stefan Behnel :

In the context of better interfacing of PyPy with Cython, it appears that 
simple looking things like PyTuple_GET_ITEM() are often rather involved in 
PyPy's C-API implementation. However, since functions/macros like these are 
used very frequently, this has an effect on the achievable performance.

It occurred to me that there are cases that involve many C-API calls where the 
intention is simply to unpack a sequence (or iterable) of known length, often 
just 2 or 3 items. Argument unpacking is one such situation (for which there 
are appropriate C-API functions), dict item iteration or iteration over 
enumerate() are other well known cases (at least in Python space). As the one 
obvious way to handle the general use case, I propose the following addition of 
a convenience function to the C-API:

int PyIter_Unpack(PyObject* iterable, Py_ssize_t min_unpack, Py_ssize_t 
max_unpack, ...)

As indicated by the names, it's meant to unpack any iterable or iterator, 
really, i.e. it would fall back to iteration if the iterable is neither a tuple 
nor list, for which special handling code makes the most sense. I thought about 
naming it PySequence_Unpack(), but that would imply that it should reject 
unordered (or, for safety, any unknown) iterables and non-sequence iterator as 
input, which IMHO would complicate matters more than it would help. A warning 
about unordered iterables in the documentation should be enough. I would expect 
that most users would actually know the type of sequence that they are 
processing.

The "max_unpack" parameter gives the number of varargs that follows, which are 
all either of type PyObject** or NULL, the latter indicating that the value is 
not of interest. Non-NULL pointers will receive a new reference to the item at 
the corresponding index.

The "min_unpack" parameter is made available for error checking. If less items 
are found in the iterable, the function sets a ValueError and returns -1. 
Assignments may or may not have taken place at this point, but no owned 
references are passed back in this case. If, on successful unpacking, the 
number of unpacked items is smaller than "max_unpack", all remaining item 
pointers will be set to NULL. Users who do not care about the number of items 
would pass 0 and those who know the exact length would pass that as both 
"min_unpack" and "max_unpack".

There is one case I'm not sure about yet, and that's how to handle the case of 
finding more items than "max_unpack" requests. I think it's just as convenient 
in some cases to automatically raise an exception, as it is in other cases to 
just ignore them. I think a way to solve this could be to not raise an 
exception, but to return 0 when all items were processed and 1 when there are 
remaining items. In this case, users who care could check the result and if 
they consider left-over items an error, clean up the returned references and 
raise an error manually. Alternatively, the function could return the number of 
unpacked items, but that may involve more work on the user side in order to 
find out what needs to be done. The drawback of a tristate return with and 
without errors set is that the straight forward "if (PyIter_Unpack(...))" check 
is no longer enough to correctly detect and propagate errors. Also, when 
passing an iterator, the function would have to eat one more value in order 
 to determine the return code. That may not be what the caller wants.

Maybe an additional flag parameter ("check_size") could solve this. If true, 
the function will check the size of sequences and report longer sequences as 
errors, and for iterators, will unpack the next item and report it as error if 
available. If false, additional values will be ignored for sequences and no 
attempt will be made for iterators to unpack more items than requested.


Because of the questions above, and because this addition involves a certain 
redundancy with what's there already (namely the argument and tuple unpacking 
functions which do not work on lists or arbitrary iterables and/or raise the 
wrong exceptions), I'm asking for comments before writing up a patch. Any 
thoughts on this?

--
components: Interpreter Core
messages: 154217
nosy: scoder
priority: normal
severity: normal
status: open
title: add a convenience C-API function for unpacking iterables
type: enhancement
versions: Python 3.3

___
Python tracker 

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



[issue14112] tutorial intro talks of "shallow copy" concept without explanation

2012-02-25 Thread Tim Golden

Tim Golden  added the comment:

On 25/02/2012 08:09, Ezio Melotti wrote:
> Even if they know the meaning of "shallow" (which is not a really common word 
> AFAICT)

FWIW it's pretty much the only way of saying what it means.
I've no idea how many people used it last year or anything,
but if I needed to express the concept of the opposite of
deep I would struggle to find another word. Except, perhaps,
the doublespeak-like "not deep". Undeep? Double-plus undeep?

--
nosy: +tim.golden

___
Python tracker 

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



[issue14112] tutorial intro talks of "shallow copy" concept without explanation

2012-02-25 Thread Ramchandra Apte

Ramchandra Apte  added the comment:

+1 for Éric Araujo's idea.

--
nosy: +ramchandra.apte

___
Python tracker 

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



[issue14114] 2.7.3rc1 chm gives JS error

2012-02-25 Thread Martin v . Löwis

Martin v. Löwis  added the comment:

For the record: this issue blocks 3.2 as well.

--
nosy: +benjamin.peterson, georg.brandl
versions: +Python 2.7, Python 3.2

___
Python tracker 

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



[issue14081] Allow "maxsplit" argument to str.split() to be passed as a keyword argument

2012-02-25 Thread Nick Coghlan

Nick Coghlan  added the comment:

Ezio: I spotted an extraneous '[' hanging around in the updated doc signature 
for split, but otherwise looked fine.

Éric: you're probably right, but I was sending them a note to suggest a simpler 
alternative, only to discover that the obvious approach of "maxsplit=1" didn't 
actually work.

--

___
Python tracker 

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



[issue14114] 2.7.3rc1 chm gives JS error

2012-02-25 Thread Ezio Melotti

Ezio Melotti  added the comment:

18bbfed9aafa is the changeset that introduced the copy button.
Maybe the part in the "extrahead" (in Doc/tools/sphinxext/layout.html, see also 
first chunk of the diff) block shouldn't be included in chm?

--

___
Python tracker 

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



[issue14082] shutil doesn't copy extended attributes

2012-02-25 Thread Hynek Schlawack

Hynek Schlawack  added the comment:

I'd tend to always copy xattrs – it seems that's what the user would expect to 
happen. A new parameter to _forbid_ it might make sense. However, I feel that 
there are already enough parameters in place. :-/

--

___
Python tracker 

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



[issue14116] Lock.__enter__() method returns True instead of self

2012-02-25 Thread Nick Coghlan

Nick Coghlan  added the comment:

"with Lock() as lock:" doesn't make any sense - you need to share the lock with 
other threads or code for it be useful, which means you can't create it inline 
in the with statement header. Instead, you have to store it somewhere else 
(usually as a closure reference or a module, class or instance attribute) and 
then merely use it in the with statement to acquire and release it 
appropriately.

Absent a compelling use case, I'm inclined to reject this one - when there's no 
specifically useful value to return from __enter__, None, True or False are all 
reasonable alternatives.

--

___
Python tracker 

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



[issue14112] tutorial intro talks of "shallow copy" concept without explanation

2012-02-25 Thread Tshepang Lekhonkhobe

Tshepang Lekhonkhobe  added the comment:

On Sat, Feb 25, 2012 at 09:35, Éric Araujo  wrote:
> What about this:
>
>  All slice operations return a new list containing the requested elements.  
> This
> -means that the following slice returns a shallow copy of the list *a*::
> +means that the following slice returns a shallow copy (see the documentation 
> of
> +the :mod:`copy` module for a definition) of the list *a*::

That's kool, though I like Ezio's idea of putting it on a glossary
(and then linking to it) even more.

> (BTW, you can use syntax like Doc/tutorial/introduction.rst:487 to have links 
> generated; see 
> http://docs.python.org/devguide/triaging#generating-special-links-in-a-comment
>  —also linked from the “Comment” label in the form, but it isn’t obvious that 
> it’s a link).

Thanks for the pointer. Note however that I chose my approach because
it shows the info at a specific revision, in case the content changes
later on.

--

___
Python tracker 

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



[issue14112] tutorial intro talks of "shallow copy" concept without explanation

2012-02-25 Thread Ezio Melotti

Ezio Melotti  added the comment:

> FWIW it's pretty much the only way of saying what it means.

However, even using "not deep" here would still be ambiguous.  What's a deep 
copy?  What's a non-deep copy?
Using "shallow" might be a problem, but the real problem is that regardless of 
the wording, the reader might not know what this is all about.

--

___
Python tracker 

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



[issue14114] 2.7.3rc1 chm gives JS error

2012-02-25 Thread Ezio Melotti

Ezio Melotti  added the comment:

JQuery and the other scripts (like sidebar.js) are part of Sphinx, whereas the 
copy button is something that was added to our instances only, by changing the 
template.

--

___
Python tracker 

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



[issue1589] New SSL module doesn't seem to verify hostname against commonName in certificate

2012-02-25 Thread Thomas Leonard

Thomas Leonard  added the comment:

Just to add a couple of data points to argue in favour of a secure-by-default 
behaviour:

0install.net:

http://secunia.com/advisories/47935 (spoofing attack due to certificate names 
not being validated)

Mozilla is recommending people avoid using Python's built-in SSL:

https://github.com/mozilla/browserid/wiki/Security-Considerations-when-Implementing-BrowserID

I find it hard to believe that anyone would be able to write an SSL client in 
Python currently without introducing some vulnerability. There are too many 
traps to fall into. Here are the three I know about:

1. Not specifying any trusted CAs means trust everyone (where for most software 
it would mean either trust no-one or trust only well-known CAs).

2. Specifiying a single trusted CA means also trust all known CAs (on MacOS X 
at least).

3. Unless you manually enable hostname checking, the attacker only needs a 
valid SSL certificate for their own site, not for the site they're spoofing.

--
nosy: +Thomas.Leonard

___
Python tracker 

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



[issue14082] shutil doesn't copy extended attributes

2012-02-25 Thread Charles-François Natali

Charles-François Natali  added the comment:

I'm also in favor of adding extended attributes to copy2:
"""
Similar to shutil.copy(), but metadata is copied as well 
"""

extended attributes are metadata. And there are already too many copy 
functions...

--

___
Python tracker 

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



[issue14082] shutil doesn't copy extended attributes

2012-02-25 Thread Hynek Schlawack

Hynek Schlawack  added the comment:

If nobody objects, I'd cook up a patch.

--

___
Python tracker 

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



[issue14118] _pickle.c structure cleanup

2012-02-25 Thread Martin v . Löwis

Martin v. Löwis  added the comment:

I'm -1 on splitting the file. This is C, splitting it up will make it *harder* 
to understand, as you have to search across multiple files to find anything.

If you want to make it more readable, I propose that you
a) put a comment in the top explaining how the file is structured
b) put

/*/

  section headers between logical groups of functions.

--
nosy: +loewis

___
Python tracker 

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



[issue14046] argparse: assertion failure if optional argument has square/round brackets in metavar

2012-02-25 Thread Steven Bethard

Steven Bethard  added the comment:

Yes, this is a known bug (Issue 11874). Patches welcome.

--
resolution:  -> duplicate
superseder:  -> argparse assertion failure with brackets in metavars

___
Python tracker 

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



[issue6884] Impossible to include file in sdist that starts with 'build' on Win32

2012-02-25 Thread Georg Brandl

Georg Brandl  added the comment:

Be sure to notify me when you have committed a fix, as changes made now in the 
3.2 branch will *not* show up in the final release.

--

___
Python tracker 

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



[issue14074] argparse allows nargs>1 for positional arguments but doesn't allow metavar to be a tuple

2012-02-25 Thread Steven Bethard

Steven Bethard  added the comment:

Looks like the problem is that "_format_action_invocation" is not being as 
careful with the different possibilities for metavar as "_format_args" is.  
Patches welcome!

--

___
Python tracker 

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



[issue14118] _pickle.c structure cleanup

2012-02-25 Thread Merlijn van Deen

Merlijn van Deen  added the comment:

See https://bitbucket.org/valhallasw/cpython/src/ee0d2beaf6a4/Modules/_pickle.c 
for a rough structure overview - which maybe also explains why I thought 
restructuring made sense in the first place.

However, I'm not the person who has to maintain the module. If you're not 
interested, I'll just split out the module until I feel comfortable editing 
stuff, make my patch for #6784 and backport it to the 6500-line version.

--

___
Python tracker 

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



[issue7433] MemoryView memory_getbuf causes segfaults, double call to tp_releasebuffer

2012-02-25 Thread Roundup Robot

Roundup Robot  added the comment:

New changeset 3f9b3b6f7ff0 by Stefan Krah in branch 'default':
- Issue #10181: New memoryview implementation fixes multiple ownership
http://hg.python.org/cpython/rev/3f9b3b6f7ff0

--
nosy: +python-dev

___
Python tracker 

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



[issue5231] Change format of a memoryview

2012-02-25 Thread Roundup Robot

Roundup Robot  added the comment:

New changeset 3f9b3b6f7ff0 by Stefan Krah in branch 'default':
- Issue #10181: New memoryview implementation fixes multiple ownership
http://hg.python.org/cpython/rev/3f9b3b6f7ff0

--
nosy: +python-dev

___
Python tracker 

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



[issue10181] Problems with Py_buffer management in memoryobject.c (and elsewhere?)

2012-02-25 Thread Roundup Robot

Roundup Robot  added the comment:

New changeset 3f9b3b6f7ff0 by Stefan Krah in branch 'default':
- Issue #10181: New memoryview implementation fixes multiple ownership
http://hg.python.org/cpython/rev/3f9b3b6f7ff0

--
nosy: +python-dev

___
Python tracker 

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



[issue12834] PyBuffer_ToContiguous() incorrect for non-contiguous arrays

2012-02-25 Thread Roundup Robot

Roundup Robot  added the comment:

New changeset 3f9b3b6f7ff0 by Stefan Krah in branch 'default':
- Issue #10181: New memoryview implementation fixes multiple ownership
http://hg.python.org/cpython/rev/3f9b3b6f7ff0

--
nosy: +python-dev

___
Python tracker 

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



[issue8305] memoview[0] creates an invalid view if ndim != 1

2012-02-25 Thread Roundup Robot

Roundup Robot  added the comment:

New changeset 3f9b3b6f7ff0 by Stefan Krah in branch 'default':
- Issue #10181: New memoryview implementation fixes multiple ownership
http://hg.python.org/cpython/rev/3f9b3b6f7ff0

--
nosy: +python-dev

___
Python tracker 

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



[issue9990] PyMemoryView_FromObject alters the Py_buffer after calling PyObject_GetBuffer when ndim 1

2012-02-25 Thread Roundup Robot

Roundup Robot  added the comment:

New changeset 3f9b3b6f7ff0 by Stefan Krah in branch 'default':
- Issue #10181: New memoryview implementation fixes multiple ownership
http://hg.python.org/cpython/rev/3f9b3b6f7ff0

--
nosy: +python-dev

___
Python tracker 

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



[issue6085] Logging in BaseHTTPServer.BaseHTTPRequestHandler causes lag

2012-02-25 Thread Charles-François Natali

Charles-François Natali  added the comment:

> +1 to Antoine’s proposal of removal.

Agreed.

Here's a patch.

Do note, however, that it's a behavior change: the address_string()
method is documented to return a resolved hostname (when possible).

--
keywords: +needs review
stage: test needed -> patch review
Added file: http://bugs.python.org/file24637/server_resolve.diff

___
Python tracker 

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



[issue14120] ARM Ubuntu 3.x buildbot failing test_dbm

2012-02-25 Thread Nadeem Vawda

Nadeem Vawda  added the comment:

Also failing on 3.2:

http://www.python.org/dev/buildbot/all/builders/ARM%20Ubuntu%203.2/builds/227/steps/test/logs/stdio

--
versions: +Python 3.2

___
Python tracker 

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



[issue13491] Fixes for sqlite3 doc

2012-02-25 Thread Petri Lehtinen

Petri Lehtinen  added the comment:

Both patches look good to me. The text_factory example is OK on 2.7 because the 
OptimizedUnicode flag works correctly there.

--

___
Python tracker 

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



[issue12801] C realpath not used by os.path.realpath

2012-02-25 Thread Charles-François Natali

Charles-François Natali  added the comment:

>  - os.realpath() uses canonicalize_file_name() if available, or use 
> realpath() with a buffer of MAXPATHLEN bytes

MAXPATHLEN is not necessarily defined (e.g. on the Hurd): if it's not
defined, it is set either to MAX_PATH (if it's defined an greater than
1024), or arbitrarily to 1024.
Thus, we can't pass it to realpath(3), since we don't know for sure
that realpath(3) won't return a string greater than MAXPATHLEN, which
could result in a buffer overflow.

Also, there's the problem that realpath(3) returns ENOENT, whereas
os.path.realpath() doesn't:

without patch:
$ python -c "import os; os.path.realpath('/nosuchfile')"

with patch:
$ ./python -c "import os; os.path.realpath('/nosuchfile')"
Traceback (most recent call last):
  File "", line 1, in 
  File "/home/cf/python/cpython/Lib/posixpath.py", line 385, in realpath
return os.realpath(filename)
FileNotFoundError: [Errno 2] No such file or directory: '/nosuchfile'

--

___
Python tracker 

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



[issue14113] Failure in test_strptime on Windows

2012-02-25 Thread Nadeem Vawda

Nadeem Vawda  added the comment:

Also failing on the Windows 7 bot:

http://www.python.org/dev/buildbot/all/builders/x86%20Windows7%203.x/builds/4453/steps/test/logs/stdio

--

___
Python tracker 

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



[issue14113] Failure in test_strptime on Windows

2012-02-25 Thread Roundup Robot

Roundup Robot  added the comment:

New changeset d5aa731bae5e by Nadeem Vawda in branch 'default':
Use assertEqual in test_strptime for better failure messages (cf. issue #14113).
http://hg.python.org/cpython/rev/d5aa731bae5e

--
nosy: +python-dev

___
Python tracker 

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



[issue14080] Sporadic test_imp failure

2012-02-25 Thread Stefan Krah

Stefan Krah  added the comment:

I've trouble debugging this: Is the new version of importlib already
being used? I'm stepping through the offending pep3147 import, which
should correspond to this line in test_dot.py:

   m = __import__('pep3147')
   

Until here Python/import.c is used:

find_module_path (fullname='pep3147', name='pep3147', path='.', 
path_hooks=[], 
path_importer_cache=
{'/home/stefan/pydev/cpython-imp/Lib/collections': None, '': None, 
'/home/stefan/pydev/cpython-imp/Lib/unittest': None, 
'/home/stefan/pydev/cpython-imp/Lib/multiprocessing': None, 
'/usr/local/lib/python3.3/site-packages/setuptools-0.6c11-py3.3.egg-info': 
, 
'/home/stefan/pydev/cpython-imp/Lib/importlib': None, 
'/home/stefan/pydev/cpython-imp/Lib/plat-linux': <_FileFinder(_cache_refresh=0, 
_path_cache={'TYPES.py', 'regen', 'DLFCN.py', 'IN.py', 'CDROM.py'}, 
path='/home/stefan/pydev/cpython-imp/Lib/plat-linux', _path_mtime=, modules=[('.cpython-33m.so', ), 
('.abi3.so', ), ('.so', ), 
('.py', ), ('.pyc', )], 
packages=[('.py', ), ('.pyc', )], _relaxed_path_cache={'types.py', 'regen', 'dlfcn.py', 'in.py', 
'cdrom.py'}) at remote 0x737d8b50>, '.'
 : <_FileFinder(_cache_refresh=...(truncated), p_path=0x7fff7f80, 
p_loader=0x7fff80b8, p_fd=0x7fff7f98) at Python/import.c:1811
1811if (importer == NULL) {
(gdb) n
1816if (importer != Py_None) {
(gdb) n
1818loader = _PyObject_CallMethodId(importer,



After descending further and further into the eval loop, all
of a sudden this appears:


3426retval = PyEval_EvalFrameEx(f,0);
(gdb) s
PyEval_EvalFrameEx (f=
Frame 0xafec40, for file 
/home/stefan/pydev/cpython-imp/Lib/importlib/_bootstrap.py, line 773, in 
find_module (self=<_FileFinder(_cache_refresh=0, _path_cache=set(), path='.', 
_path_mtime=, modules=[('.cpython-33m.so', ), ('.abi3.so', ), ('.so', ), ('.py', ), ('.pyc', )], packages=[('.py', ), ('.pyc', )], _relaxed_path_cache=set()) at remote 0x74128390>, 
fullname='pep3147'), 
throwflag=0) at Python/ceval.c:808


Is this expected?

--

___
Python tracker 

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



[issue14080] Sporadic test_imp failure

2012-02-25 Thread Nadeem Vawda

Changes by Nadeem Vawda :


--
nosy: +nadeem.vawda

___
Python tracker 

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



[issue14082] shutil doesn't copy extended attributes

2012-02-25 Thread Éric Araujo

Éric Araujo  added the comment:

Sounds good.

--

___
Python tracker 

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



[issue13491] Fixes for sqlite3 doc

2012-02-25 Thread Éric Araujo

Changes by Éric Araujo :


--
assignee: docs@python -> petri.lehtinen

___
Python tracker 

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



[issue6884] Impossible to include file in sdist that starts with 'build' on Win32

2012-02-25 Thread Éric Araujo

Éric Araujo  added the comment:

I’ll commit the 2.7 patch, see how buildbots fare, then commit the 3.2 version 
(also attached in case someone wants to test it).

--
Added file: http://bugs.python.org/file24638/filelist-regex-bugs-3.2.diff

___
Python tracker 

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



[issue14080] Sporadic test_imp failure

2012-02-25 Thread Antoine Pitrou

Antoine Pitrou  added the comment:

> Is this expected?

Looks rather strange to me, it means we have importlib importers on
sys.path_importer_cache. Still, the fact that path == '.' above (in
_FileFinder as well as find_module_path) makes it difficult to
understand the later failure.

--

___
Python tracker 

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



[issue14112] tutorial intro talks of "shallow copy" concept without explanation

2012-02-25 Thread Éric Araujo

Éric Araujo  added the comment:

A link to a glossary may be better than a link to the top of the copy module.  
I wonder if we could use glossary markup in the copy module docs instead of 
adding terms to the global glossary.

Tim: I like “undeep”, it’s used to describe a geographical feature of a river 
in The Lord of the Rings.

--

___
Python tracker 

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



[issue6884] Impossible to include file in sdist that starts with 'build' on Win32

2012-02-25 Thread Nadeem Vawda

Nadeem Vawda  added the comment:

3.2 patch looks good on Windows.

--

___
Python tracker 

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



[issue9691] sdist includes files that are not in MANIFEST.in

2012-02-25 Thread Roundup Robot

Roundup Robot  added the comment:

New changeset 47788c90f80b by Éric Araujo in branch '2.7':
Fix long-standing bugs with MANIFEST.in parsing on Windows (#6884).
http://hg.python.org/cpython/rev/47788c90f80b

--
nosy: +python-dev

___
Python tracker 

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



[issue13193] packaging.tests.test_manifest and distutils.tests.test_filelist failures

2012-02-25 Thread Roundup Robot

Roundup Robot  added the comment:

New changeset 47788c90f80b by Éric Araujo in branch '2.7':
Fix long-standing bugs with MANIFEST.in parsing on Windows (#6884).
http://hg.python.org/cpython/rev/47788c90f80b

--

___
Python tracker 

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



[issue14004] Distutils filelist selects too many files on Windows

2012-02-25 Thread Roundup Robot

Roundup Robot  added the comment:

New changeset 47788c90f80b by Éric Araujo in branch '2.7':
Fix long-standing bugs with MANIFEST.in parsing on Windows (#6884).
http://hg.python.org/cpython/rev/47788c90f80b

--
nosy: +python-dev

___
Python tracker 

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



[issue6884] Impossible to include file in sdist that starts with 'build' on Win32

2012-02-25 Thread Roundup Robot

Roundup Robot  added the comment:

New changeset 47788c90f80b by Éric Araujo in branch '2.7':
Fix long-standing bugs with MANIFEST.in parsing on Windows (#6884).
http://hg.python.org/cpython/rev/47788c90f80b

--
nosy: +python-dev

___
Python tracker 

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



[issue13447] Add tests for some scripts in Tools/scripts

2012-02-25 Thread Roundup Robot

Roundup Robot  added the comment:

New changeset 020364d3e359 by Éric Araujo in branch '2.7':
Add test file for scripts in Tools (#13447).
http://hg.python.org/cpython/rev/020364d3e359

--
nosy: +python-dev

___
Python tracker 

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



[issue14004] Distutils filelist selects too many files on Windows

2012-02-25 Thread Roundup Robot

Roundup Robot  added the comment:

New changeset 73aa4c9305b3 by Éric Araujo in branch '3.2':
Fix long-standing bugs with MANIFEST.in parsing on Windows (#6884).
http://hg.python.org/cpython/rev/73aa4c9305b3

--

___
Python tracker 

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



[issue6884] Impossible to include file in sdist that starts with 'build' on Win32

2012-02-25 Thread Roundup Robot

Roundup Robot  added the comment:

New changeset 73aa4c9305b3 by Éric Araujo in branch '3.2':
Fix long-standing bugs with MANIFEST.in parsing on Windows (#6884).
http://hg.python.org/cpython/rev/73aa4c9305b3

--

___
Python tracker 

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



[issue9691] sdist includes files that are not in MANIFEST.in

2012-02-25 Thread Roundup Robot

Roundup Robot  added the comment:

New changeset 73aa4c9305b3 by Éric Araujo in branch '3.2':
Fix long-standing bugs with MANIFEST.in parsing on Windows (#6884).
http://hg.python.org/cpython/rev/73aa4c9305b3

--

___
Python tracker 

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



[issue13193] packaging.tests.test_manifest and distutils.tests.test_filelist failures

2012-02-25 Thread Roundup Robot

Roundup Robot  added the comment:

New changeset 73aa4c9305b3 by Éric Araujo in branch '3.2':
Fix long-standing bugs with MANIFEST.in parsing on Windows (#6884).
http://hg.python.org/cpython/rev/73aa4c9305b3

--

___
Python tracker 

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



[issue13637] binascii.a2b_* functions could accept unicode strings

2012-02-25 Thread R. David Murray

R. David Murray  added the comment:

Discussion resolved in favor of patch.

--
status: open -> closed

___
Python tracker 

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



[issue13641] decoding functions in the base64 module could accept unicode strings

2012-02-25 Thread R. David Murray

Changes by R. David Murray :


--
status: open -> closed

___
Python tracker 

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



[issue1531415] parsetok.c emits warnings by writing to stderr

2012-02-25 Thread Éric Araujo

Changes by Éric Araujo :


--
versions:  -Python 3.2

___
Python tracker 

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



[issue13447] Add tests for some scripts in Tools/scripts

2012-02-25 Thread Éric Araujo

Éric Araujo  added the comment:

Hi Mark.  You’re the author of Tools/parser/test_unparse.py; any objection if I 
move it to the new Lib/test/test_tools.py file, so that all tests for Tools are 
in one place?

--
nosy: +mark.dickinson

___
Python tracker 

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



[issue14080] Sporadic test_imp failure

2012-02-25 Thread Stefan Krah

Stefan Krah  added the comment:

OK, I stepped in parallel through the non-failing and the failing
versions. The first divergence is when _path_mtime is compared.
The comparison in the good version returns 'False', the one in the
bad version returns 'True':

Good version (Fedora 16):

(gdb) p w
$148 = '_path_mtime'
(gdb) n
2278x = PyObject_GetAttr(v, w);
(gdb) p v
$149 = 
<_FileFinder(_relaxed_path_cache=set(), path='.', packages=[('.py', ), ('.pyc', )], _path_cache=set(), 
_cache_refresh=0, _path_mtime=, 
modules=[('.cpython-33m.so', ), ('.abi3.so', ), ('.so', ), ('.py', ), ('.pyc', )]) at remote 0x74128310>
(gdb) n
2279Py_DECREF(v);
(gdb) p x
$150 = 
(gdb) n
2280SET_TOP(x);
(gdb) n
2281if (x != NULL) DISPATCH();
(gdb) n
2284TARGET(COMPARE_OP)
(gdb) n
2285w = POP();
(gdb) n
2286v = TOP();
(gdb) n
2287x = cmp_outcome(oparg, v, w);
(gdb) n
2288Py_DECREF(v);
(gdb) p w
$151 = 
(gdb) p v
$152 = 
(gdb) p x
$153 = False
(gdb) 


Bad version (Ubuntu):

(gdb) p w
$154 = '_path_mtime'
(gdb) p v
$155 = 
(gdb) n
2278x = PyObject_GetAttr(v, w);
(gdb) p v
$156 = 
<_FileFinder(_relaxed_path_cache=set(), packages=[('.py', ), ('.p
yc', )], _path_mtime=, 
modules=[('.cpython-
33m.so', ), ('.abi3.so', ), 
('.so', ), ('.py', ), ('.pyc', )], 
path='.', _cache_refresh=0, _path_cache=set()) at remote 0x72b7f6d0>
(gdb) n
2279Py_DECREF(v);
(gdb) p x
$157 = 
(gdb) n
2280SET_TOP(x);
(gdb) n
2281if (x != NULL) DISPATCH();
(gdb) n
2284TARGET(COMPARE_OP)
(gdb) n
2285w = POP();
(gdb) n
2286v = TOP();
(gdb) n
2287x = cmp_outcome(oparg, v, w);
(gdb) n
2288Py_DECREF(v);
(gdb) p w
$158 = 
(gdb) p v
$159 = 
(gdb) p x
$160 = True

--

___
Python tracker 

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



[issue14080] Sporadic test_imp failure

2012-02-25 Thread Stefan Krah

Stefan Krah  added the comment:

So I think the good version proceeds to refresh the _path_cache:

<_FileFinder(_relaxed_path_cache={'pep3147'}, packages=[('.py', ), ('.pyc', )], _path_mtime=, modules=[('.cpython-33m.so', ), 
('.abi3.so', ), ('.so', ), 
('.py', ), ('.pyc', )], 
path='.', _cache_refresh=0, _path_cache={'pep3147'}) at remote 0x72b7f6d0>


And the bad version still contains an empty set:
  
<_FileFinder(_relaxed_path_cache=set(), path='.', packages=[('.py', ), ('.pyc', )], _path_cache=set(), 
_cache_refresh=0, _path_mtime=, 
modules=[('.cpython-33m.so', ), ('.abi3.so', ), ('.so', ), ('.py', ), ('.pyc', )]) at remote 0x74128310>



This probably means that the case split occurs in  
Lib/importlib/_bootstrap.py:780 ...


if mtime != self._path_mtime or _cache_refresh != self._cache_refresh:
self._fill_cache()
self._path_mtime = mtime
self._cache_refresh = _cache_refresh

--

___
Python tracker 

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



[issue14075] argparse: unused method?

2012-02-25 Thread Éric Araujo

Éric Araujo  added the comment:

Hm, I was about to commit this and did a double-take.  Are we 100% sure that 
nobody in the whole wide world of Python does not override this function?  
Shouldn’t we first send a DeprecationWarning in 3.3 and remove later?

--

___
Python tracker 

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



[issue6884] Impossible to include file in sdist that starts with 'build' on Win32

2012-02-25 Thread Éric Araujo

Éric Araujo  added the comment:

Most buildbots are green, one has a test_subprocess failure, others are still 
building, but I have to go soon.  Release managers, please graft this fix when 
the bots are all green.

--
resolution:  -> fixed
stage: patch review -> committed/rejected

___
Python tracker 

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



[issue3445] Ignore missing attributes in functools.update_wrapper

2012-02-25 Thread Yaniv Aknin

Yaniv Aknin  added the comment:

Shouldn't this be fixed in 2.7 as well?

It's a bug, it's in the wild, and it's causing people to do ugly (and maybe not 
100% reliable) things like 
https://code.djangoproject.com/browser/django/trunk/django/utils/decorators.py#L68

--
nosy: +Yaniv.Aknin

___
Python tracker 

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



[issue3445] Ignore missing attributes in functools.update_wrapper

2012-02-25 Thread Éric Araujo

Changes by Éric Araujo :


--
nosy: +eric.araujo

___
Python tracker 

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



[issue14080] Sporadic test_imp failure

2012-02-25 Thread Stefan Krah

Stefan Krah  added the comment:

And indeed, with this patch ...

diff --git a/Lib/importlib/_bootstrap.py b/Lib/importlib/_bootstrap.py
--- a/Lib/importlib/_bootstrap.py
+++ b/Lib/importlib/_bootstrap.py
@@ -777,6 +777,8 @@
 mtime = _os.stat(self.path).st_mtime
 except OSError:
 mtime = -1
+if fullname == 'pep3147':
+print(mtime, self._path_mtime)
 if mtime != self._path_mtime or _cache_refresh != self._cache_refresh:
 self._fill_cache()
 self._path_mtime = mtime


Good version:

[stefan@fedora-16-amd64 cpython]$ ./python -m test test_dot
[1/1] test_dot
1330188705.3170006 1330188705.314
Warning -- sys.path was modified by test_dot


Bad version:

$ ./python -m test test_dot
[1/1] test_dot
1330188676.0 1330188676.0
Warning -- sys.path was modified by test_dot
test test_dot crashed -- Traceback (most recent call last):
  File "/home/stefan/pydev/cpython-imp/Lib/test/regrtest.py", line 1214, in 
runtest_inner
the_package = __import__(abstest, globals(), locals(), [])
  File "/home/stefan/pydev/cpython-imp/Lib/test/test_dot.py", line 20, in 

assert m.__file__ == expected___file__
AssertionError

1 test failed:
test_dot



It looks like os.stat.st_mtime returns full seconds here on Ubuntu:

Python 3.3.0a0 (default:3f9b3b6f7ff0, Feb 25 2012, 17:42:23) 
[GCC 4.4.3] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import os
>>> os.stat("./python").st_mtime
1330189060.0
>>> 










Nanoseconds, anyone? :)

--

___
Python tracker 

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



[issue14080] Sporadic test_imp failure

2012-02-25 Thread Antoine Pitrou

Antoine Pitrou  added the comment:

Le samedi 25 février 2012 à 16:59 +, Stefan Krah a écrit :
> Bad version:
> 
> $ ./python -m test test_dot
> [1/1] test_dot
> 1330188676.0 1330188676.0

This might be what triggers the issue, but it's not the cause. Even with
a bad mtime, the __file__ should still be the right one, so there must
be something else.

> It looks like os.stat.st_mtime returns full seconds here on Ubuntu:

It probably depends on the filesystem.

--

___
Python tracker 

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



[issue14080] Sporadic test_imp failure

2012-02-25 Thread Stefan Krah

Stefan Krah  added the comment:

Antoine Pitrou  wrote:
> This might be what triggers the issue, but it's not the cause. Even with
> a bad mtime, the __file__ should still be the right one, so there must
> be something else.

It definitely triggers the issue because the problem also occurs on Fedora
if I use:

if fullname != 'pep3147' and (mtime != self._path_mtime or _cache_refresh 
!= self._cache_refresh):
...

But you're right, the real problem should be elsewhere. At least with
the line above everyone should be able to reproduce the issue.

--

___
Python tracker 

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



[issue14114] 2.7.3rc1 chm gives JS error

2012-02-25 Thread Roundup Robot

Roundup Robot  added the comment:

New changeset 64b9ff3c91bb by Ezio Melotti in branch '2.7':
#14114: don't include copybutton.js in the htmlhelp output.
http://hg.python.org/cpython/rev/64b9ff3c91bb

New changeset 7f651187b25c by Ezio Melotti in branch '3.2':
#14114: don't include copybutton.js in the htmlhelp output.
http://hg.python.org/cpython/rev/7f651187b25c

New changeset 39ddcc5c7fb9 by Ezio Melotti in branch 'default':
#14114: merge with 3.2.
http://hg.python.org/cpython/rev/39ddcc5c7fb9

--
nosy: +python-dev

___
Python tracker 

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



[issue14114] 2.7.3rc1 chm gives JS error

2012-02-25 Thread Ezio Melotti

Ezio Melotti  added the comment:

Should be fixed now, let me know if it works.
(Thanks Georg for the help.)

--
stage:  -> committed/rejected
type:  -> behavior

___
Python tracker 

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



[issue14122] operator: div() instead of truediv() in documention since 3.1.2

2012-02-25 Thread Félix-Antoine Fortin

New submission from Félix-Antoine Fortin :

A small regression was introduced by the changeset 57209 in the documentation 
of the operator module.

In the abstract operations table in section 9.3.1 the first of two lines
on Division, the associated function should be : truediv() instead of div(), 
since operator.div() no longer exists in Python 3. In changeset 57209, this 
corresponds to the line 408 of the file Doc/library/operator.rst.

I have included the patch for the changeset 57209.

--
assignee: docs@python
components: Documentation
files: operator_div.patch
keywords: patch
messages: 154277
nosy: docs@python, felixantoinefortin
priority: normal
severity: normal
status: open
title: operator: div() instead of truediv() in documention since 3.1.2
versions: Python 3.1, Python 3.2, Python 3.3, Python 3.4
Added file: http://bugs.python.org/file24639/operator_div.patch

___
Python tracker 

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



[issue14122] operator: div() instead of truediv() in documention since 3.1.2

2012-02-25 Thread Ezio Melotti

Changes by Ezio Melotti :


--
nosy: +ezio.melotti
stage:  -> patch review
type:  -> enhancement
versions:  -Python 3.1, Python 3.4

___
Python tracker 

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



[issue14118] _pickle.c structure cleanup

2012-02-25 Thread Martin v . Löwis

Martin v. Löwis  added the comment:

Thanks for the effort, but I'm rejecting this now. I'm not fundamentally 
opposed to restructing this code, but I do think that your change would be a 
slight loss of readability. If you absolutely cannot stand working with such a 
large code, please try to find supporters of a change on python-dev.

Wrt. your actual approach: this is somewhat uncoommon. If you have many files, 
you'd rather expect them to be integrated in the build process (i.e. with 
Makefile and VS project file changes) rather than recursive includes.

--
resolution:  -> rejected
status: open -> closed

___
Python tracker 

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



[issue14120] ARM Ubuntu 3.x buildbot failing test_dbm

2012-02-25 Thread Antoine Pitrou

Changes by Antoine Pitrou :


--
nosy: +barry

___
Python tracker 

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



[issue14080] Sporadic test_imp failure

2012-02-25 Thread Stefan Krah

Stefan Krah  added the comment:

What happens is that if self._fill_cache() is called, cache_module='pep3147'
and cache={'pep3147'}. Then 'cache_module in cache' is true and the function
returns:

loader('pep3147', './pep3147/__init__.py')

Otherwise, find_module() returns None, control is handed back to
find_module_path() ...

loader = _PyObject_CallMethodId(importer, &PyId_find_module, "O", fullname)

... and then to find_module_path_list(), where the search continues:

for (i = 0; i < npath; i++) {
path = PyList_GetItem(search_path_list, i);
...

search_path_list = ['.', '', ... ], but now i==1, so the dot is disregarded
and '' is used as the path, leading to the dotless result.

I don't know the contract for importlib's find_module, but it seems to me
that either find_module should handle this case or we have an unwanted
interaction between C and Python code.

--

___
Python tracker 

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



[issue13999] Queue references in multiprocessing doc points to Queue module and not to self

2012-02-25 Thread Roundup Robot

Roundup Robot  added the comment:

New changeset 5d4f2f994f75 by Sandro Tosi in branch '2.7':
Issue #13999: refer to multiprocessing.Queue when needed
http://hg.python.org/cpython/rev/5d4f2f994f75

--
nosy: +python-dev

___
Python tracker 

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



[issue13999] Queue references in multiprocessing doc points to Queue module and not to self

2012-02-25 Thread Sandro Tosi

Changes by Sandro Tosi :


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

___
Python tracker 

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



[issue14080] Sporadic test_imp failure

2012-02-25 Thread Antoine Pitrou

Antoine Pitrou  added the comment:

> What happens is that if self._fill_cache() is called, cache_module='pep3147'
> and cache={'pep3147'}. Then 'cache_module in cache' is true and the function
> returns:
> 
> loader('pep3147', './pep3147/__init__.py')
> 
> Otherwise, find_module() returns None, control is handed back to
> find_module_path() ...

Ah, thanks. So the test needs to be fixed to call
importlib.invalidate_caches() after creating the module.

--

___
Python tracker 

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



[issue6784] byte/unicode pickle incompatibilities between python2 and python3

2012-02-25 Thread Merlijn van Deen

Merlijn van Deen  added the comment:

Ok, this is my first attempt at the Pickler part of the C implementation. I'll 
have to adapt the python implementation to match this one.

All BytestrPicklerTests in test_bytestrpickle.py pass, and ./python -m test -G 
-v test_pickle passes.

Comments on style etc. are very welcome.

--
Added file: http://bugs.python.org/file24640/BytestrPickler_c.diff

___
Python tracker 

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



[issue14123] Doc change re old and new string formatting

2012-02-25 Thread Christine Jones

New submission from Christine Jones :

Change to this http://docs.python.org/py3k/library/stdtypes.html as suggested 
by Nick Coghlan here http://www.gossamer-threads.com/lists/python/dev/969817

"The formatting operations described here are modelled on C's printf()
syntax. They only support formatting of certain builtin types, and the
use of a binary operator means that care may be needed in order to
format tuples and dictionaries correctly. As the new string formatting
syntax is more powerful, flexible, extensible and handles tuples and
dictionaries naturally, it is recommended for new code. However, there
are no current plans to deprecate printf-style formatting."

--
assignee: docs@python
components: Documentation
messages: 154283
nosy: docs@python, telephonebook
priority: normal
severity: normal
status: open
title: Doc change re old and new string formatting

___
Python tracker 

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



[issue14124] _pickle.c comment/documentation improvement

2012-02-25 Thread Merlijn van Deen

New submission from Merlijn van Deen :

As suggested by loewis in msg154233, I created some documentation to help 
people get started with _pickle.c.

--
assignee: docs@python
components: Documentation, Extension Modules
files: _pickle_c_doc.diff
keywords: patch
messages: 154284
nosy: docs@python, valhallasw
priority: normal
severity: normal
status: open
title: _pickle.c comment/documentation improvement
type: enhancement
versions: Python 3.3
Added file: http://bugs.python.org/file24641/_pickle_c_doc.diff

___
Python tracker 

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



[issue14124] _pickle.c comment/documentation improvement

2012-02-25 Thread Merlijn van Deen

Changes by Merlijn van Deen :


--
nosy: +loewis

___
Python tracker 

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



[issue14123] Doc change re old and new string formatting

2012-02-25 Thread Eric V. Smith

Changes by Eric V. Smith :


--
nosy: +eric.smith

___
Python tracker 

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



[issue14123] Doc change re old and new string formatting

2012-02-25 Thread Glenn Linderman

Changes by Glenn Linderman :


--
nosy: +v+python

___
Python tracker 

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



[issue14112] tutorial intro talks of "shallow copy" concept without explanation

2012-02-25 Thread Ned Deily

Ned Deily  added the comment:

"shallow copy" and "deep copy" are both standard computer science terms by no 
means unique to or invented by Python.  We should be cautious about 
documentation bloat and trying to redefine standard terms.

http://en.wikipedia.org/wiki/Object_copy#Shallow_copy

--
nosy: +ned.deily

___
Python tracker 

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



[issue14120] ARM Ubuntu 3.x buildbot failing test_dbm

2012-02-25 Thread Barry A. Warsaw

Barry A. Warsaw  added the comment:

I was missing libgdbm-dev on that machine.  I just installed it and am running
the buildbottests by hand.  Let's see if that fixes things.

--

___
Python tracker 

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



[issue14125] Windows: failures in refleak mode

2012-02-25 Thread Stefan Krah

New submission from Stefan Krah :

The following tests fail on Windows in refleak mode. I used
24ca28cc9c9c as a reference point. Build is x64. Summary:

test_concurrent_futures, test_datetime, test_multiprocessing, test_strftime and 
test_time are leaking.



C:\Users\stefan\hg\master\PCbuild>amd64\python_d.exe -m test -uall -R :: 
test_multiprocessing  
[1/1] test_multiprocessing  
   
beginning 9 repetitions 
   
123456789   
   
.   
   
test_multiprocessing leaked [3, 3, 3, 3] references, sum=12 
   
Warning -- multiprocessing.process._dangling was modified by 
test_multiprocessing  
1 test failed:  
   
test_multiprocessing
   
[172456 refs]


C:\Users\stefan\hg\master\PCbuild>amd64\python_d.exe -m test -uall -R :: 
test_concurrent_futures   
[1/1] test_concurrent_futures   
   
beginning 9 repetitions 
   
123456789   
   
.   
   
test_concurrent_futures leaked [1524, 1518, 1524, 1524] references, sum=6090
   
1 test failed:  
   
test_concurrent_futures 
   
[170398 refs]   
   
 

C:\Users\stefan\hg\master\PCbuild>amd64\python_d.exe -m test -uall -R :: 
test_datetime 
[1/1] test_datetime 
   
beginning 9 repetitions 
   
123456789   
   
.   
   
test_datetime leaked [20, 20, 20, 20] references, sum=80
   
1 test failed:  
   
test_datetime   
   
[191763 refs]   
 



C:\Users\stefan\hg\master\PCbuild>amd64\python_d.exe -m test -uall -R :: 
test_strftime 
[1/1] test_strftime 
   
beginning 9 repetitions 
   
123456789   
   
.   
   
test_strftime leaked [2761, 2761, 2761, 2761] references, sum=11044 
   
1 test failed:  
   
test_strftime   
   
[170275 refs]   
   


C:\Users\stefan\hg\master\PCbuild>amd64\python_d.exe -m test -uall -R :: 
test_time 
[1/1] test_time 
   
beginning 9 repetitions 
   
123456789   
   
.   
   
test_time leaked [1, 1, 1, 1] references, sum=4 
   
1 test failed:  
   
test_tim

  1   2   >