GeezerVison

cc. @corona10

What about this change? #117819

What about this change? #117819

IMO that should remain. We want to filter emcc out in ac_cv_cc_name switches.

#if defined(__EMSCRIPTEN__)
  emcc

But this code is no reason to remain if we purge them no?

#if defined(__EMSCRIPTEN__)
  emcc

But this code is no reason to remain if we purge them no?

emcc also defines __clang__ and __GNUC__ :(

Okay, so to filter out emcc, we need emcc :(

Would you like to add a comment on that code to annotate that we do not support each according to PEP?

Okay, so to filter out emcc, we need emcc :(

Or to put it another way: to filter out emcc, we need to know how to identify it :)

Would you like to add a comment on that code to annotate that we do not support each according to PEP?

We already have the switch in L1130. Did you think about a comment in the check that defines ac_cv_cc_name?

Alternatively, we can just keep this code; there is not a huge maintenance burden; a solution could be to add a comment above the lines that are proposed to be purged by this PR.

I don't get why this code has to be removed right now. CPython is full of code specific to "unsupported" platforms such as MinGW or OpenBSD. There are CPython users using this build mode: build CPython for the browser.

I don't get why this code has to be removed right now. CPython is full of code specific to "unsupported" platforms such as MinGW or OpenBSD. There are CPython users using this build mode: build CPython for the browser.

Yes, let's go with my suggestion from #117836 (comment).

"CPython in the browser" is a top topic with https://pyodide.org/ and https://pyscript.net/ projects.