bpo-89886: Clean up MACHDEP and _PYTHON_HOST_PLATFORM checks

[erlend-aasland]

https://bugs.python.org/issue45723

[erlend-aasland]

If When the basic CI passes, I'll send it to the bots.

[tiran]

There is no CI for cross building yet.

[erlend-aasland]

There is no CI for cross building yet.

Let's run it on the Emscripten container then; that's a cross-build, AFAIU.

[erlend-aasland]

Accidentally, I've got some ARM toolchains installed. I'll throw it at them and see how it fares.

[arhadthedev]

Just out of curiosity: what distributions still don't have Autoconf so we need pregenerated configure under version control? Won't it be better to let a user or a package manager to generate it on their system instead?

[tiran]

Package managers are not our concern. We keep configure and other scripts in git to lower the barrier for contributors.

Our configure.ac more than just autotools. It also requires automake, a recent version of autoconf-archive and pkg-config's M4 macros. Not every distro has a recent enough autoconf archive package. Some distros have the pkg-config M4 files in a separate package. Until a couple of weeks ago our configure.ac did not work with autoconf 2.71.

[arhadthedev]

Well, if this file cannot be removed because of specific requirements for its generation, maybe it can be marked as generated akin to outputs of Argument Clinic, Freeze, and Parser/asdl_c.py? It will allow GitHub to collapse it. By the way, I created #30031 to address this.

[erlend-aasland]

BTW, this is a case where the configure diff matters at little bit more than normal: we have no CI for changes like this, so being able to see that the refactored code actually produces the same result is crucial. Luckily, this diff is pretty short 🙂

[erlend-aasland]

Accidentally, I've got some ARM toolchains installed. I'll throw it at them and see how it fares.

Works for me.

[github-actions]

This PR is stale because it has been open for 30 days with no activity.

[MaxwellDupre]

From the number of tests passed on my machine (420) I would say this looks ok to me.
But I cant test for all the supported architectures.
Also, I saw 61 fatal error in the config.log.
Is this ok?
BTW how do you check the results on configure? A part from running full test suite and checking the log.

[erlend-aasland]

I have no time to follow up this clean-up. Closing.