GH-156554: Update PY_SUPPORT_TIER according to PEP 11 - #156576
GH-156554: Update PY_SUPPORT_TIER according to PEP 11#156576chris-eibl wants to merge 6 commits into
Conversation
StanFromIreland
left a comment
There was a problem hiding this comment.
configure.ac also needs updating (for completeness):
Line 1269 in d94c4ad
However I see aarch64-*-linux-gnu/gcc also needs moving to tier 1 in that list.
Hm, I'd never have assumed the Windows part in configure.ac - and slightly above it states Line 1253 in d94c4ad Anyway, let's fix for completeness.
Hm, separate issue & PR or just clean this up in this PR? |
Indeed it's a little odd, I assume it's all there for completeness, it's easier to see if anything is missing if you have all of PEP 11 "documented" there. I think we can include the other one here as well. |
| [x86_64-*-linux-gnu/clang], [PY_SUPPORT_TIER=2], dnl Linux on AMD64, any vendor, glibc, clang | ||
|
|
||
| [aarch64-pc-windows-msvc/msvc], [PY_SUPPORT_TIER=3], dnl Windows ARM64, MSVC | ||
| [aarch64-pc-windows-msvc/msvc], [PY_SUPPORT_TIER=2], dnl Windows ARM64, MSVC |
There was a problem hiding this comment.
Please move it after aarch64-*-linux-gnu/clang, like in the PEP.
|
Oh, and |
|
Oh well, the exception proves the rule ;-) I think we can do them all here, but if you prefer to do it separately I won't object. |
|
Yupp, let's do them all here, I'll update the issue and the PR and the news entry. |
Uh oh!
There was an error while loading. Please reload this page.