blob: 3fef8a2d733ca58e3517ab4f5e337b71b7c7eb29 (
plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
|
// Copyright 2015 The Bazel Authors. All rights reserved.
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
// http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.
package com.google.devtools.build.lib.analysis.config;
import com.google.devtools.build.lib.analysis.config.transitions.PatchTransition;
import com.google.devtools.build.lib.skyframe.serialization.autocodec.AutoCodec;
/** Dynamic transition to the host configuration. */
public final class HostTransition implements PatchTransition {
@AutoCodec public static final HostTransition INSTANCE = new HostTransition();
private HostTransition() {}
@Override
public boolean isHostTransition() {
return true;
}
@Override
public BuildOptions patch(BuildOptions options) {
if (options.get(BuildConfiguration.Options.class).isHost) {
// If the input already comes from the host configuration, just return the existing values.
//
// We don't do this just for convenience: if an
// {@link com.google.devtools.build.lib.runtime.proto.InvocationPolicyOuterClass.InvocationPolicy}
// overrides option defaults, {@link FragmentOptions#getHost} won't honor that policy. That's
// because it uses its own options parser that's not aware of the policy. This can create
// problems for, e.g., {@link JavaOptions#getHost}, which promotes --host_foo flags to
// --foo flags. That works the first time you do it (since both of the original values
// were policy-processed). But not subsequent times.
//
// There are various ways to solve this problem (pass the policy to host options computation,
// manually set host.hostFoo = original.hostFoo). But those raise larger questions about the
// nature of host/target relationships, so for the time being this is a straightforward
// and practical fix.
return options.clone();
} else {
return options.createHostOptions();
}
}
}
|
