Skip to content

Commit 71367a3

Browse files
rozelefacebook-github-bot
authored andcommitted
Remove hacks to metro BUCK for handling macos_legacy
Summary: Backs out all of the changes introduced in D68246115 as they are no longer needed. Reviewed By: IanChilds Differential Revision: D74860891 fbshipit-source-id: aa528e70c7c3674b69a02e2cb7ff8d1927c26852
1 parent 614105e commit 71367a3

File tree

2 files changed

+3
-5
lines changed

2 files changed

+3
-5
lines changed

prelude/js/js.bzl

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,8 +21,6 @@ def _select_platform():
2121
"config//os/constraints:windows": "windows",
2222
}),
2323
"config//react-native:macos": "macos",
24-
# TODO(T210407097): Remove after deleting //third-party/microsoft-fork-of-react-native
25-
"fbsource//tools/build_defs/js/config:macos_legacy": "macos_legacy",
2624
"fbsource//tools/build_defs/js/constraints/metro_js_platform_override:android": "android",
2725
"fbsource//tools/build_defs/js/constraints/metro_js_platform_override:ios": "ios",
2826
"fbsource//tools/build_defs/js/constraints/metro_js_platform_override:macos": "macos",

prelude/js/js_utils.bzl

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -58,10 +58,10 @@ ASSET_EXTENSIONS = [
5858
]
5959

6060
# Matches the default value for resolver.platforms in metro-config
61-
ASSET_PLATFORMS = ["ios", "android", "windows", "macos", "macos_legacy", "web"]
61+
ASSET_PLATFORMS = ["ios", "android", "windows", "macos", "web"]
6262

6363
def get_apple_resource_providers_for_js_bundle(ctx: AnalysisContext, js_bundle_info: JsBundleInfo, platform: str, skip_resources: bool) -> list[ResourceGraphInfo]:
64-
if platform != "ios" and platform != "macos" and platform != "macos_legacy":
64+
if platform != "ios" and platform != "macos":
6565
return []
6666

6767
# `skip_resources` controls whether the JS resources should be skipped, not whether
@@ -126,7 +126,7 @@ def get_bundle_name(ctx: AnalysisContext, default_bundle_name: str) -> str:
126126
flavors = bundle_name_for_flavor_map.keys()
127127
for flavor in flavors:
128128
expect(
129-
flavor == "android" or flavor == "ios" or flavor == "macos" or flavor == "macos_legacy" or flavor == "windows" or flavor == "vr",
129+
flavor == "android" or flavor == "ios" or flavor == "macos" or flavor == "windows" or flavor == "vr",
130130
"Currently only support picking bundle name by platform!",
131131
)
132132

0 commit comments

Comments
 (0)