[issue20184] Derby #16: Convert 50 sites to Argument Clinic across 9 files

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

Here is a patch that converts 3 functions in the _json module to Argument 
Clinic. All other functions doesn't fit with Argument Clinic.

--
Added file: http://bugs.python.org/file39292/json_clinic.patch

___
Python tracker 

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

[issue24113] shlex constructor unreachable code

[Berker Peksag]

Changes by Berker Peksag :


--
stage: needs patch -> resolved

___
Python tracker 

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

[issue24113] shlex constructor unreachable code

[Raymond Hettinger]

Changes by Raymond Hettinger :


--
priority: normal -> low
resolution:  -> fixed
status: open -> closed

___
Python tracker 

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

[issue24113] shlex constructor unreachable code

[Roundup Robot]

Roundup Robot added the comment:

New changeset 395e190ead36 by Raymond Hettinger in branch 'default':
Issue #24113:  Remove unreachable code in shlex.
https://hg.python.org/cpython/rev/395e190ead36

--
nosy: +python-dev

___
Python tracker 

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

[issue24117] Wrong range checking in GB18030 decoder.

[Ma Lin]

Ma Lin added the comment:

I found another bug in hz codec.
hz encoding uses 7-bit ASCII to represent Chinese characters, it was popular in 
USENET networks in the late 1980s and early 1990s.

I will do more check and fix them together, then I will invite you to review 
the patch.


u = 'hi~python'
b = u.encode('hz')   # bug in this step, the right sequence should be 
b"hi~~python"
print(b)# the output is b"hi~python"

u = b.decode('hz')   # so can't decode, UnicodeDecodeError raised
print(u)

--

___
Python tracker 

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

[issue24054] Invalid syntax in inspect_fodder2.py (on Python 2.x)

[David D. Riddle]

David D. Riddle added the comment:

I have made the changes you suggested. Please tell me if any further changes 
are needed.

> The unclosed file suggests a cleanup bug in linecache which we should
> fix for hygiene, but is separate :)

Should I file a bug report?

--
Added file: http://bugs.python.org/file39291/mywork.patch

___
Python tracker 

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

[issue24125] Fix for #23865 breaks docutils

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

Here is a patch that keeps error column and line after closing ExpatParser if 
error was occurred.

--
keywords: +patch
stage:  -> patch review
type:  -> behavior
versions: +Python 3.5
Added file: http://bugs.python.org/file39290/expatreader_keep_col_line.patch

___
Python tracker 

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

[issue24085] large memory overhead when pyc is recompiled

[Buck Evan]

Buck Evan added the comment:

@serhiy.storchaka This is a very stable piece of a legacy code base, so we're 
not keen to refactor it so dramatically, although we could. 

We've worked around this issue by compiling pyc files ahead of time and taking 
extra care that they're preserved through deployment. This isn't blocking our 
2.7 transition anymore.

--

___
Python tracker 

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

[issue24125] Fix for #23865 breaks docutils

[Dmitry Shachnev]

Changes by Dmitry Shachnev :


--
nosy: +larry
versions: +Python 3.4

___
Python tracker 

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

[issue24000] More fixes for the Clinic mapping of converters to format units

[Larry Hastings]

Larry Hastings added the comment:

I don't know why you're bringing up previous versions of Python.  The clinic.py 
under review here is for 3.5.

--

___
Python tracker 

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

[issue24125] Fix for #23865 breaks docutils

[Matthias Klose]

Changes by Matthias Klose :


--
nosy: +benjamin.peterson
priority: normal -> release blocker

___
Python tracker 

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

[issue24000] More fixes for the Clinic mapping of converters to format units

[Larry Hastings]

Larry Hastings added the comment:

The length attribute is an internal implementation detail, so its name is not 
relevant.  It's used in the generation of the accompanying "length" parameter 
for the impl function for this converter.  "length" is a good name for it.  
"zeroes" is not a good name for it.  "zeroes" is, however, a decent name for 
the converter parameter.

I have no quarrel with the documentation using the term NUL.  But I don't think 
the term translates well to use parameter names.

--

___
Python tracker 

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

[issue23865] Fix possible leaks in close methods

[Dmitry Shachnev]

Dmitry Shachnev added the comment:

This broke docutils, see issue #24125.

--
nosy: +mitya57

___
Python tracker 

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

[issue24125] Fix for #23865 breaks docutils

[Dmitry Shachnev]

New submission from Dmitry Shachnev:

