Use subprocess.check_call

[mass-build.git] / mass_build.py

diff --git a/mass_build.py b/mass_build.py
index 800917d558bd3c6db6c9023fa8f4af0629cc5b89..e9e372530966463cfc4165f391bf60570764f8e5 100755 (executable)
--- a/mass_build.py
+++ b/mass_build.py
@@ -1,4 +1,21 @@
-#!/usr/bin/python
+#!/usr/bin/python3
+# mass-build - Easily Build Software Involving a Large Amount of Source Repositories
+# Copyright (C) 2012-2013 Ralf Jung <post@ralfj.de>
+#
+# This program is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 2 of the License, or
+# (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program; if not, write to the Free Software
+# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
+
 import vcs, build_system, imp
 import argparse, os, sys, subprocess
 from collections import OrderedDict
@@ -11,7 +28,7 @@ class Project:
                # VCS
                vcsName = config['vcs']
                if vcsName == 'git':
-                       self.vcs = vcs.Git(self.sourceFolder(), config['url'], config['version'])
+                       self.vcs = vcs.Git(self.sourceFolder(), config)
                elif vcsName == 'svn':
                        self.vcs = vcs.SVN(self.sourceFolder(), config['url'])
                else:
@@ -40,7 +57,7 @@ parser.add_argument("-c, --config",
 parser.add_argument("--reconfigure",
                     action="store_true", dest="reconfigure",
                     help="Force configuration to be run")
-parser.add_argument("--wait-after-config",
+parser.add_argument("-w", "--wait-after-config",
                     action="store_true", dest="wait_after_config",
                     help="Wait for user confirmation after configuration is finished")
 parser.add_argument("--reset-source",
@@ -52,16 +69,22 @@ parser.add_argument("--no-update",
 parser.add_argument("--resume-from", metavar='PROJECT',
                     dest="resume_from",
                     help="From the projects specified, continue building with this one (i.e., remove all projects before this one from the list - this never adds new projects)")
+parser.add_argument("--check-versions",
+                    action="store_true", dest="version_check",
+                    help="Check the repositories for newer tags, if possible (does not perform any building steps)")
 parser.add_argument("projects",  metavar='PROJECT', nargs='*',
                     help="Manually specify projects or folders to be built (project names take precedence)")
 args = parser.parse_args()
-# sanitize
 if args.reset_source and not args.update:
-       raise Exception("When no update is performed, no reset to the given version can be done either")
+       raise Exception("Can not reset sources without doing an update")
 
 # load config
+old_val = sys.dont_write_bytecode
+sys.dont_write_bytecode = True
 config = imp.load_source('config', args.config).__dict__
-os.remove(args.config+'c') # remove compiled python file
+sys.dont_write_bytecode = old_val
+
+# initialise variables holding the configuration
 allProjects = OrderedDict() # all projects
 allFolders = {} # all folders
 workProjects = [] # projects we work on
@@ -104,7 +127,7 @@ if args.projects:
                else:
                        raise Exception("Project or folder %s does not exist" % name)
 else:
-       workProjects = allProjects.values() # all the projects
+       workProjects = list(allProjects.values()) # all the projects
 # apply the "resume from"
 if args.resume_from is not None:
        # find project index
@@ -122,18 +145,23 @@ if args.resume_from is not None:
 # and do it!
 for project in workProjects:
        try:
-               if args.update:
-                       print "Updating project",project.sourceFolder()
-                       project.vcs.update(forceVersion=args.reset_source)
-               print "Building project",project.sourceFolder()
-               project.buildSystem.build(reconfigure=args.reconfigure, waitAfterConfig=args.wait_after_config)
-               print
+               if args.version_check:
+                       print("Checking project",project.sourceFolder())
+                       project.vcs.checkVersions()
+               else:
+                       if args.update:
+                               print("Updating project",project.sourceFolder())
+                               project.vcs.update(mode = vcs.MODE_RESET if args.reset_source else vcs.MODE_REBASE)
+                       print("Building project",project.sourceFolder())
+                       project.buildSystem.build(reconfigure=args.reconfigure, waitAfterConfig=args.wait_after_config)
+               print()
        except (subprocess.CalledProcessError, KeyboardInterrupt) as e: # for some exceptions, a stackrace is usually pointless
-               print >> sys.stderr
-               print >> sys.stderr
+               print(file=sys.stderr)
+               print(file=sys.stderr)
                if isinstance(e, KeyboardInterrupt): # str(e) would be the empty string
-                       print >> sys.stderr, "Interruped by user while processing %s" % (project.name)
+                       print("Interruped by user while processing %s" % (project.name), file=sys.stderr)
                else:
-                       print >> sys.stderr, "Error while processing %s: %s" % (project.name, str(e))
-               print >> sys.stderr
+                       print("Error while processing %s: %s" % (project.name, str(e)), file=sys.stderr)
+               print(file=sys.stderr)
                sys.exit(1)
+print("All operations successfully completed")