SCons: Fix potential Windows ANSI exception

[Repiteo]

Fixes #92065

Rectified the issue by looking up the explicit values these functions expect¹, which should prevent futher type errors from occuring. As far as making sure that a build progresses regardless, this adds an exception wrapper to that bit of code that caused the issue. If an exception does occur, it assumes that the Windows ANSI setup failed & explicitly disables color output. Also moved the check down slightly in the main SConstruct file so that it can output an error message via methods.print_error instead of a generic print.

A future solution could be to overhaul the color system to something closer to what colorama achieves, which has a dedicated catch/conversion for Windows specifically. I'm not wholly knowledgeable on that approach, so I didn't attempt to implement it here.

Footnotes

1. https://learn.microsoft.com/en-us/windows/console/getstdhandle
   https://learn.microsoft.com/en-us/windows/console/getconsolemode
   https://learn.microsoft.com/en-us/windows/console/setconsolemode ↩

[Calinou]

Should this be applied to doc/tools/make_rst.py as well? I believe it has a similar function.

[huwpascoe]

Would it be possible to wrap those repeated paragraphs into a utility function like try_to_enable_win32_console_colors() or something?

[Repiteo]

Should this be applied to doc/tools/make_rst.py as well? I believe it has a similar function.

Oh shoot, good catch! I'll apply it there as well

Would it be possible to wrap those repeated paragraphs into a utility function like try_to_enable_win32_console_colors() or something?

Technically yes, but that'd require some hackery in the situations where these files aren't run in the repo root (would be very common for the GDExtension SConstruct files). Making the entire repo "smarter" in this respect would be quite a logistical hurdle, and well outside the scope of this PR.

[akien-mga]

Thanks!