Fix for issue #23865 (i.e. https://hg.python.org/cpython/rev/f7ddec2e9e93 in 
2.7 branch) introduced a change for xml.sax.expatreader.ExpatParser class that 
makes its close() method delete _parser object.

This breaks docutils, which handles exceptions in parse() and tries to use 
getColumnNumber() after it fails. The log is available here: 
.

I believe that particular chunk should be reverted.

--
components: Library (Lib)
messages: 242579
nosy: doko, goodger, mitya57, serhiy.storchaka
priority: normal
severity: normal
status: open
title: Fix for #23865 breaks docutils
versions: Python 2.7

___
Python tracker 

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

[issue8372] socket: Buffer overrun while reading unterminated AF_UNIX addresses

[Mark Lawrence]

Mark Lawrence added the comment:

As this is flagged as a high priority security issue shouldn't we be 
implementing needed source code changes? According to msg138224 "The patches 
look good to me".

--
nosy: +BreamoreBoy
versions: +Python 3.5

___
Python tracker 

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

[issue24000] More fixes for the Clinic mapping of converters to format units

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

In Python 2 and earlier in Python 3 there were no restrictions that arguments 
for format units "s", "y", "u", "z" and "Z" (matching Argument Clinic 
converters with length=False) shouldn't contain NUL. And this was considered as 
a bug.

As for NUL, ask Victor. It argued for naming it "null character/byte" and it 
was consistently changed almost everywhere (left some documentation, but this 
will be fixed soon).

larry.one.more.clinic.format.unit.map.cleanup.5.txt looks almost good to me, 
but I hesitate about the zeroes parameter. str_converter has the zeroes 
parameter, but the length attribute.

--

___
Python tracker 

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

[issue21069] test_fileno of test_urllibnet intermittently fails when using www.example.com

[Mark Lawrence]

Mark Lawrence added the comment:

I tried reproducing this on Windows 8.1 with the code snippet from msg216429 
but got

Traceback (most recent call last):
  File "C:\Users\Mark\Documents\MyPython\mytest.py", line 9, in 
with os.fdopen(fd, 'rb') as f:
  File "c:\python34\lib\os.py", line 980, in fdopen
return io.open(fd, *args, **kwargs)
OSError: [Errno 9] Bad file descriptor

Is there something else that I can try?  Is this still an issue that needs 
looking into?

--
nosy: +BreamoreBoy

___
Python tracker 

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

[issue24000] More fixes for the Clinic mapping of converters to format units

[Larry Hastings]

Larry Hastings added the comment:

As for "why not length instead of zeroes": Because the primary reason for the 
parameter is specifying that the string can contain embedded zeroes.  Returning 
the length is a side-effect of this, not the main point.  If the string didn't 
have embedded zeroes, we wouldn't need the length.

The only reason the code didn't have "zeroes=True" everywhere was because I 
screwed up and didn't realize all those mappings *should* have specified it.

The documentation is very consistent about calling it a NUL.  I don't think 
"NUL=True" or "allow_NUL=true" is particularly attractive; we never (almost 
never?) use capital letters in parameter names.  So any other name is going to 
be a compromise.  "allow_null" and "allow_nul" are misspellings, and don't 
convey the idea any better; they can confuse the reader with the related 
concept of NULL or None.  At least "zeroes" has the benefit of being an actual 
word, representing a related concept.

Will you be done bikeshedding soon?

--

___
Python tracker 

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

[issue24000] More fixes for the Clinic mapping of converters to format units

[Larry Hastings]

Larry Hastings added the comment:

I don't think those are better names.

--

___
Python tracker 

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

[issue24000] More fixes for the Clinic mapping of converters to format units

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

May be "string", or "data", or "buffer" would be better names? "str" looks as 
Python type.

--

___
Python tracker 

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

[issue24000] More fixes for the Clinic mapping of converters to format units

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

Why not left the length parameter instead? First, current code uses "length". 
Second, the main effect from C side is that an argument is a pair (pointer, 
length), not just a pointer. Third, currently everywhere in Python 
documentation and error messages the used term is "null character/byte", so if 
left the zeros parameter, it should be renamed to allow_nulls or allow_nuls. 
Fourth, "y#" needs zeros=True for distinguish from "y", but "y*" allows nulls 
and has no the zeros parameter.

--

___
Python tracker 

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

[issue23911] Move path-based bootstrap code to a separate frozen file.

[Eric Snow]

Changes by Eric Snow :


--
status: pending -> closed

___
Python tracker 

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

