-
Notifications
You must be signed in to change notification settings - Fork 4.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support exec_compatible_with on cc_binary/cc_library. #8026
Labels
P2
We'll consider working on this in future. (Assignee optional)
team-Configurability
platforms, toolchains, cquery, select(), config transitions
type: feature request
Comments
@katre for triage |
This is a fairly simple request to implement, the open question is "which rules should allow adding exec_compatible_with?" Right now, the answer is:
|
Discussed this today with the configurability team, and we decided to just throw it open and not restrict the target types. I'll send a PR shortly. |
katre
added a commit
to katre/bazel
that referenced
this issue
Apr 24, 2019
the execution platform. Remove the ExecutionPlatformConstraintsAllowed enum entirely. Part of work on bazelbuild#8026.
Okay, not as easy as a single PR. Plan:
|
katre
added a commit
to katre/bazel
that referenced
this issue
Apr 24, 2019
additional constraints on the execution platform. Part of work on bazelbuild#8134. Fixes bazelbuild#8026.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
P2
We'll consider working on this in future. (Assignee optional)
team-Configurability
platforms, toolchains, cquery, select(), config transitions
type: feature request
Description of the problem / feature request:
Currently, exec_compatible_with is only available on genrules, shell rules and *_test: https://docs.bazel.build/versions/master/be/common-definitions.html#exec_compatible_with
Feature requests: what underlying problem are you trying to solve with this feature?
A Remote Build Execution customer has a C++ compile action which runs out of memory on
n1-standard-2
machines, and it would be more cost-effective for them if they could target a small pool of larger machines, rather than replacing all of their machines with larger machines.Bugs: what's the simplest, easiest way to reproduce this bug? Please provide a minimal example if possible.
What operating system are you running Bazel on?
0.23.2
What's the output of
bazel info release
?Don't know (reporting for a customer).
If
bazel info release
returns "development version" or "(@Non-Git)", tell us how you built Bazel.N/A
What's the output of
git remote get-url origin ; git rev-parse master ; git rev-parse HEAD
?N/A
Have you found anything relevant by searching the web?
No
Any other information, logs, or outputs that you want to share?
The text was updated successfully, but these errors were encountered: