Skip to content

Commit

Permalink
Add notes on (the lack of) fat binary support (#273)
Browse files Browse the repository at this point in the history
  • Loading branch information
katre committed Aug 11, 2022
1 parent a5435a8 commit 912ae9d
Showing 1 changed file with 7 additions and 3 deletions.
10 changes: 7 additions & 3 deletions designs/2022-08-03-platforms-on-targets.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
---
created: 2022-08-03
last updated: 2022-08-03
last updated: 2022-08-11
status: Under Review
reviewers: @gregestren
title: Platforms on Targets
Expand Down Expand Up @@ -38,6 +38,12 @@ targets will be added in three phases:
Stage 3 may be delayed or dropped based on the progress with the first two, and
whether or not users need this feature.

Bazel will support specifying a single platform per target. Some build rules
(such as builds for Android or iOS) allow builds for so-called "fat" binaries
with multiple CPU architectures. These rules will need their own support for
these richer sets of platforms, presumably via their own flags and attributes,
and will not be handled via this proposal.

## Syntax

To support declaring platforms on targets, a new attribute will be added to
Expand All @@ -46,8 +52,6 @@ be optional with no default value, and will require the value to provide the
PlatformInfo provider. The attribute will not be configurable (so a select
cannot be used to set the value).

**Open question:** Should this be a `label_list` valued `platforms` attribute?

```
platform(
name = "windows",
Expand Down

0 comments on commit 912ae9d

Please sign in to comment.