commit 97216ab72e8fbadbbb75004192ea5da8e9629646
author Petteri Aimonen <jpa@git.mail.kapsi.fi> Mon Jun 22 12:35:21 2020 +0300
committer Petteri Aimonen <jpa@git.mail.kapsi.fi> Mon Jun 22 12:35:21 2020 +0300
tree 54ddacd34c4249102b524a73dccd5c766e03d8bb
parent 6062e2d396aae36d8bcbb49e49ed0b7aa97759c9

Improve generator error handling

generator/nanopb_generator.py

diff --git a/generator/nanopb_generator.py b/generator/nanopb_generator.py
index 6afea96..a6e0d31 100755
--- a/generator/nanopb_generator.py
+++ b/generator/nanopb_generator.py
@@ -1747,7 +1747,7 @@
             sys.stderr.write("%s:%d: " % (infile.name, i + 1) +
                              "Option lines should have space between field name and options. " +
                              "Skipping line: '%s'\n" % line)
-            continue
+            sys.exit(1)
 
         opts = nanopb_pb2.NanoPBOptions()
 
@@ -1757,7 +1757,7 @@
             sys.stderr.write("%s:%d: " % (infile.name, i + 1) +
                              "Unparseable option line: '%s'. " % line +
                              "Error: %s\n" % str(e))
-            continue
+            sys.exit(1)
         results.append((parts[0], opts))
 
     return results
@@ -1984,7 +1984,8 @@
         if filename.endswith(".proto"):
             with TemporaryDirectory() as tmpdir:
                 tmpname = os.path.join(tmpdir, os.path.basename(filename) + ".pb")
-                invoke_protoc(["protoc"] + include_path + ['--include_imports', '-o' + tmpname, filename])
+                status = invoke_protoc(["protoc"] + include_path + ['--include_imports', '-o' + tmpname, filename])
+                if status != 0: sys.exit(status)
                 data = open(tmpname, 'rb').read()
         else:
             data = open(filename, 'rb').read()