From 07bd86df11d00a56466fa9bf3f1220d2429cfab7 Mon Sep 17 00:00:00 2001 From: Googler Date: Thu, 3 May 2018 11:06:52 -0700 Subject: CppDebugPackageProvider is useful for more than just C++, so rename it. RELNOTES: None. PiperOrigin-RevId: 195279335 --- .../devtools/build/lib/rules/cpp/CcBinary.java | 4 +- .../lib/rules/cpp/CppDebugPackageProvider.java | 80 ---------------------- .../build/lib/rules/cpp/DebugPackageProvider.java | 77 +++++++++++++++++++++ 3 files changed, 79 insertions(+), 82 deletions(-) delete mode 100644 src/main/java/com/google/devtools/build/lib/rules/cpp/CppDebugPackageProvider.java create mode 100644 src/main/java/com/google/devtools/build/lib/rules/cpp/DebugPackageProvider.java (limited to 'src/main/java/com/google/devtools/build/lib') diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/CcBinary.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/CcBinary.java index d84045cf28..801606fc4a 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/cpp/CcBinary.java +++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/CcBinary.java @@ -542,8 +542,8 @@ public abstract class CcBinary implements RuleConfiguredTargetFactory { return ruleBuilder .addProvider(RunfilesProvider.class, RunfilesProvider.simple(runfiles)) .addProvider( - CppDebugPackageProvider.class, - new CppDebugPackageProvider( + DebugPackageProvider.class, + new DebugPackageProvider( ruleContext.getLabel(), strippedFile, executable, explicitDwpFile)) .setRunfilesSupport(runfilesSupport, executable) .addProvider( diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppDebugPackageProvider.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppDebugPackageProvider.java deleted file mode 100644 index 31d3f9d81f..0000000000 --- a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppDebugPackageProvider.java +++ /dev/null @@ -1,80 +0,0 @@ -// Copyright 2014 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.rules.cpp; - -import com.google.common.base.Preconditions; -import com.google.devtools.build.lib.actions.Artifact; -import com.google.devtools.build.lib.analysis.TransitiveInfoProvider; -import com.google.devtools.build.lib.cmdline.Label; -import com.google.devtools.build.lib.concurrent.ThreadSafety.Immutable; -import com.google.devtools.build.lib.skyframe.serialization.autocodec.AutoCodec; -import javax.annotation.Nullable; - -/** - * Provides the binary artifact and its associated .dwp files, if fission is enabled. If Fission - * ({@url https://gcc.gnu.org/wiki/DebugFission}) is not enabled, the dwp file will be null. - */ -@Immutable -@AutoCodec -public final class CppDebugPackageProvider implements TransitiveInfoProvider { - private final Label targetLabel; - private final Artifact strippedArtifact; - private final Artifact unstrippedArtifact; - @Nullable private final Artifact dwpArtifact; - - @AutoCodec.Instantiator - public CppDebugPackageProvider( - Label targetLabel, - Artifact strippedArtifact, - Artifact unstrippedArtifact, - @Nullable Artifact dwpArtifact) { - Preconditions.checkNotNull(strippedArtifact); - Preconditions.checkNotNull(unstrippedArtifact); - this.targetLabel = targetLabel; - this.strippedArtifact = strippedArtifact; - this.unstrippedArtifact = unstrippedArtifact; - this.dwpArtifact = dwpArtifact; - } - - /** - * Returns the label for the cc_binary target. - */ - public final Label getTargetLabel() { - return targetLabel; - } - - /** - * Returns the stripped file (the explicit ".stripped" target). - */ - public final Artifact getStrippedArtifact() { - return strippedArtifact; - } - - /** - * Returns the unstripped file (the default executable target). - */ - public final Artifact getUnstrippedArtifact() { - return unstrippedArtifact; - } - - /** - * Returns the .dwp file (for fission builds) or null if --fission=no. - */ - @Nullable - public final Artifact getDwpArtifact() { - return dwpArtifact; - } - -} diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/DebugPackageProvider.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/DebugPackageProvider.java new file mode 100644 index 0000000000..f287c5e719 --- /dev/null +++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/DebugPackageProvider.java @@ -0,0 +1,77 @@ +// Copyright 2014 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.rules.cpp; + +import com.google.common.base.Preconditions; +import com.google.devtools.build.lib.actions.Artifact; +import com.google.devtools.build.lib.analysis.TransitiveInfoProvider; +import com.google.devtools.build.lib.cmdline.Label; +import com.google.devtools.build.lib.concurrent.ThreadSafety.Immutable; +import com.google.devtools.build.lib.skyframe.serialization.autocodec.AutoCodec; +import javax.annotation.Nullable; + +/** + * Provides the binary artifact and its associated .dwp files, if fission is enabled. If Fission + * ({@url https://gcc.gnu.org/wiki/DebugFission}) is not enabled, the dwp file will be null. + */ +@Immutable +@AutoCodec +public final class DebugPackageProvider implements TransitiveInfoProvider { + private final Label targetLabel; + private final Artifact strippedArtifact; + private final Artifact unstrippedArtifact; + @Nullable private final Artifact dwpArtifact; + + @AutoCodec.Instantiator + public DebugPackageProvider( + Label targetLabel, + @Nullable Artifact strippedArtifact, + Artifact unstrippedArtifact, + @Nullable Artifact dwpArtifact) { + Preconditions.checkNotNull(unstrippedArtifact); + this.targetLabel = targetLabel; + this.strippedArtifact = strippedArtifact; + this.unstrippedArtifact = unstrippedArtifact; + this.dwpArtifact = dwpArtifact; + } + + /** Returns the label for the *_binary target. */ + public final Label getTargetLabel() { + return targetLabel; + } + + /** + * Returns the stripped file (the explicit ".stripped" target). + */ + public final Artifact getStrippedArtifact() { + return strippedArtifact; + } + + /** + * Returns the unstripped file (the default executable target). + */ + public final Artifact getUnstrippedArtifact() { + return unstrippedArtifact; + } + + /** + * Returns the .dwp file (for fission builds) or null if --fission=no. + */ + @Nullable + public final Artifact getDwpArtifact() { + return dwpArtifact; + } + +} -- cgit v1.2.3