Include a trailing newline in printStatusMap output

This makes it harder to screw up when doing things like concatenating
bazel-out/{stable,volatile}-status.txt in a genrule, and makes console
output slightly nicer when you print them to stdout.

--
Change-Id: Ib558ddcb09e66452a4780355716d9d6d1d143355
Reviewed-on: https://cr.bazel.build/7190
PiperOrigin-RevId: 141044546
MOS_MIGRATED_REVID=141044546

1 files changed, 13 insertions, 7 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/bazel/BazelWorkspaceStatusModule.java b/src/main/java/com/google/devtools/build/lib/bazel/BazelWorkspaceStatusModule.java
index cc8c96d826..4282c762e6 100644
--- a/src/main/java/com/google/devtools/build/lib/bazel/BazelWorkspaceStatusModule.java
+++ b/src/main/java/com/google/devtools/build/lib/bazel/BazelWorkspaceStatusModule.java
@@ -150,13 +150,19 @@ public class BazelWorkspaceStatusModule extends BlazeModule {
     }
 
     private static byte[] printStatusMap(Map<String, String> map) {
-      return Joiner.on("\n").join(Iterables.transform(map.entrySet(),
-          new Function<Map.Entry<String, String>, String>() {
-            @Override
-            public String apply(@Nullable Entry<String, String> entry) {
-              return entry.getKey() + " " + entry.getValue();
-            }
-          })).getBytes(StandardCharsets.UTF_8);
+      String s =
+          Joiner.on("\n")
+              .join(
+                  Iterables.transform(
+                      map.entrySet(),
+                      new Function<Map.Entry<String, String>, String>() {
+                        @Override
+                        public String apply(@Nullable Entry<String, String> entry) {
+                          return entry.getKey() + " " + entry.getValue();
+                        }
+                      }));
+      s += "\n";
+      return s.getBytes(StandardCharsets.UTF_8);
     }
 
     @Override