[issue24000] More fixes for the Clinic mapping of converters to format units

[Larry Hastings]

Larry Hastings added the comment:

As for "str doesn't even accept str for y and y#", the name "str" is not for 
the Python type, it's for the C type.

--

___
Python tracker 

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

[issue17477] update the bsddb module do build with db 5.x versions

[Mark Lawrence]

Mark Lawrence added the comment:

>From 
>http://www.oracle.com/technetwork/database/database-technologies/berkeleydb/downloads/index.html
> the latest version is 6.1.23.  Given the extended life span of 2.7 I'd assume 
>that the patch here is worth updating.  See also #18734.

--
nosy: +BreamoreBoy

___
Python tracker 

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

[issue1100942] Add datetime.time.strptime and datetime.date.strptime

[Mark Lawrence]

Mark Lawrence added the comment:

@Alexander as the datetime expert could you get this committed in time for 3.5?

--
nosy: +BreamoreBoy

___
Python tracker 

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

[issue24000] More fixes for the Clinic mapping of converters to format units

[Larry Hastings]

Larry Hastings added the comment:

Here's an updated patch where I've removed the "length" parameter to 
converters, instead relying solely on the "zeroes" parameter.

--
Added file: 
http://bugs.python.org/file39289/larry.one.more.clinic.format.unit.map.cleanup.5.txt

___
Python tracker 

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

[issue24000] More fixes for the Clinic mapping of converters to format units

[Larry Hastings]

Larry Hastings added the comment:

I have no interest in naming "str" to "pchar".

Yes, *most* of the other converters are named after the C type they translate 
to.  But so far converter names doesn't mention whether or not they represent 
pointers to types--it's "object", not "pobject", it's "PyBytesObject", not 
"pPyBytesObject".  That suggests the name "char" for the converter.  But we've 
already got a "char", and it would be confusing to use that one converter for 
both chars (small ints / single characters) and strings.

Adding "p" in front is not a convention we've ever used.  The word "pchar" does 
not appear in the Python source tree.  So the name "pchar" has no mnemonic 
value.

If I were to follow your advice, I should prefer the name "char_star".  But now 
we're using nine letters for what is almost certainly the most common 
converter.  And, again, the generic converter for objects is called "object", I 
do not propose to rename it to "object_star".  So this converter's name would 
be an exception to the rule.

But then again, C strings themselves are an exception to the rule.  They're not 
a built-in type as much as they are a *convention*.  So any name we give it 
will ultimately be something of a compromise.  And as compromises go "str" is 
great.  So far nobody has been confused by it.  It's short, and universally, 
instantly clear as to its meaning.

Furthermore, converters don't actually represent a C type.  They represent a 
*mapping*, from a Python type (or types) to a single C type.  So while it's a 
useful and productive convention to name converters after the type they convert 
to, it's hardly mandatory.  And it would be a shame to squander clarity in 
service to a needless consistency.

p.s. If we hold ourselves to this firm ideal, where every converter is named 
after its C type, what should we call the "bool" converter?  What should we 
call the "self" converter?  What should you call your proposed "boolint" 
converter?

--

___
Python tracker 

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

[issue3367] Uninitialized value read in parsetok.c

[Mark Lawrence]

Mark Lawrence added the comment:

The fix proposed by Alexander in issue3367.diff has never been applied.  How 
would I go about reproducing the original issue on Windows?

--
nosy: +BreamoreBoy
versions: +Python 3.4, Python 3.5 -Python 3.3

___
Python tracker 

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

[issue23911] Move path-based bootstrap code to a separate frozen file.

[Eric Snow]

Changes by Eric Snow :


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

___
Python tracker 

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

[issue23911] Move path-based bootstrap code to a separate frozen file.

[Roundup Robot]

Roundup Robot added the comment:

New changeset 36b902bbc992 by Eric Snow in branch 'default':
Issue #23911: Fix mixed bytes/strings.
https://hg.python.org/cpython/rev/36b902bbc992

--

___
Python tracker 

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

[issue24086] Configparser interpolation is unexpected

[Trevor Bekolay]

Trevor Bekolay added the comment:

Thanks for the quick response! I can see the use case for using interpolation 
in .pypirc. Unfortunately for me, I push releases for both Python 2 and Python 
3, so having the double percent sign will cause problems for me on Python 2.

