diff options
| author | Andreia Gaita <shana@spoiledcat.net> | 2024-02-05 20:55:06 +0100 |
|---|---|---|
| committer | Andreia Gaita <shana@spoiledcat.net> | 2024-02-13 22:48:43 +0100 |
| commit | 21e524a798bf3c3728f63657ff2e7a79f6de9ee6 (patch) | |
| tree | 5c0cd368101dd87bd1cafc86859a68338e93b0fa /modules/mono/config.py | |
| parent | 3a8524dd923e9d9a79e4979a47fd427388ea7010 (diff) | |
| download | redot-engine-21e524a798bf3c3728f63657ff2e7a79f6de9ee6.tar.gz | |
C#: Let platforms signal if they support it or not
Instead of hardcoding platform names that support C#, let platforms
set a flag indicating if they support it. All public platforms
except web already support it, and it's a pain to maintain a patch
for this list just to add additional names of proprietary console
platforms.
This makes adding new platforms or variants or existing platforms
much easier, as the platform can signal what it supports/doesn't
support directly, and we can avoid harcoding platform names.
Diffstat (limited to 'modules/mono/config.py')
| -rw-r--r-- | modules/mono/config.py | 10 |
1 files changed, 3 insertions, 7 deletions
diff --git a/modules/mono/config.py b/modules/mono/config.py index 859d77b262..3d087c9e27 100644 --- a/modules/mono/config.py +++ b/modules/mono/config.py @@ -1,8 +1,3 @@ -# Prior to .NET Core, we supported these: ["windows", "macos", "linuxbsd", "android", "web", "ios"] -# Eventually support for each them should be added back. -supported_platforms = ["windows", "macos", "linuxbsd", "android", "ios"] - - def can_build(env, platform): if env["arch"].startswith("rv"): return False @@ -14,9 +9,10 @@ def can_build(env, platform): def configure(env): - platform = env["platform"] + # Check if the platform has marked mono as supported. + supported = env.get("supported", []) - if platform not in supported_platforms: + if not "mono" in supported: raise RuntimeError("This module does not currently support building for this platform") env.add_module_version_string("mono") |