refine createimage to return None zero if failed

Change-Id: Idf0ba080fb2c32ec5f0ece3c35021fbe84399c90

diff --git a/gitbuildsys/cmd_createimage.py b/gitbuildsys/cmd_createimage.py
index a88301d..f7fa60c 100644 (file)
--- a/gitbuildsys/cmd_createimage.py
+++ b/gitbuildsys/cmd_createimage.py
@@ -34,13 +34,17 @@ def createimage(args, ks_file):
     extra_mic_opts += ['--record-pkgs=name']
     mic_cmd = 'sudo mic create auto %s %s' % (ks_file, ' '.join(extra_mic_opts))
     log.debug(mic_cmd)
-    os.system(mic_cmd)
+    return os.system(mic_cmd)
 
 def main(args):
     '''main entrance for createimage'''
 
-    if args.ks_file:
-        if not os.path.exists(args.ks_file):
-            raise GbsError('specified ks file %s does not exist' % args.ks_file)
-        log.info('creating image for ks file: %s' % args.ks_file)
-        createimage(args, args.ks_file)
+    if not os.path.exists(args.ks_file):
+        raise GbsError('specified ks file %s does not exist' % args.ks_file)
+
+    log.info('creating image for ks file: %s' % args.ks_file)
+    retcode = createimage(args, args.ks_file)
+    if retcode != 0:
+        raise GbsError('failed to create image')
+    else:
+        log.info('Done')

diff --git a/tools/gbs b/tools/gbs
index 823c292..162c4a4 100755 (executable)
--- a/tools/gbs
+++ b/tools/gbs
@@ -263,10 +263,13 @@ def build_parser(parser):
 
 @subparser
 def createimage_parser(parser):
-    """create image using ks file """
-
-    parser.add_argument('-K', '--ks-file',
-                        help='ks file to be used')
+    """create image using ks file
+    Examples:
+      $ gbs createimage --ks-file=path/to/ks-file
+      $ gbs createimage --ks-file=path/to/ks-file --tmpfs
+    """
+    parser.add_argument('-K', '--ks-file', required=True,
+                        help='ks file to be used for image creation')
     parser.add_argument('-O', '--outdir',
                         help='image output directory')
     parser.add_argument('--tmpfs', action='store_true',