The exception that's being raised is at line 442 
(https://hg.python.org/cpython/file/default/Lib/configparser.py#l442). The 
message itself seems to have the right information, but setuptools seems to 
mangle it  (perhaps InterpolationSyntaxError needs a __repr__ or __str__?)

--

___
Python tracker 

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

[issue24000] More fixes for the Clinic mapping of converters to format units

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

I would say it is very unlikely. In any case, if we have a pointer and a 
length, we always can check for zeros after parsing.

May be rename the str converter to pchar and the Py_UNICODE converter to 
pwchar? Usually the converter is named by C type, not Python type. "y" and "y#" 
even don't accept str.

--

___
Python tracker 

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

[issue24000] More fixes for the Clinic mapping of converters to format units

[Larry Hastings]

Larry Hastings added the comment:

Here's a freshened version of the patch.  I updated the Clinic HOWTO.

Serhiy: You're right, length and zeroes always have the same value.  Would you 
ever want length without allowing zeroes?  Like, in the future, would we ever 
want
str(length=True)
so we're passed in the length but we still don't want to allow zeroes?

--
Added file: 
http://bugs.python.org/file39288/larry.one.more.clinic.format.unit.map.cleanup.4.txt

___
Python tracker 

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

[issue24001] Clinic: use raw types in types= set

[Larry Hastings]

Changes by Larry Hastings :


--
resolution:  -> fixed
stage: patch review -> resolved
status: open -> closed

___
Python tracker 

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

[issue24119] Carry comments with the AST

[Christian Heimes]

Changes by Christian Heimes :


--
nosy: +christian.heimes

___
Python tracker 

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

[issue24088] yield expression confusion

[R. David Murray]

R. David Murray added the comment:

I don't think anything about frames is guaranteed as part of the language, so 
I'm not sure that mention of it belongs in the description.  Personally, I find 
your reformulation more confusing that the original with 'a' replaced by 'the'.

--

___
Python tracker 

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

[issue24119] Carry comments with the AST

[Brett Cannon]

Brett Cannon added the comment:

Normally I would agree comments don't belong there, but if we are going to 
start giving them semantic meaning then I don't think it's not so clear to me 
anymore.

As to where to attach, simple place is off of the Module node. Another is to 
have it be fundamental like lineno and only attach it when it is a 
line-trailing comment.

Yes, the tokenize module will give you the comments as well, but it is 
unfortunate you have to parse the code twice in order to get the comments and 
the AST.

--

___
Python tracker 

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

[issue24088] yield expression confusion

[Jim Jewett]

Jim Jewett added the comment:

OK, then how about

Current:
"""
When a generator function is called, it returns an iterator known as a 
generator. That generator then controls the execution of a generator function. 
The execution starts when one of the generator’s methods is called.
"""

Proposed:
"""
When a generator function is called, it does not complete its execution 
immediately.  Instead, it keeps its execution frame intact, and returns a 
special kind of iterator known as a generator. The iteration starts when one of 
the generator’s methods is called, and the generator executes within the 
existing generator frame, rather than creating a new one.
"""

This still seems to suggest that the generator uses the same frame as the 
generator function that created it; I was not aware that this was a guarantee.  
(Reusing the same frame, yes.  Reusing that particular frame, no.)

--

___
Python tracker 

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

[issue24001] Clinic: use raw types in types= set

[Roundup Robot]

Roundup Robot added the comment:

New changeset 41fb7fd04b5d by Larry Hastings in branch 'default':
Issue #24001: Argument Clinic converters now use accept={type}
https://hg.python.org/cpython/rev/41fb7fd04b5d

--
nosy: +python-dev

___
Python tracker 

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

[issue23911] Move path-based bootstrap code to a separate frozen file.

[Larry Hastings]

Larry Hastings added the comment:

This checkin broke the buildbots.  If you build trunk then run

./python -bb -m test test_site

the test fails.  "-bb" is used by the normal test runner ("make test").

The problem is in the lines

self.assertTrue(os.path.isabs(os__file__),
"expected absolute path, got {}".format(os__file__))
self.assertTrue(os.path.isabs(os__cached__),
"expected absolute path, got {}".format(os__cached__))

os__file__ and os__cached__ are bytes but you're passing them into .format() on 
a str.

--
nosy: +larry
resolution: fixed -> 
status: closed -> open

___
Python tracker 

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

[issue24102] Multiple type confusions in unicode error handlers

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

Also fixed handling errors of PyObject_IsSubclass() (issue24115) in the _codecs 
module.

--
Added file: 
http://bugs.python.org/file39287/codecs_error_handlers_issubclass_3.patch

___
Python tracker 

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

[issue24124] Two versions of instructions for installing Python modules

[Skip Montanaro]

New submission from Skip Montanaro:

I was hunting around the current website for notes on installing Python. 
Couldn't find any (BTW), so I looked in the Python source. Imagine my surprise 
to find two apparently overlapping files describing how to build Python modules:

./Doc/install/index.rst
./Doc/installing/index.rst

The former is marked "Legacy version." The latter seems to cover mostly 
installation of third-party modules using tools like pip. The former covers 
distutils.  Oddly enough, the "legacy" version seems to have been updated more 
recently than the presumably current version. Do we really need to keep both 
versions around at this point? Should they be merged into a single document? If 
they are to both be retained, should the legacy version be marked as such more 
clearly and refer readers to the pip/pyvenv/virtualenv version?

--
assignee: docs@python
components: Documentation
messages: 242555
nosy: docs@python, skip.montanaro
priority: normal
severity: normal
status: open
title: Two versions of instructions for installing Python modules
versions: Python 3.5

___
Python tracker 

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

[issue20159] Derby #7: Convert 51 sites to Argument Clinic across 3 files -> Derby: Convert the ElementTree module to use Argument Clinic

[Serhiy Storchaka]

Changes by Serhiy Storchaka :


--
nosy:  -serhiy.storchaka

___
Python tracker 

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

[issue20159] Derby #7: Convert 51 sites to Argument Clinic across 3 files -> Derby: Convert the ElementTree module to use Argument Clinic

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

Original issue is resolved and I have no any relations to modules added by 
Larry.

--
dependencies:  -Add docstrings for ElementTree module
stage: patch review -> needs patch

___
Python tracker 

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

[issue20148] Derby: Convert the _sre module to use Argument Clinic

[Serhiy Storchaka]

Changes by Serhiy Storchaka :


--
resolution:  -> fixed
status: open -> closed

___
Python tracker 

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

[issue20168] Derby: Convert the _tkinter module to use Argument Clinic

[Serhiy Storchaka]

Changes by Serhiy Storchaka :


--
resolution:  -> fixed
status: open -> closed

___
Python tracker 

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

[issue24116] --with-pydebug has no effect when the final python binary is compiled

[Skip Montanaro]

Skip Montanaro added the comment:

> It's confusing that the CFLAGS specified when running make are passed further 
> sometimes but not always. So I guess that's a workaround? Or CFLAGS should 
> never be specified to make directly?

In my experience, all the magic happens in configure. Most of the time, the 
only flag I give to make is -j to specify parallelism.

Is there somewhere in the build docs this could be made clearer?

--
nosy: +skip.montanaro

___
Python tracker 

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

[issue20159] Derby #7: Convert 51 sites to Argument Clinic across 3 files -> Derby: Convert the ElementTree module to use Argument Clinic

[Roundup Robot]

Roundup Robot added the comment:

New changeset fea94f9cb5a0 by Serhiy Storchaka in branch 'default':
Issue #20159. Converted the _elementtree module to Argument Clinic.
https://hg.python.org/cpython/rev/fea94f9cb5a0

--
nosy: +python-dev

___
Python tracker 

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

[issue20179] Derby #10: Convert 50 sites to Argument Clinic across 4 files

[Larry Hastings]

Larry Hastings added the comment:

Clinic's syntax is diverging from what shipped with 3.4.  So if you copied 
_ssl.c over, it wouldn't work with the Clinic that shipped with 3.4.

Maybe the best thing is if Clinic in trunk supports "legacy mode", where the 
code it generates is compatible with previous Python versions.  That's 
basically B but without doing something crazy like shipping Clinic with 2.7.

--

___
Python tracker 

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

[issue20179] Derby #10: Convert 50 sites to Argument Clinic across 4 files

[Antoine Pitrou]

Antoine Pitrou added the comment:

> I can propose three remedies:
> 
> A) back out the Clinic conversion in _ssl.c
> B) support Clinic in 2.7 just for _ssl.c
> C) do a one-time backport of the Clinic generated code for _ssl.c

