Fix up the exit status for description fetch failures.

google · May 21, 2020 · 146a8b6 · 146a8b6
1 parent a2c66a4
commit 146a8b6
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/Source/Tools/GenerateCheckedInServices b/Source/Tools/GenerateCheckedInServices
@@ -2,7 +2,7 @@
 # This script does work currently needed to regenerate the sources found
 # in Source/GeneratedServices.
 
-set -e
+set -eu
 
 # Where does this script live so we can hit things in the tree.
 readonly ToolsDir=$(dirname "$(echo $0 | sed -e "s,^\([^/]\),$(pwd)/\1,")")

diff --git a/Source/Tools/ServiceGenerator/SGMain.m b/Source/Tools/ServiceGenerator/SGMain.m
@@ -347,7 +347,6 @@ - (instancetype)initWithArgc:(int)argc argv:(char * const *)argv {
 }
 
 - (int)run {
-
   while (self.state != SGMain_Done) {
     @autoreleasepool {
       switch (self.state) {
@@ -685,6 +684,7 @@ - (BOOL)collectAPIFromURL:(NSURL *)url
       if (parseErr) {
         [self reportError:@"(reply was JSON, but it failed to parse: %@)", parseErr];
       }
+      self.status = 14;
       self.state = SGMain_Done;
       return;
     }