Skip to content

Commit 6f807cd

Browse files
naveen521kknaveen521kk
committed
sysconfig.get_platform(): use consistent naming
Currently, the platform names are hardcoded in many places, and are not named consistently. This commit fixes it by standardizing the names to be returned by `sysconfig.get_platform`. The naming is based on #167 (comment) Similarly, `EXT_SUFFIX` is also standardized to be consistent with the platform names. Signed-off-by: Naveen M K <[email protected]>
1 parent 038178f commit 6f807cd

File tree

5 files changed

+83
-75
lines changed

5 files changed

+83
-75
lines changed

‎Lib/site.py‎

Lines changed: 19 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -314,21 +314,27 @@ def joinuser(*args):
314314
def _get_platform():
315315
if os.name == 'nt':
316316
if 'gcc' in sys.version.lower():
317+
platform = 'mingw'
318+
if 'amd64' in sys.version.lower():
319+
platform += '_x86_64'
320+
elif 'arm64' in sys.version.lower():
321+
platform += '_aarch64'
322+
elif 'arm' in sys.version.lower():
323+
platform += '_armv7'
324+
else:
325+
platform += '_i686'
326+
317327
if 'ucrt' in sys.version.lower():
318-
if 'amd64' in sys.version.lower():
319-
return 'mingw_x86_64_ucrt'
320-
return 'mingw_i686_ucrt'
328+
platform += '_ucrt'
329+
else:
330+
platform += "_msvcrt"
331+
321332
if 'clang' in sys.version.lower():
322-
if 'amd64' in sys.version.lower():
323-
return 'mingw_x86_64_clang'
324-
if 'arm64' in sys.version.lower():
325-
return 'mingw_aarch64'
326-
if 'arm' in sys.version.lower():
327-
return 'mingw_armv7'
328-
return 'mingw_i686_clang'
329-
if 'amd64' in sys.version.lower():
330-
return 'mingw_x86_64'
331-
return 'mingw_i686'
333+
platform += "_llvm"
334+
else:
335+
platform += "_gnu"
336+
337+
return platform
332338
return sys.platform
333339

334340
# Same to sysconfig.get_path('purelib', os.name+'_user')

‎Lib/sysconfig.py‎