I'd rather have A or C than B.
By the way, this discussion seems to focus on 2.7, but the same issue happens 
with 3.4 (although Clinic already exists here, so we can just backport Serhiy's 
work).

--

___
Python tracker 

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

[issue24100] Use after free in siftdown (2)

[Raymond Hettinger]

Changes by Raymond Hettinger :


--
resolution:  -> fixed
status: open -> closed

___
Python tracker 

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

[issue24101] Use after free in siftup

[Raymond Hettinger]

Changes by Raymond Hettinger :


--
resolution:  -> fixed
status: open -> closed

___
Python tracker 

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

[issue24099] Use after free in siftdown (1)

[Raymond Hettinger]

Changes by Raymond Hettinger :


--
resolution:  -> fixed
status: open -> closed

___
Python tracker 

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

[issue24123] Python 2.7 Tutorial Conflicting behavior with WeakValueDictionary.

[R. David Murray]

R. David Murray added the comment:

Well, technically it is probably not a bug.  IPython is doubtless holding on to 
a reference to 'a' because it was defined at the prompt.  Perhaps it could use 
a weakvaluedict for that, though :)  On the other hand they might consider it a 
feature that objects don't go out of scope.

--
stage:  -> resolved

___
Python tracker 

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

[issue15993] Windows: 3.3.0-rc2.msi: test_buffer fails

