Add generator option for packed structs.
[apps/agl-service-can-low-level.git] / generator / nanopb_generator.py
1 '''Generate header file for nanopb from a ProtoBuf FileDescriptorSet.'''
2 nanopb_version = "nanopb-0.1.9-dev"
3
4 try:
5     import google.protobuf.descriptor_pb2 as descriptor
6 except:
7     print
8     print "*************************************************************"
9     print "*** Could not import the Google protobuf Python libraries ***"
10     print "*** Try installing package 'python-protobuf' or similar.  ***"
11     print "*************************************************************"
12     print
13     raise
14
15 try:
16     import nanopb_pb2
17 except:
18     print
19     print "***************************************************************"
20     print "*** Could not import the precompiled nanopb_pb2.py.         ***"
21     print "*** Run 'make' in the 'generator' folder to update the file.***"
22     print "***************************************************************"
23     print
24     raise
25
26
27
28
29
30
31 # ---------------------------------------------------------------------------
32 #                     Generation of single fields
33 # ---------------------------------------------------------------------------
34
35 import time
36 import os.path
37
38 # Values are tuple (c type, pb ltype)
39 FieldD = descriptor.FieldDescriptorProto
40 datatypes = {
41     FieldD.TYPE_BOOL: ('bool', 'PB_LTYPE_VARINT'),
42     FieldD.TYPE_DOUBLE: ('double', 'PB_LTYPE_FIXED64'),
43     FieldD.TYPE_FIXED32: ('uint32_t', 'PB_LTYPE_FIXED32'),
44     FieldD.TYPE_FIXED64: ('uint64_t', 'PB_LTYPE_FIXED64'),
45     FieldD.TYPE_FLOAT: ('float', 'PB_LTYPE_FIXED32'),
46     FieldD.TYPE_INT32: ('int32_t', 'PB_LTYPE_VARINT'),
47     FieldD.TYPE_INT64: ('int64_t', 'PB_LTYPE_VARINT'),
48     FieldD.TYPE_SFIXED32: ('int32_t', 'PB_LTYPE_FIXED32'),
49     FieldD.TYPE_SFIXED64: ('int64_t', 'PB_LTYPE_FIXED64'),
50     FieldD.TYPE_SINT32: ('int32_t', 'PB_LTYPE_SVARINT'),
51     FieldD.TYPE_SINT64: ('int64_t', 'PB_LTYPE_SVARINT'),
52     FieldD.TYPE_UINT32: ('uint32_t', 'PB_LTYPE_VARINT'),
53     FieldD.TYPE_UINT64: ('uint64_t', 'PB_LTYPE_VARINT')
54 }
55
56 class Names:
57     '''Keeps a set of nested names and formats them to C identifier.
58     You can subclass this with your own implementation.
59     '''
60     def __init__(self, parts = ()):
61         if isinstance(parts, Names):
62             parts = parts.parts
63         self.parts = tuple(parts)
64     
65     def __str__(self):
66         return '_'.join(self.parts)
67
68     def __add__(self, other):
69         if isinstance(other, (str, unicode)):
70             return Names(self.parts + (other,))
71         elif isinstance(other, tuple):
72             return Names(self.parts + other)
73         else:
74             raise ValueError("Name parts should be of type str")
75     
76     def __eq__(self, other):
77         return isinstance(other, Names) and self.parts == other.parts
78     
79 def names_from_type_name(type_name):
80     '''Parse Names() from FieldDescriptorProto type_name'''
81     if type_name[0] != '.':
82         raise NotImplementedError("Lookup of non-absolute type names is not supported")
83     return Names(type_name[1:].split('.'))
84
85 class Enum:
86     def __init__(self, names, desc, enum_options):
87         '''desc is EnumDescriptorProto'''
88         
89         self.options = enum_options
90         self.names = names + desc.name
91         
92         if enum_options.long_names:
93             self.values = [(self.names + x.name, x.number) for x in desc.value]            
94         else:
95             self.values = [(names + x.name, x.number) for x in desc.value] 
96         
97         self.value_longnames = [self.names + x.name for x in desc.value]
98     
99     def __str__(self):
100         result = 'typedef enum _%s {\n' % self.names
101         result += ',\n'.join(["    %s = %d" % x for x in self.values])
102         result += '\n} %s;' % self.names
103         return result
104
105 class Field:
106     def __init__(self, struct_name, desc, field_options):
107         '''desc is FieldDescriptorProto'''
108         self.tag = desc.number
109         self.struct_name = struct_name
110         self.name = desc.name
111         self.default = None
112         self.max_size = None
113         self.max_count = None
114         self.array_decl = ""
115         
116         # Parse field options
117         if field_options.HasField("max_size"):
118             self.max_size = field_options.max_size
119         
120         if field_options.HasField("max_count"):
121             self.max_count = field_options.max_count
122         
123         if desc.HasField('default_value'):
124             self.default = desc.default_value
125            
126         # Decide HTYPE
127         # HTYPE is the high-order nibble of nanopb field description,
128         # defining whether value is required/optional/repeated.
129         can_be_static = True
130         if desc.label == FieldD.LABEL_REQUIRED:
131             self.htype = 'PB_HTYPE_REQUIRED'
132         elif desc.label == FieldD.LABEL_OPTIONAL:
133             self.htype = 'PB_HTYPE_OPTIONAL'
134         elif desc.label == FieldD.LABEL_REPEATED:
135             if self.max_count is None:
136                 can_be_static = False
137             else:
138                 self.htype = 'PB_HTYPE_ARRAY'
139                 self.array_decl = '[%d]' % self.max_count
140         else:
141             raise NotImplementedError(desc.label)
142         
143         # Decide LTYPE and CTYPE
144         # LTYPE is the low-order nibble of nanopb field description,
145         # defining how to decode an individual value.
146         # CTYPE is the name of the c type to use in the struct.
147         if datatypes.has_key(desc.type):
148             self.ctype, self.ltype = datatypes[desc.type]
149         elif desc.type == FieldD.TYPE_ENUM:
150             self.ltype = 'PB_LTYPE_VARINT'
151             self.ctype = names_from_type_name(desc.type_name)
152             if self.default is not None:
153                 self.default = self.ctype + self.default
154         elif desc.type == FieldD.TYPE_STRING:
155             self.ltype = 'PB_LTYPE_STRING'
156             if self.max_size is None:
157                 can_be_static = False
158             else:
159                 self.ctype = 'char'
160                 self.array_decl += '[%d]' % self.max_size
161         elif desc.type == FieldD.TYPE_BYTES:
162             self.ltype = 'PB_LTYPE_BYTES'
163             if self.max_size is None:
164                 can_be_static = False
165             else:
166                 self.ctype = self.struct_name + self.name + 't'
167         elif desc.type == FieldD.TYPE_MESSAGE:
168             self.ltype = 'PB_LTYPE_SUBMESSAGE'
169             self.ctype = self.submsgname = names_from_type_name(desc.type_name)
170         else:
171             raise NotImplementedError(desc.type)
172         
173         if field_options.type == nanopb_pb2.FT_DEFAULT:
174             if can_be_static:
175                 field_options.type = nanopb_pb2.FT_STATIC
176             else:
177                 field_options.type = nanopb_pb2.FT_CALLBACK
178         
179         if field_options.type == nanopb_pb2.FT_STATIC and not can_be_static:
180             raise Exception("Field %s is defined as static, but max_size or max_count is not given." % self.name)
181         
182         if field_options.type == nanopb_pb2.FT_CALLBACK:
183             self.htype = 'PB_HTYPE_CALLBACK'
184             self.ctype = 'pb_callback_t'
185             self.array_decl = ''
186     
187     def __cmp__(self, other):
188         return cmp(self.tag, other.tag)
189     
190     def __str__(self):
191         if self.htype == 'PB_HTYPE_OPTIONAL':
192             result = '    bool has_' + self.name + ';\n'
193         elif self.htype == 'PB_HTYPE_ARRAY':
194             result = '    size_t ' + self.name + '_count;\n'
195         else:
196             result = ''
197         result += '    %s %s%s;' % (self.ctype, self.name, self.array_decl)
198         return result
199     
200     def types(self):
201         '''Return definitions for any special types this field might need.'''
202         if self.ltype == 'PB_LTYPE_BYTES' and self.max_size is not None:
203             result = 'typedef struct {\n'
204             result += '    size_t size;\n'
205             result += '    uint8_t bytes[%d];\n' % self.max_size
206             result += '} %s;\n' % self.ctype
207         else:
208             result = None
209         return result
210     
211     def default_decl(self, declaration_only = False):
212         '''Return definition for this field's default value.'''
213         if self.default is None:
214             return None
215         
216         if self.ltype == 'PB_LTYPE_STRING':
217             ctype = 'char'
218             if self.max_size is None:
219                 return None # Not implemented
220             else:
221                 array_decl = '[%d]' % (self.max_size + 1)
222             default = str(self.default).encode('string_escape')
223             default = default.replace('"', '\\"')
224             default = '"' + default + '"'
225         elif self.ltype == 'PB_LTYPE_BYTES':
226             data = self.default.decode('string_escape')
227             data = ['0x%02x' % ord(c) for c in data]
228             
229             if self.max_size is None:
230                 return None # Not implemented
231             else:
232                 ctype = self.ctype
233             
234             default = '{%d, {%s}}' % (len(data), ','.join(data))
235             array_decl = ''
236         else:
237             ctype, default = self.ctype, self.default
238             array_decl = ''
239         
240         if declaration_only:
241             return 'extern const %s %s_default%s;' % (ctype, self.struct_name + self.name, array_decl)
242         else:
243             return 'const %s %s_default%s = %s;' % (ctype, self.struct_name + self.name, array_decl, default)
244     
245     def pb_field_t(self, prev_field_name):
246         '''Return the pb_field_t initializer to use in the constant array.
247         prev_field_name is the name of the previous field or None.
248         '''
249         result = '    {%d, ' % self.tag
250         result += '(pb_type_t) ((int) ' + self.htype
251         if self.ltype is not None:
252             result += ' | (int) ' + self.ltype
253         result += '),\n'
254         
255         if prev_field_name is None:
256             result += '    offsetof(%s, %s),' % (self.struct_name, self.name)
257         else:
258             result += '    pb_delta_end(%s, %s, %s),' % (self.struct_name, self.name, prev_field_name)
259         
260         if self.htype == 'PB_HTYPE_OPTIONAL':
261             result += '\n    pb_delta(%s, has_%s, %s),' % (self.struct_name, self.name, self.name)
262         elif self.htype == 'PB_HTYPE_ARRAY':
263             result += '\n    pb_delta(%s, %s_count, %s),' % (self.struct_name, self.name, self.name)
264         else:
265             result += ' 0,'
266         
267         
268         if self.htype == 'PB_HTYPE_ARRAY':
269             result += '\n    pb_membersize(%s, %s[0]),' % (self.struct_name, self.name)
270             result += ('\n    pb_membersize(%s, %s) / pb_membersize(%s, %s[0]),'
271                        % (self.struct_name, self.name, self.struct_name, self.name))
272         else:
273             result += '\n    pb_membersize(%s, %s),' % (self.struct_name, self.name)
274             result += ' 0,'
275         
276         if self.ltype == 'PB_LTYPE_SUBMESSAGE':
277             result += '\n    &%s_fields}' % self.submsgname
278         elif self.default is None or self.htype == 'PB_HTYPE_CALLBACK':
279             result += ' 0}'
280         else:
281             result += '\n    &%s_default}' % (self.struct_name + self.name)
282         
283         return result
284     
285     def largest_field_value(self):
286         '''Determine if this field needs 16bit or 32bit pb_field_t structure to compile properly.
287         Returns numeric value or a C-expression for assert.'''
288         if self.ltype == 'PB_LTYPE_SUBMESSAGE':
289             if self.htype == 'PB_HTYPE_ARRAY':
290                 return 'pb_membersize(%s, %s[0])' % (self.struct_name, self.name)
291             else:
292                 return 'pb_membersize(%s, %s)' % (self.struct_name, self.name)
293
294         return max(self.tag, self.max_size, self.max_count)        
295
296
297
298
299
300
301 # ---------------------------------------------------------------------------
302 #                   Generation of messages (structures)
303 # ---------------------------------------------------------------------------
304
305
306 class Message:
307     def __init__(self, names, desc, message_options):
308         self.name = names
309         self.fields = [Field(self.name, f, get_nanopb_suboptions(f, message_options)) for f in desc.field]
310         self.packed = message_options.packed_struct
311         self.ordered_fields = self.fields[:]
312         self.ordered_fields.sort()
313
314     def get_dependencies(self):
315         '''Get list of type names that this structure refers to.'''
316         return [str(field.ctype) for field in self.fields]
317     
318     def __str__(self):
319         result = 'typedef struct _%s {\n' % self.name
320         result += '\n'.join([str(f) for f in self.ordered_fields])
321         result += '\n}'
322         
323         if self.packed:
324             result += ' pb_packed'
325         
326         result += ' %s;' % self.name
327         return result
328     
329     def types(self):
330         result = ""
331         for field in self.fields:
332             types = field.types()
333             if types is not None:
334                 result += types + '\n'
335         return result
336     
337     def default_decl(self, declaration_only = False):
338         result = ""
339         for field in self.fields:
340             default = field.default_decl(declaration_only)
341             if default is not None:
342                 result += default + '\n'
343         return result
344
345     def fields_declaration(self):
346         result = 'extern const pb_field_t %s_fields[%d];' % (self.name, len(self.fields) + 1)
347         return result
348
349     def fields_definition(self):
350         result = 'const pb_field_t %s_fields[%d] = {\n' % (self.name, len(self.fields) + 1)
351         
352         prev = None
353         for field in self.ordered_fields:
354             result += field.pb_field_t(prev)
355             result += ',\n\n'
356             prev = field.name
357         
358         result += '    PB_LAST_FIELD\n};'
359         return result
360
361
362
363
364
365
366 # ---------------------------------------------------------------------------
367 #                    Processing of entire .proto files
368 # ---------------------------------------------------------------------------
369
370
371 def iterate_messages(desc, names = Names()):
372     '''Recursively find all messages. For each, yield name, DescriptorProto.'''
373     if hasattr(desc, 'message_type'):
374         submsgs = desc.message_type
375     else:
376         submsgs = desc.nested_type
377     
378     for submsg in submsgs:
379         sub_names = names + submsg.name
380         yield sub_names, submsg
381         
382         for x in iterate_messages(submsg, sub_names):
383             yield x
384
385 def parse_file(fdesc, file_options):
386     '''Takes a FileDescriptorProto and returns tuple (enum, messages).'''
387     
388     enums = []
389     messages = []
390     
391     if fdesc.package:
392         base_name = Names(fdesc.package.split('.'))
393     else:
394         base_name = Names()
395     
396     for enum in fdesc.enum_type:
397         enum_options = get_nanopb_suboptions(enum, file_options)
398         enums.append(Enum(base_name, enum, enum_options))
399     
400     for names, message in iterate_messages(fdesc, base_name):
401         message_options = get_nanopb_suboptions(message, file_options)
402         messages.append(Message(names, message, message_options))
403         for enum in message.enum_type:
404             enum_options = get_nanopb_suboptions(enum, message_options)
405             enums.append(Enum(names, enum, enum_options))
406     
407     # Fix field default values where enum short names are used.
408     for enum in enums:
409         if not enum.options.long_names:
410             for message in messages:
411                 for field in message.fields:
412                     if field.default in enum.value_longnames:
413                         idx = enum.value_longnames.index(field.default)
414                         field.default = enum.values[idx][0]
415     
416     return enums, messages
417
418 def toposort2(data):
419     '''Topological sort.
420     From http://code.activestate.com/recipes/577413-topological-sort/
421     This function is under the MIT license.
422     '''
423     for k, v in data.items():
424         v.discard(k) # Ignore self dependencies
425     extra_items_in_deps = reduce(set.union, data.values(), set()) - set(data.keys())
426     data.update(dict([(item, set()) for item in extra_items_in_deps]))
427     while True:
428         ordered = set(item for item,dep in data.items() if not dep)
429         if not ordered:
430             break
431         for item in sorted(ordered):
432             yield item
433         data = dict([(item, (dep - ordered)) for item,dep in data.items()
434                 if item not in ordered])
435     assert not data, "A cyclic dependency exists amongst %r" % data
436
437 def sort_dependencies(messages):
438     '''Sort a list of Messages based on dependencies.'''
439     dependencies = {}
440     message_by_name = {}
441     for message in messages:
442         dependencies[str(message.name)] = set(message.get_dependencies())
443         message_by_name[str(message.name)] = message
444     
445     for msgname in toposort2(dependencies):
446         if msgname in message_by_name:
447             yield message_by_name[msgname]
448
449 def generate_header(dependencies, headername, enums, messages):
450     '''Generate content for a header file.
451     Generates strings, which should be concatenated and stored to file.
452     '''
453     
454     yield '/* Automatically generated nanopb header */\n'
455     yield '/* Generated by %s at %s. */\n\n' % (nanopb_version, time.asctime())
456     
457     symbol = headername.replace('.', '_').upper()
458     yield '#ifndef _PB_%s_\n' % symbol
459     yield '#define _PB_%s_\n' % symbol
460     yield '#include <pb.h>\n\n'
461     
462     for dependency in dependencies:
463         noext = os.path.splitext(dependency)[0]
464         yield '#include "%s.pb.h"\n' % noext
465     
466     yield '#ifdef __cplusplus\n'
467     yield 'extern "C" {\n'
468     yield '#endif\n\n'
469     
470     yield '/* Enum definitions */\n'
471     for enum in enums:
472         yield str(enum) + '\n\n'
473     
474     yield '/* Struct definitions */\n'
475     for msg in sort_dependencies(messages):
476         yield msg.types()
477         yield str(msg) + '\n\n'
478         
479     yield '/* Default values for struct fields */\n'
480     for msg in messages:
481         yield msg.default_decl(True)
482     yield '\n'
483     
484     yield '/* Struct field encoding specification for nanopb */\n'
485     for msg in messages:
486         yield msg.fields_declaration() + '\n'
487     
488     if messages:
489         count_required_fields = lambda m: len([f for f in msg.fields if f.htype == 'PB_HTYPE_REQUIRED'])
490         largest_msg = max(messages, key = count_required_fields)
491         largest_count = count_required_fields(largest_msg)
492         if largest_count > 64:
493             yield '\n/* Check that missing required fields will be properly detected */\n'
494             yield '#if PB_MAX_REQUIRED_FIELDS < %d\n' % largest_count
495             yield '#error Properly detecting missing required fields in %s requires \\\n' % largest_msg.name
496             yield '       setting PB_MAX_REQUIRED_FIELDS to %d or more.\n' % largest_count
497             yield '#endif\n'
498     
499     worst = 0
500     worst_field = ''
501     checks = []
502     checks_msgnames = []
503     for msg in messages:
504         checks_msgnames.append(msg.name)
505         for field in msg.fields:
506             status = field.largest_field_value()
507             if isinstance(status, (str, unicode)):
508                 checks.append(status)
509             elif status > worst:
510                 worst = status
511                 worst_field = str(field.struct_name) + '.' + str(field.name)
512
513     if worst > 255 or checks:
514         yield '\n/* Check that field information fits in pb_field_t */\n'
515         
516         if worst < 65536:
517             yield '#if !defined(PB_FIELD_16BIT) && !defined(PB_FIELD_32BIT)\n'
518             if worst > 255:
519                 yield '#error Field descriptor for %s is too large. Define PB_FIELD_16BIT to fix this.\n' % worst_field
520             else:
521                 assertion = ' && '.join(str(c) + ' < 256' for c in checks)
522                 msgs = '_'.join(str(n) for n in checks_msgnames)
523                 yield 'STATIC_ASSERT((%s), YOU_MUST_DEFINE_PB_FIELD_16BIT_FOR_MESSAGES_%s)\n'%(assertion,msgs)
524             yield '#endif\n\n'
525         
526         if worst > 65535 or checks:
527             yield '#if !defined(PB_FIELD_32BIT)\n'
528             if worst > 65535:
529                 yield '#error Field descriptor for %s is too large. Define PB_FIELD_32BIT to fix this.\n' % worst_field
530             else:
531                 assertion = ' && '.join(str(c) + ' < 65536' for c in checks)
532                 msgs = '_'.join(str(n) for n in checks_msgnames)
533                 yield 'STATIC_ASSERT((%s), YOU_MUST_DEFINE_PB_FIELD_32BIT_FOR_MESSAGES_%s)\n'%(assertion,msgs)
534             yield '#endif\n'
535     
536     yield '\n#ifdef __cplusplus\n'
537     yield '} /* extern "C" */\n'
538     yield '#endif\n'
539     
540     # End of header
541     yield '\n#endif\n'
542
543 def generate_source(headername, enums, messages):
544     '''Generate content for a source file.'''
545     
546     yield '/* Automatically generated nanopb constant definitions */\n'
547     yield '/* Generated by %s at %s. */\n\n' % (nanopb_version, time.asctime())
548     yield '#include "%s"\n\n' % headername
549     
550     for msg in messages:
551         yield msg.default_decl(False)
552     
553     yield '\n\n'
554     
555     for msg in messages:
556         yield msg.fields_definition() + '\n\n'
557
558
559
560 # ---------------------------------------------------------------------------
561 #                         Command line interface
562 # ---------------------------------------------------------------------------
563
564 import sys
565 import os.path    
566 from optparse import OptionParser
567 import google.protobuf.text_format as text_format
568
569 optparser = OptionParser(
570     usage = "Usage: nanopb_generator.py [options] file.pb ...",
571     epilog = "Compile file.pb from file.proto by: 'protoc -ofile.pb file.proto'. " +
572              "Output will be written to file.pb.h and file.pb.c.")
573 optparser.add_option("-x", dest="exclude", metavar="FILE", action="append", default=[],
574     help="Exclude file from generated #include list.")
575 optparser.add_option("-q", "--quiet", dest="quiet", action="store_true", default=False,
576     help="Don't print anything except errors.")
577 optparser.add_option("-v", "--verbose", dest="verbose", action="store_true", default=False,
578     help="Print more information.")
579 optparser.add_option("-s", dest="settings", metavar="OPTION:VALUE", action="append", default=[],
580     help="Set generator option (max_size, max_count etc.).")
581
582 def get_nanopb_suboptions(subdesc, options):
583     '''Get copy of options, and merge information from subdesc.'''
584     new_options = nanopb_pb2.NanoPBOptions()
585     new_options.CopyFrom(options)
586     
587     if isinstance(subdesc.options, descriptor.FieldOptions):
588         ext_type = nanopb_pb2.nanopb
589     elif isinstance(subdesc.options, descriptor.FileOptions):
590         ext_type = nanopb_pb2.nanopb_fileopt
591     elif isinstance(subdesc.options, descriptor.MessageOptions):
592         ext_type = nanopb_pb2.nanopb_msgopt
593     elif isinstance(subdesc.options, descriptor.EnumOptions):
594         ext_type = nanopb_pb2.nanopb_enumopt
595     else:
596         raise Exception("Unknown options type")
597     
598     if subdesc.options.HasExtension(ext_type):
599         ext = subdesc.options.Extensions[ext_type]
600         new_options.MergeFrom(ext)
601     
602     return new_options
603
604 def process(filenames, options):
605     '''Process the files given on the command line.'''
606     
607     if not filenames:
608         optparser.print_help()
609         return False
610     
611     if options.quiet:
612         options.verbose = False
613     
614     toplevel_options = nanopb_pb2.NanoPBOptions()
615     for s in options.settings:
616         text_format.Merge(s, toplevel_options)
617     
618     for filename in filenames:
619         data = open(filename, 'rb').read()
620         fdesc = descriptor.FileDescriptorSet.FromString(data)
621         
622         file_options = get_nanopb_suboptions(fdesc.file[0], toplevel_options)
623         
624         if options.verbose:
625             print "Options for " + filename + ":"
626             print text_format.MessageToString(file_options)
627         
628         enums, messages = parse_file(fdesc.file[0], file_options)
629         
630         noext = os.path.splitext(filename)[0]
631         headername = noext + '.pb.h'
632         sourcename = noext + '.pb.c'
633         headerbasename = os.path.basename(headername)
634         
635         if not options.quiet:
636             print "Writing to " + headername + " and " + sourcename
637         
638         # List of .proto files that should not be included in the C header file
639         # even if they are mentioned in the source .proto.
640         excludes = ['nanopb.proto', 'google/protobuf/descriptor.proto'] + options.exclude
641         dependencies = [d for d in fdesc.file[0].dependency if d not in excludes]
642         
643         header = open(headername, 'w')
644         for part in generate_header(dependencies, headerbasename, enums, messages):
645             header.write(part)
646
647         source = open(sourcename, 'w')
648         for part in generate_source(headerbasename, enums, messages):
649             source.write(part)
650
651     return True
652
653 if __name__ == '__main__':
654     options, filenames = optparser.parse_args()
655     status = process(filenames, options)
656     
657     if not status:
658         sys.exit(1)
659