Skip to content

Commit b55538d

Browse files
JakobDegenfacebook-github-bot
authored andcommitted
prelude: Add fat_mac_linux platform to os lookup platforms
Summary: Currently this just returns `linux` which is... completely wrong Reviewed By: dtolnay Differential Revision: D73008565 fbshipit-source-id: c1407790d93db6874823a327043df0b0bc1ebc32
1 parent f2744ed commit b55538d

File tree

2 files changed

+6
-0
lines changed

2 files changed

+6
-0
lines changed

Diff for: prelude/os_lookup/defs.bzl

+1
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@
88
load("@prelude//decls:core_rules.bzl", "PlatformExePlatform", "TargetCpuType")
99

1010
Os = enum(
11+
"fat_mac_linux",
1112
"freebsd",
1213
"unknown",
1314
*PlatformExePlatform

Diff for: prelude/os_lookup/targets/BUCK

+5
Original file line numberDiff line numberDiff line change
@@ -13,9 +13,14 @@ os_lookup(
1313
"config//cpu:x86_64": "x86_64",
1414
}),
1515
os = select({
16+
# FIXME(JakobDegen): No default
1617
"DEFAULT": "linux",
18+
"config//os:linux": "linux",
1719
"config//os:macos": "macos",
1820
"config//os:windows": "windows",
21+
# FIXME(JakobDegen): We assuem fat means fat mac linux, but that might not be true in the
22+
# future.
23+
"prelude//platforms:fat_platform_enabled": "fat_mac_linux",
1924
}),
2025
script = select({
2126
"DEFAULT": "sh",

0 commit comments

Comments
 (0)