[Mark Lawrence]

Mark Lawrence added the comment:

Is this now fixed in VS?  I don't believe I can test myself as I've only got 
express/community editions.

--
components: +Windows
nosy: +BreamoreBoy

___
Python tracker 

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

[issue24122] Install fails after configure sets the extending/embedding install directory to NONE

[Xavier de Gaye]

Changes by Xavier de Gaye :


--
nosy: +larry

___
Python tracker 

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

[issue3786] _curses, _curses_panel & _multiprocessing can't be build in 2.6b3 w/ SunStudio 12

[Wessel Badenhorst]

Changes by Wessel Badenhorst :


--
type:  -> compile error

___
Python tracker 

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

[issue3539] Problem with testembed make dependencies in certain circumstances

[Wessel Badenhorst]

Changes by Wessel Badenhorst :


--
type:  -> compile error

___
Python tracker 

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

[issue2824] zipfile to handle duplicate files in archive

[Wessel Badenhorst]

Changes by Wessel Badenhorst :


--
type:  -> enhancement

___
Python tracker 

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

[issue23670] Modifications to support iOS as a cross-compilation target

[Russell Keith-Magee]

Russell Keith-Magee added the comment:

This new patch (20150504.diff) adds support for running the Python test suite 
The new patch is standalone, and contains everything in the previous patch.

An XCode project (Tools/iOS-test) has been added to the source tree; this 
project contains bootstrap C code to start and run the full Python regression 
test suite.

There's also a new target in the iOS/Makefile meta-buildfile - "make test" - 
which will compile a debug version of the Python framework, and install it into 
the iOS-test project.

Getting the test suite to run has revealed one major limitation of the iOS 
platform - system calls like fork, exec*, and spawn* don't work. The OS calls 
*exist* at an API level - but if you use them, they either crash, or they lock 
up the device while you wait for a subprocess that will never execute. This 
makes sense considering the platform itself - there's no such thing as a 
"background process" in iOS; background tasks are very heavily sandboxed. 

A number of other minor problems have been identified as a result of running 
the full test suite; they have been addressed in the patch.

The test suite still has 5 failures on the simulator. I'm investigating the 
cause of these failures.

There's a couple of more failures on a physical device - 13 failures in total. 
These device-specific failures appear to be largely due to ctypes problems and 
a permissions issue with os.mkdir. 

If you run the test suite in XCode as a debug binary, the debugger will stop 
whenever a SIGPIPE, SIGINT, SIGXFSZ, SIGUSR1 or SIGUSR2 signal is raised. You 
can just hit continue in the debugger, and the test will continue. To work 
around this, you need to run the following debugger commands before the first 
signal is raised:

process handle SIGPIPE -n true -p true -s false
process handle SIGINT -n true -p true -s false
process handle SIGXFSZ -n true -p true -s false
process handle SIGUSR1 -n true -p true -s false
process handle SIGUSR2 -n true -p true -s false

I've been doing this by setting a breakpoint in the main.c method; I'm 
investigating other ways to automate this.

--
Added file: http://bugs.python.org/file39286/20150504.diff

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

[issue20159] Derby #7: Convert 51 sites to Argument Clinic across 3 files -> Derby: Convert the ElementTree module to use Argument Clinic

[Larry Hastings]

Larry Hastings added the comment:

LGTM

--

___
Python tracker 

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

[issue20159] Derby #7: Convert 51 sites to Argument Clinic across 3 files -> Derby: Convert the ElementTree module to use Argument Clinic

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

Made compatible with Windows compiler (I hope).

--
Added file: http://bugs.python.org/file39285/etree_clinic_3.patch

___
Python tracker 

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