[issue21120] PyArena type is used in headers from the limited API

[Mark Shannon]

Change by Mark Shannon :


--
status: open -> closed

___
Python tracker 

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

[issue21120] PyArena type is used in headers from the limited API

[Mark Shannon]

Change by Mark Shannon :


--
stage: patch review -> resolved

___
Python tracker 

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

[issue21120] PyArena type is used in headers from the limited API

[Mark Shannon]

Mark Shannon  added the comment:


New changeset 421a72af4deaec96a49a79951b9c2546a2faa13d by Mark Shannon (Zackery 
Spytz) in branch 'master':
bpo-21120: Exclude Python-ast.h, ast.h and asdl.h from the limited API (#14634)
https://github.com/python/cpython/commit/421a72af4deaec96a49a79951b9c2546a2faa13d


--

___
Python tracker 

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

[issue21120] PyArena type is used in headers from the limited API

[Mark Shannon]

Mark Shannon  added the comment:

This seems like the correct thing to do.
Since the AST changes from version to version, I don't see how these files 
could reasonably be part of the limited API.

--
nosy: +Mark.Shannon

___
Python tracker 

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

[issue21120] PyArena type is used in headers from the limited API

[Zackery Spytz]

Change by Zackery Spytz :


--
components: +Build -Library (Lib)
nosy: +ZackerySpytz
versions: +Python 3.9 -Python 3.5

___
Python tracker 

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

[issue21120] PyArena type is used in headers from the limited API

[Zackery Spytz]

Change by Zackery Spytz :


--
keywords: +patch
pull_requests: +14448
stage:  -> patch review
pull_request: https://github.com/python/cpython/pull/14634

___
Python tracker 

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

[issue21120] PyArena type is used in headers from the limited API

[Mark Lawrence]

Change by Mark Lawrence :


--
nosy:  -BreamoreBoy

___
Python tracker 

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

[issue21120] PyArena type is used in headers from the limited API

[Mark Lawrence]

Mark Lawrence added the comment:

I'm not sure if this should be a compile error or an enhancement request, can 
someone please advise.

--
nosy: +BreamoreBoy

___
Python tracker 

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

[issue21120] PyArena type is used in headers from the limited API

[Jesús Cea Avión]

Changes by Jesús Cea Avión :


--
nosy: +jcea

___
Python tracker 

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

[issue21120] PyArena type is used in headers from the limited API

[STINNER Victor]

Changes by STINNER Victor :


--
nosy: +loewis

___
Python tracker 

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

[issue21120] PyArena type is used in headers from the limited API

[Andrey]

New submission from Andrey:

Hi all,

The PyArena data type is defined in the pyarena.h under the #ifndef 
Py_LIMITED_API statement, so it's not included in the limited api. But this 
type is used in Python-ast.h, ast.h and asdl.h headers that included in the 
limited api, because they don't contain any checks for Py_LIMITED_API.

May be all these header files (Python-ast.h, ast.h, asdl.h) should begin with 
"#ifndef Py_LIMITED_API" (excluded from the limited api)?

Thanks.

--
components: Library (Lib)
messages: 215300
nosy: aponomarenko
priority: normal
severity: normal
status: open
title: PyArena type is used in headers from the limited API
type: compile error
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