Lines changed: 19 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -800,21 +800,27 @@ def get_platform():
800800
"""
801801
if os.name == 'nt':
802802
if 'gcc' in sys.version.lower():
803+
platform = 'mingw'
804+
if 'amd64' in sys.version.lower():
805+
platform += '_x86_64'
806+
elif 'arm64' in sys.version.lower():
807+
platform += '_aarch64'
808+
elif 'arm' in sys.version.lower():
809+
platform += '_armv7'
810+
else:
811+
platform += '_i686'
812+
803813
if 'ucrt' in sys.version.lower():
804-
if 'amd64' in sys.version.lower():
805-
return 'mingw_x86_64_ucrt'
806-
return 'mingw_i686_ucrt'
814+
platform += '_ucrt'
815+
else:
816+
platform += "_msvcrt"
817+
807818
if 'clang' in sys.version.lower():
808-
if 'amd64' in sys.version.lower():
809-
return 'mingw_x86_64_clang'
810-
if 'arm64' in sys.version.lower():
811-
return 'mingw_aarch64'
812-
if 'arm' in sys.version.lower():
813-
return 'mingw_armv7'
814-
return 'mingw_i686_clang'
815-
if 'amd64' in sys.version.lower():
816-
return 'mingw_x86_64'
817-
return 'mingw_i686'
819+
platform += "_llvm"
820+
else:
821+
platform += "_gnu"
822+
823+
return platform
818824
if 'amd64' in sys.version.lower():
819825
return 'win-amd64'
820826
if '(arm)' in sys.version.lower():

‎Lib/test/test_importlib/test_windows.py‎

Lines changed: 22 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -25,22 +25,28 @@ def get_platform():
2525
'arm' : 'win-arm32',
2626
}
2727
if os.name == 'nt':
28-
if 'gcc' in sys.version.lower():
29-
if 'ucrt' in sys.version.lower():
30-
if 'amd64' in sys.version.lower():
31-
return 'mingw_x86_64_ucrt'
32-
return 'mingw_i686_ucrt'
33-
if 'clang' in sys.version.lower():
34-
if 'amd64' in sys.version.lower():
35-
return 'mingw_x86_64_clang'
36-
if 'arm64' in sys.version.lower():
37-
return 'mingw_aarch64'
38-
if 'arm' in sys.version.lower():
39-
return 'mingw_armv7'
40-
return 'mingw_i686_clang'
41-
if 'amd64' in sys.version.lower():
42-
return 'mingw_x86_64'
43-
return 'mingw_i686'
28+
if "gcc" in sys.version.lower():
29+
platform = "mingw"
30+
if "amd64" in sys.version.lower():
31+
platform += "_x86_64"
32+
elif "arm64" in sys.version.lower():
33+
platform += "_aarch64"
34+
elif "arm" in sys.version.lower():
35+
platform += "_armv7"
36+
else:
37+
platform += "_i686"
38+
39+
if "ucrt" in sys.version.lower():
40+
platform += "_ucrt"
41+
else:
42+
platform += "_msvcrt"
43+
44+
if "clang" in sys.version.lower():
45+
platform += "_llvm"
46+
else:
47+
platform += "_gnu"
48+
49+
return platform
4450
if ('VSCMD_ARG_TGT_ARCH' in os.environ and
4551
os.environ['VSCMD_ARG_TGT_ARCH'] in TARGET_TO_PLAT):
4652
return TARGET_TO_PLAT[os.environ['VSCMD_ARG_TGT_ARCH']]

‎configure.ac‎

Lines changed: 22 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -6313,16 +6313,12 @@ AC_C_BIGENDIAN
63136313

63146314
AC_SUBST(PYD_PLATFORM_TAG)
63156315
# Special case of PYD_PLATFORM_TAG with python build with mingw.
6316-
# Python can with compiled with clang or gcc and linked
6317-
# to msvcrt or ucrt. To avoid conflicts between them
6318-
# we are selecting the extension as based on the compiler
6319-
# and the runtime they link to
6320-
# gcc + x86_64 + msvcrt = cp{version number}-x86_64
6321-
# gcc + i686 + msvcrt = cp{version number}-i686
6322-
# gcc + x86_64 + ucrt = cp{version number}-x86_64-ucrt
6323-
# clang + x86_64 + ucrt = cp{version number}-x86_64-clang
6324-
# clang + i686 + ucrt = cp{version number}-i686-clang
6325-
6316+
# Python can with different cpu arch and c runtime as well as different
6317+
# toolchain. We follow this`mingw_<cpu_arch>_<c_runtime>_<toolchain>`
6318+
# convention for PYD_PLATFORM_TAG. Where:
6319+
# `cpu_arch` = `x86_64`, `aarch64` or `i686`
6320+
# `c_runtime` = `msvcrt` or `ucrt`
6321+
# `toolchain` = `gnu` or `llvm`
63266322
PYD_PLATFORM_TAG=""
63276323
case $host in
63286324
*-*-mingw*)
@@ -6341,38 +6337,32 @@ esac
63416337
case $host_os in
63426338
mingw*)
63436339
AC_MSG_CHECKING(PYD_PLATFORM_TAG)
6340+
PYD_PLATFORM_TAG="mingw"
63446341
case $host in
63456342
i686-*-mingw*)
6346-
if test -n "${cc_is_clang}"; then
6347-
# it is CLANG32
6348-
PYD_PLATFORM_TAG="mingw_i686_clang"
6349-
else
6350-
if test $linking_to_ucrt = no; then
6351-
PYD_PLATFORM_TAG="mingw_i686"
6352-
else
6353-
PYD_PLATFORM_TAG="mingw_i686_ucrt"
6354-
fi
6355-
fi
6343+
PYD_PLATFORM_TAG+="_i686"
63566344
;;
63576345
x86_64-*-mingw*)
6358-
if test -n "${cc_is_clang}"; then
6359-
# it is CLANG64
6360-
PYD_PLATFORM_TAG="mingw_x86_64_clang"
6361-
else
6362-
if test $linking_to_ucrt = no; then
6363-
PYD_PLATFORM_TAG="mingw_x86_64"
6364-
else
6365-
PYD_PLATFORM_TAG="mingw_x86_64_ucrt"
6366-
fi
6367-
fi
6346+
PYD_PLATFORM_TAG+="_x86_64"
63686347
;;
63696348
aarch64-*-mingw*)
6370-
PYD_PLATFORM_TAG+="mingw_aarch64"
6349+
PYD_PLATFORM_TAG+="_aarch64"
63716350
;;
63726351
armv7-*-mingw*)
6373-
PYD_PLATFORM_TAG+="mingw_armv7"
6352+
PYD_PLATFORM_TAG+="_armv7"
63746353
;;
63756354
esac
6355+
if test $linking_to_ucrt = no; then
6356+
PYD_PLATFORM_TAG+="_msvcrt"
6357+
else
6358+
PYD_PLATFORM_TAG += "_ucrt"
6359+
fi
6360+
if test -n "${cc_is_clang}"; then
6361+
# it is CLANG32
6362+
PYD_PLATFORM_TAG+="_llvm"
6363+
else
6364+
PYD_PLATFORM_TAG+="_gnu"
6365+
fi
63766366
AC_MSG_RESULT($PYD_PLATFORM_TAG)
63776367
esac
63786368

‎mingw_smoketests.py‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@
3737
if sysconfig.is_python_build():
3838
os.environ["PYTHONLEGACYWINDOWSDLLLOADING"] = "1"
3939

40-
_UCRT = sysconfig.get_platform() not in ('mingw_x86_64', 'mingw_i686')
40+
_UCRT = 'ucrt' in sysconfig.get_platform()
4141

4242

4343
class Tests(unittest.TestCase):

0 commit comments

Comments
 (0)