MSVC builds are broken

[SaracenOne]

Godot version

f03ac21

System information

Windows 10, Vulkan, Intel® Core™ i7-8700K, Nvidia GTX 1080 (471.11)

Issue description

Bisected issue to this #55778. Seems to consistently break with the error 'Arch argument should not be used when using VCINSTALLDIR' when compiling with the Visual Studio toolchain regardless whether the arch argument is absent or any other are seemingly used. Seems to compile fine though if this block of code is outright removed:

    print(env["arch"])
    if env["arch"] != "auto":
        print(
            """
            Arch argument is not supported for MSVC manual configuration (VCINSTALLDIR configured).
            Architecture depends on the Native/Cross Compile Tools Prompt/Developer Console (or Visual Studio settings) that is being used to run SCons.
            As a consequence, the arch argument is disabled. Run scons again without arch argument (example: scons p=windows)
            and SCons will attempt to detect what MSVC compiler will be executed and inform you.
            """
        )
        raise SCons.Errors.UserError("Arch argument should not be used when using VCINSTALLDIR")```

Steps to reproduce

Simply attempt to run SCons with the MSVC toolchain. This is the command I'm using
"C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Auxiliary\Build\vcvarsall.bat" x86_amd64 && scons platform=windows

Minimal reproduction project

N/A

[akien-mga]

I believe @bruvzg is working on cleaning this up.

[aaronfranke]

We should probably have it either allow auto or the same value as the MSVC manual configuration's arch, and only error if the arch mismatches what the MSVC configuration is providing.

[bruvzg]

I guess custom version of detect_arch() to detect environment and set default arch value (for both MSVC and MinGW, with a test runs of compiler and windres in case of MinGW), and check for mismatching arch when using vcvarsall.

[akien-mga]

BTW, this part is not urgent, but the whole Windows detect.py seems really convoluted and outdated. It could use a thorough cleanup.

[SaracenOne]

Yeah, there are messages to that effect. I know I pointed this whole regression out earlier, but it felt right to properly log this issue as soon as possible so it can be tracked

[RandomShaper]

Same here. But it seems it just works if I don't call any of the vcvars*.bat before invoking SCons.

Aside, @SaracenOne, x86_amd64 in your command line has drawn my attention. That's meant for cross-compiling to x64 from a 32-bit host, isn't it? I just call vcvars64.bat, which boils down to vcvarsall.bat" x64 for native x64 compilation.