RELNOTES[INC]: Set stamping to false by default (i.e., --nostamp)

Stamping make build non-deterministic and should not be the default.

--
MOS_MIGRATED_REVID=113040937

2 files changed, 3 insertions, 1 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/analysis/config/BuildConfiguration.java b/src/main/java/com/google/devtools/build/lib/analysis/config/BuildConfiguration.java
index 3f0933032b..cb094c3017 100644
--- a/src/main/java/com/google/devtools/build/lib/analysis/config/BuildConfiguration.java
+++ b/src/main/java/com/google/devtools/build/lib/analysis/config/BuildConfiguration.java
@@ -540,7 +540,7 @@ public final class BuildConfiguration {
     public List<Map.Entry<String, String>> pluginCoptList;
 
     @Option(name = "stamp",
-        defaultValue = "true",
+        defaultValue = "false",
         category = "semantics",
         help = "Stamp binaries with the date, username, hostname, workspace information, etc.")
     public boolean stampBinaries;
diff --git a/src/test/java/com/google/devtools/build/lib/analysis/util/BuildViewTestCase.java b/src/test/java/com/google/devtools/build/lib/analysis/util/BuildViewTestCase.java
index 32f8027723..05a5fbe06d 100644
--- a/src/test/java/com/google/devtools/build/lib/analysis/util/BuildViewTestCase.java
+++ b/src/test/java/com/google/devtools/build/lib/analysis/util/BuildViewTestCase.java
@@ -269,6 +269,8 @@ public abstract class BuildViewTestCase extends FoundationTestCase {
           ruleClassProvider.getConfigurationOptions()));
     try {
       List<String> configurationArgs = new ArrayList<>();
+      // TODO(dmarting): Add --stamp option only to test that requires it.
+      configurationArgs.add("--stamp");  // Stamp is now defaulted to false.
       configurationArgs.add("--experimental_extended_sanity_checks");
       configurationArgs.addAll(getAnalysisMock().getOptionOverrides());