Fix previous.
[cdist.git] / cdist
diff --git a/cdist b/cdist
index c463cba26ff6cfa573522900273c1b7385c2ab72..c583990bb3e5cbdc5f2f2ae76139a22ab04cf95c 100755 (executable)
--- a/cdist
+++ b/cdist
@@ -1,6 +1,6 @@
 #!/usr/bin/python
 
-#    Copyright (C) 2012 Carl Hetherington <cth@carlh.net>
+#    Copyright (C) 2012-2014 Carl Hetherington <cth@carlh.net>
 #
 #    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
@@ -26,12 +26,21 @@ import datetime
 import subprocess
 import re
 import copy
+import inspect
+
+TEMPORARY_DIRECTORY = '/tmp'
+
+class Globals:
+    quiet = False
+    command = None
+
+globals = Globals()
 
 class Error(Exception):
     def __init__(self, value):
         self.value = value
     def __str__(self):
-        return '\x1b[31m%s\x1b[0m' % repr(self.value)
+        return self.value
     def __repr__(self):
         return str(self)
 
@@ -39,24 +48,39 @@ class Error(Exception):
 # Configuration
 #
 
+class Option(object):
+    def __init__(self, key, default=None):
+        self.key = key
+        self.value = default
+
+    def offer(self, key, value):
+        if key == self.key:
+            self.value = value
+
+class BoolOption(object):
+    def __init__(self, key):
+        self.key = key
+        self.value = False
+
+    def offer(self, key, value):
+        if key == self.key:
+            self.value = (value == 'yes' or value == '1' or value == 'true')
+
 class Config:
     def __init__(self):
-        self.keys = ['linux_dir_in_chroot',
-                     'linux_chroot_prefix',
-                     'windows_environment_prefix',
-                     'mingw_prefix',
-                     'git_prefix',
-                     'osx_build_host',
-                     'osx_dir_in_host',
-                     'osx_environment_prefix',
-                     'osx_sdk_prefix',
-                     'osx_sdk']
-
-        self.dict = dict()
+        self.options = [ Option('linux_chroot_prefix'),
+                         Option('windows_environment_prefix'),
+                         Option('mingw_prefix'),
+                         Option('git_prefix'),
+                         Option('osx_build_host'),
+                         Option('osx_environment_prefix'),
+                         Option('osx_sdk_prefix'),
+                         Option('osx_sdk'),
+                         Option('parallel', 4) ]
 
         try:
             f = open('%s/.config/cdist' % os.path.expanduser('~'), 'r')
-            while 1:
+            while True:
                 l = f.readline()
                 if l == '':
                     break
@@ -66,15 +90,15 @@ class Config:
 
                 s = l.strip().split()
                 if len(s) == 2:
-                    for k in self.keys:
-                        if k == s[0]:
-                            self.dict[k] = s[1]
+                    for k in self.options:
+                        k.offer(s[0], s[1])
         except:
             raise
 
     def get(self, k):
-        if k in self.dict:
-            return self.dict[k]
+        for o in self.options:
+            if o.key == k:
+                return o.value
 
         raise Error('Required setting %s not found' % k)
 
@@ -85,16 +109,25 @@ config = Config()
 # 
 
 def log(m):
-    if not args.quiet:
+    if not globals.quiet:
         print '\x1b[33m* %s\x1b[0m' % m
 
+def scp_escape(n):
+    s = n.split(':')
+    assert(len(s) == 1 or len(s) == 2)
+    if len(s) == 2:
+        s[1] = '"\'%s\'"' % s[1]
+        return '%s:%s' % (s[0], s[1])
+    else:
+        return n
+
 def copytree(a, b):
     log('copy %s -> %s' % (a, b))
-    shutil.copytree(a, b)
+    os.system('scp -r %s %s' % (scp_escape(a), scp_escape(b)))
 
 def copyfile(a, b):
     log('copy %s -> %s' % (a, b))
-    shutil.copyfile(a, b)
+    os.system('scp %s %s' % (scp_escape(a), scp_escape(b)))
 
 def rmdir(a):
     log('remove %s' % a)
@@ -118,7 +151,7 @@ def command_and_read(c):
 
 def read_wscript_variable(directory, variable):
     f = open('%s/wscript' % directory, 'r')
-    while 1:
+    while True:
         l = f.readline()
         if l == '':
             break
@@ -173,7 +206,7 @@ class Version:
         self.devel = False
 
     def __str__(self):
-        s = '%d.%02d.%d' % (self.major, self.minor, self.micro)
+        s = '%d.%d.%d' % (self.major, self.minor, self.micro)
         if self.devel:
             s += 'devel'
 
@@ -184,9 +217,19 @@ class Version:
 #
 
 class Target(object):
-    def __init__(self, platform, parallel):
+    # @param directory directory to work in; if None we will use a temporary directory
+    # Temporary directories will be removed after use; specified directories will not
+    def __init__(self, platform, directory=None):
         self.platform = platform
-        self.parallel = parallel
+        self.parallel = int(config.get('parallel'))
+
+        if directory is None:
+            self.directory = tempfile.mkdtemp('', 'tmp', TEMPORARY_DIRECTORY)
+            self.rmdir = True
+        else:
+            self.directory = directory
+            self.rmdir = False
+
         # Environment variables that we will use when we call cscripts
         self.variables = {}
         self.debug = False
@@ -194,7 +237,6 @@ class Target(object):
     def build_dependencies(self, project):
         cwd = os.getcwd()
         if 'dependencies' in project.cscript:
-            print project.cscript['dependencies'](self)
             for d in project.cscript['dependencies'](self):
                 log('Building dependency %s %s of %s' % (d[0], d[1], project.name))
                 dep = Project(d[0], '.', d[1])
@@ -216,8 +258,10 @@ class Target(object):
 
     def build(self, project, options=None):
         variables = copy.copy(self.variables)
-        print 'Target %s builds %s with %s' % (self.platform, project.name, self.variables)
-        project.cscript['build'](self, options)
+        if len(inspect.getargspec(project.cscript['build']).args) == 2:
+            project.cscript['build'](self, options)
+        else:
+            project.cscript['build'](self)
         self.variables = variables
 
     def package(self, project):
@@ -233,7 +277,6 @@ class Target(object):
         project.cscript['test'](self)
 
     def set(self, a, b):
-        print "Target set %s=%s" % (a, b)
         self.variables[a] = b
 
     def unset(self, a):
@@ -242,6 +285,12 @@ class Target(object):
     def get(self, a):
         return self.variables[a]
 
+    def append_with_space(self, k, v):
+        if not k in self.variables:
+            self.variables[k] = v
+        else:
+            self.variables[k] = '%s %s' % (self.variables[k], v)
+
     def variables_string(self, escaped_quotes=False):
         e = ''
         for k, v in self.variables.iteritems():
@@ -251,24 +300,18 @@ class Target(object):
         return e
 
     def cleanup(self):
-        pass
+        if self.rmdir:
+            rmtree(self.directory)
 
 # 
 # Windows
 #
 
 class WindowsTarget(Target):
-    # @param directory directory to work in; if None, we will use a temporary directory
     def __init__(self, bits, directory=None):
-        super(WindowsTarget, self).__init__('windows', 2)
+        super(WindowsTarget, self).__init__('windows', directory)
         self.bits = bits
-        if directory is None:
-            self.directory = tempfile.mkdtemp()
-            self.rmdir = True
-        else:
-            self.directory = directory
-            self.rmdir = False
-        
+
         self.windows_prefix = '%s/%d' % (config.get('windows_environment_prefix'), self.bits)
         if not os.path.exists(self.windows_prefix):
             raise Error('windows prefix %s does not exist' % self.windows_prefix)
@@ -282,79 +325,49 @@ class WindowsTarget(Target):
         self.mingw_prefixes = ['/%s/%d' % (config.get('mingw_prefix'), self.bits), '%s/%d/%s-w64-mingw32' % (config.get('mingw_prefix'), bits, self.mingw_name)]
 
         self.set('PKG_CONFIG_LIBDIR', '%s/lib/pkgconfig' % self.windows_prefix)
-        self.set('PKG_CONFIG_PATH', '%s/lib/pkgconfig:%s/bin/pkgconfig' % (self.work_dir_cscript(), self.work_dir_cscript()))
+        self.set('PKG_CONFIG_PATH', '%s/lib/pkgconfig:%s/bin/pkgconfig' % (self.directory, self.directory))
         self.set('PATH', '%s/bin:%s:%s' % (self.windows_prefix, mingw_path, os.environ['PATH']))
         self.set('CC', '%s-w64-mingw32-gcc' % self.mingw_name)
         self.set('CXX', '%s-w64-mingw32-g++' % self.mingw_name)
         self.set('LD', '%s-w64-mingw32-ld' % self.mingw_name)
         self.set('RANLIB', '%s-w64-mingw32-ranlib' % self.mingw_name)
         self.set('WINRC', '%s-w64-mingw32-windres' % self.mingw_name)
-        cxx = '-I%s/include -I%s/include' % (self.windows_prefix, self.work_dir_cscript())
-        link = '-L%s/lib -L%s/lib' % (self.windows_prefix, self.work_dir_cscript())
+        cxx = '-I%s/include -I%s/include' % (self.windows_prefix, self.directory)
+        link = '-L%s/lib -L%s/lib' % (self.windows_prefix, self.directory)
         for p in self.mingw_prefixes:
             cxx += ' -I%s/include' % p
             link += ' -L%s/lib' % p
         self.set('CXXFLAGS', '"%s"' % cxx)
+        self.set('CPPFLAGS', '')
         self.set('LINKFLAGS', '"%s"' % link)
 
-    def work_dir_cdist(self):
-        return '%s/%d' % (self.directory, self.bits)
-
-    def work_dir_cscript(self):
-        return '%s/%d' % (self.directory, self.bits)
-
     def command(self, c):
         log('host -> %s' % c)
         command('%s %s' % (self.variables_string(), c))
 
-    def cleanup(self):
-        if self.rmdir:
-            rmtree(self.directory)
-
 #
 # Linux
 #
 
 class LinuxTarget(Target):
     def __init__(self, distro, version, bits, directory=None):
-        "directory -- directory to work in; if None, we will use the configured linux_dir_in_chroot"
-        super(LinuxTarget, self).__init__('linux', 2)
+        super(LinuxTarget, self).__init__('linux', directory)
         self.distro = distro
         self.version = version
         self.bits = bits
+        # e.g. ubuntu-14.04-64
         self.chroot = '%s-%s-%d' % (self.distro, self.version, self.bits)
-        if directory is None:
-            self.dir_in_chroot = config.get('linux_dir_in_chroot')
-        else:
-            self.dir_in_chroot = directory
+        # e.g. /home/carl/Environments/ubuntu-14.04-64
+        self.chroot_prefix = '%s/%s' % (config.get('linux_chroot_prefix'), self.chroot)
 
-        for g in glob.glob('%s/*' % self.work_dir_cdist()):
-            rmtree(g)
-
-        self.set('CXXFLAGS', '-I%s/include' % self.work_dir_cscript())
-        self.set('LINKFLAGS', '-L%s/lib' % self.work_dir_cscript())
-        self.set('PKG_CONFIG_PATH', '%s/lib/pkgconfig:/usr/local/lib/pkgconfig' % self.work_dir_cscript())
+        self.set('CXXFLAGS', '-I%s/include' % self.directory)
+        self.set('CPPFLAGS', '')
+        self.set('LINKFLAGS', '-L%s/lib' % self.directory)
+        self.set('PKG_CONFIG_PATH', '%s/lib/pkgconfig:/usr/local/lib/pkgconfig' % self.directory)
         self.set('PATH', '%s:/usr/local/bin' % (os.environ['PATH']))
 
-    def work_dir_cdist(self):
-        return '%s/%s%s' % (config.get('linux_chroot_prefix'), self.chroot, self.dir_in_chroot)
-
-    def work_dir_cscript(self):
-        return self.dir_in_chroot
-
     def command(self, c):
-        # Work out the cwd for the chrooted command
-        cwd = os.getcwd()
-        prefix = '%s/%s' % (config.get('linux_chroot_prefix'), self.chroot)
-        assert(cwd.startswith(prefix))
-        cwd = cwd[len(prefix):]
-
-        log('schroot [%s] -> %s' % (cwd, c))
-        command('%s schroot -c %s -d %s -p -- %s' % (self.variables_string(), self.chroot, cwd, c))
-
-    def cleanup(self):
-        for g in glob.glob('%s/*' % self.work_dir_cdist()):
-            rmtree(g)
+        command('%s schroot -c %s -p -- %s' % (self.variables_string(), self.chroot, c))
 
 #
 # OS X
@@ -362,16 +375,7 @@ class LinuxTarget(Target):
 
 class OSXTarget(Target):
     def __init__(self, directory=None):
-        "directory -- directory to work in; if None, we will use the configured osx_dir_in_host"
-        super(OSXTarget, self).__init__('osx', 4)
-
-        if directory is None:
-            self.dir_in_host = config.get('osx_dir_in_host')
-        else:
-            self.dir_in_host = directory
-
-        for g in glob.glob('%s/*' % self.dir_in_host):
-            rmtree(g)
+        super(OSXTarget, self).__init__('osx', directory)
 
     def command(self, c):
         command('%s %s' % (self.variables_string(False), c))
@@ -391,20 +395,15 @@ class OSXSingleTarget(OSXTarget):
         enviro = '%s/%d' % (config.get('osx_environment_prefix'), bits)
 
         # Environment variables
-        self.set('CFLAGS', '"-I%s/include -I%s/include %s"' % (self.work_dir_cscript(), enviro, flags))
-        self.set('CXXFLAGS', '"-I%s/include -I%s/include %s"' % (self.work_dir_cscript(), enviro, flags))
-        self.set('LDFLAGS', '"-L%s/lib -L%s/lib %s"' % (self.work_dir_cscript(), enviro, flags))
-        self.set('LINKFLAGS', '"-L%s/lib -L%s/lib %s"' % (self.work_dir_cscript(), enviro, flags))
-        self.set('PKG_CONFIG_PATH', '%s/lib/pkgconfig:%s/lib/pkgconfig:/usr/lib/pkgconfig' % (self.work_dir_cscript(), enviro))
+        self.set('CFLAGS', '"-I%s/include -I%s/include %s"' % (self.directory, enviro, flags))
+        self.set('CPPFLAGS', '')
+        self.set('CXXFLAGS', '"-I%s/include -I%s/include %s"' % (self.directory, enviro, flags))
+        self.set('LDFLAGS', '"-L%s/lib -L%s/lib %s"' % (self.directory, enviro, flags))
+        self.set('LINKFLAGS', '"-L%s/lib -L%s/lib %s"' % (self.directory, enviro, flags))
+        self.set('PKG_CONFIG_PATH', '%s/lib/pkgconfig:%s/lib/pkgconfig:/usr/lib/pkgconfig' % (self.directory, enviro))
         self.set('PATH', '$PATH:/usr/bin:/sbin:/usr/local/bin:%s/bin' % enviro)
         self.set('MACOSX_DEPLOYMENT_TARGET', config.get('osx_sdk'))
 
-    def work_dir_cdist(self):
-        return self.work_dir_cscript()
-
-    def work_dir_cscript(self):
-        return '%s/%d' % (self.dir_in_host, self.bits)
-
     def package(self, project):
         raise Error('cannot package non-universal OS X versions')
 
@@ -413,11 +412,8 @@ class OSXUniversalTarget(OSXTarget):
     def __init__(self, directory=None):
         super(OSXUniversalTarget, self).__init__(directory)
         self.parts = []
-        self.parts.append(OSXSingleTarget(32, directory))
-        self.parts.append(OSXSingleTarget(64, directory))
-
-    def work_dir_cscript(self):
-        return self.dir_in_host
+        self.parts.append(OSXSingleTarget(32, os.path.join(self.directory, '32')))
+        self.parts.append(OSXSingleTarget(64, os.path.join(self.directory, '64')))
 
     def package(self, project):
         for p in self.parts:
@@ -434,14 +430,7 @@ class OSXUniversalTarget(OSXTarget):
 
 class SourceTarget(Target):
     def __init__(self):
-        super(SourceTarget, self).__init__('source', 2)
-        self.directory = tempfile.mkdtemp()
-
-    def work_dir_cdist(self):
-        return self.directory
-
-    def work_dir_cscript(self):
-        return self.directory
+        super(SourceTarget, self).__init__('source')
 
     def command(self, c):
         log('host -> %s' % c)
@@ -478,7 +467,7 @@ def target_factory(s, debug, work):
     elif s.startswith('osx-'):
         target = OSXSingleTarget(int(s.split('-')[1]), work)
     elif s == 'osx':
-        if args.command == 'build':
+        if globals.command == 'build':
             target = OSXSingleTarget(64, work)
         else:
             target = OSXUniversalTarget(work)
@@ -501,23 +490,25 @@ class Project(object):
         self.directory = directory
         self.version = None
         self.specifier = specifier
+        self.git_commit = None
         if self.specifier is None:
             self.specifier = 'master'
 
     def checkout(self, target):
         flags = ''
         redirect = ''
-        if args.quiet:
+        if globals.quiet:
             flags = '-q'
             redirect = '>/dev/null'
-        command('git clone %s %s/%s.git %s/src/%s' % (flags, config.get('git_prefix'), self.name, target.work_dir_cdist(), self.name))
-        os.chdir('%s/src/%s' % (target.work_dir_cdist(), self.name))
+        command('git clone %s %s/%s.git %s/src/%s' % (flags, config.get('git_prefix'), self.name, target.directory, self.name))
+        os.chdir('%s/src/%s' % (target.directory, self.name))
         command('git checkout %s %s %s' % (flags, self.specifier, redirect))
-        command('git submodule init')
-        command('git submodule update')
+        self.git_commit = command_and_read('git rev-parse --short=7 HEAD').readline().strip()
+        command('git submodule init --quiet')
+        command('git submodule update --quiet')
         os.chdir(self.directory)
 
-        proj = '%s/src/%s/%s' % (target.work_dir_cdist(), self.name, self.directory)
+        proj = '%s/src/%s/%s' % (target.directory, self.name, self.directory)
 
         self.read_cscript('%s/cscript' % proj)
         
@@ -533,7 +524,7 @@ class Project(object):
 def set_version_in_wscript(version):
     f = open('wscript', 'rw')
     o = open('wscript.tmp', 'w')
-    while 1:
+    while True:
         l = f.readline()
         if l == '':
             break
@@ -571,216 +562,278 @@ def append_version_to_debian_changelog(version):
 
     command('dch -b -v %s-1 "New upstream release."' % version)
 
+def devel_to_git(project, filename):
+    if project.git_commit is not None:
+        filename = filename.replace('devel', '-%s' % project.git_commit)
+    return filename
+
+
 #
 # Command-line parser
 #
 
-parser = argparse.ArgumentParser()
-parser.add_argument('command')
-parser.add_argument('-p', '--project', help='project name')
-parser.add_argument('-d', '--directory', help='directory within project repo', default='.')
-parser.add_argument('--minor', help='minor version number bump', action='store_true')
-parser.add_argument('--micro', help='micro version number bump', action='store_true')
-parser.add_argument('-c', '--checkout', help='string to pass to git for checkout')
-parser.add_argument('-o', '--output', help='output directory', default='.')
-parser.add_argument('-q', '--quiet', help='be quiet', action='store_true')
-parser.add_argument('-t', '--target', help='target')
-parser.add_argument('-k', '--keep', help='keep working tree', action='store_true')
-parser.add_argument('--debug', help='build with debugging symbols where possible', action='store_true')
-parser.add_argument('-w', '--work', help='override default work directory')
-args = parser.parse_args()
-
-args.output = os.path.abspath(args.output)
-if args.work is not None:
-    args.work = os.path.abspath(args.work)
-
-if args.project is None and args.command != 'shell':
-    raise Error('you must specify -p or --project')
-
-project = Project(args.project, args.directory, args.checkout)
-
-if args.command == 'build':
-    if args.target is None:
-        raise Error('you must specify -t or --target')
-
-    target = target_factory(args.target, args.debug, args.work)
-    project.checkout(target)
-    target.build_dependencies(project)
-    target.build(project)
-    if not args.keep:
-        target.cleanup()
-
-elif args.command == 'package':
-    if args.target is None:
-        raise Error('you must specify -t or --target')
+def main():
+
+    commands = {
+        "build": "build project",
+        "package": "package and build project",
+        "release": "release a project using its next version number (changing wscript and tagging)",
+        "pot": "build the project's .pot files",
+        "changelog": "generate a simple HTML changelog",
+        "manual": "build the project's manual",
+        "doxygen": "build the project's Doxygen documentation",
+        "latest": "print out the latest version",
+        "test": "run the project's unit tests",
+        "shell": "build the project then start a shell in its chroot",
+        "revision": "print the head git revision number"
+    }
+
+    one_of = "Command is one of:\n"
+    summary = ""
+    for k, v in commands.iteritems():
+        one_of += "\t%s\t%s\n" % (k, v)
+        summary += k + " "
+
+    parser = argparse.ArgumentParser()
+    parser.add_argument('command', help=summary)
+    parser.add_argument('-p', '--project', help='project name')
+    parser.add_argument('-d', '--directory', help='directory within project repo', default='.')
+    parser.add_argument('--minor', help='minor version number bump', action='store_true')
+    parser.add_argument('--micro', help='micro version number bump', action='store_true')
+    parser.add_argument('--major', help='major version to return with latest', type=int)
+    parser.add_argument('-c', '--checkout', help='string to pass to git for checkout')
+    parser.add_argument('-o', '--output', help='output directory', default='.')
+    parser.add_argument('-q', '--quiet', help='be quiet', action='store_true')
+    parser.add_argument('-t', '--target', help='target')
+    parser.add_argument('-k', '--keep', help='keep working tree', action='store_true')
+    parser.add_argument('--debug', help='build with debugging symbols where possible', action='store_true')
+    parser.add_argument('-w', '--work', help='override default work directory')
+    args = parser.parse_args()
+
+    if args.output.find(':') == -1:
+        # This isn't of the form host:path so make it absolute
+        args.output = os.path.abspath(args.output) + '/'
+
+    # Now, args.output is 'host:' or 'path/'
+
+    if args.work is not None:
+        args.work = os.path.abspath(args.work)
+
+    if args.project is None and args.command != 'shell':
+        raise Error('you must specify -p or --project')
         
-    target = target_factory(args.target, args.debug, args.work)
+    globals.quiet = args.quiet
+    globals.command = args.command
 
-    packages = target.package(project)
-    if hasattr(packages, 'strip') or (not hasattr(packages, '__getitem__') and not hasattr(packages, '__iter__')):
-        packages = [packages]
+    project = Project(args.project, args.directory, args.checkout)
 
-    if target.platform == 'linux':
-        out = '%s/%s-%s-%d' % (args.output, target.distro, target.version, target.bits)
-        try:
-            os.makedirs(out)
-        except:
-            pass
-        for p in packages:
-            copyfile(p, '%s/%s' % (out, os.path.basename(p)))
-    else:
-        for p in packages:
-            copyfile(p, '%s/%s' % (args.output, os.path.basename(p)))
+    if not globals.command in commands:
+        e = 'command must be one of:\n' + one_of
+        raise Error('command must be one of:\n%s' % one_of)
 
-    if not args.keep:
-        target.cleanup()
+    if globals.command == 'build':
+        if args.target is None:
+            raise Error('you must specify -t or --target')
 
-elif args.command == 'release':
-    if args.minor is False and args.micro is False:
-        raise Error('you must specify --minor or --micro')
+        target = target_factory(args.target, args.debug, args.work)
+        project.checkout(target)
+        target.build_dependencies(project)
+        target.build(project)
+        if not args.keep:
+            target.cleanup()
 
-    target = SourceTarget()
-    project.checkout(target)
+    elif globals.command == 'package':
+        if args.target is None:
+            raise Error('you must specify -t or --target')
 
-    version = project.version
-    version.to_release()
-    if args.minor:
-        version.bump_minor()
-    else:
-        version.bump_micro()
+        target = target_factory(args.target, args.debug, args.work)
 
-    set_version_in_wscript(version)
-    append_version_to_changelog(version)
-    append_version_to_debian_changelog(version)
+        packages = target.package(project)
+        if hasattr(packages, 'strip') or (not hasattr(packages, '__getitem__') and not hasattr(packages, '__iter__')):
+            packages = [packages]
+
+        if target.platform == 'linux':
+            out = '%s%s-%s-%d' % (args.output, target.distro, target.version, target.bits)
+            try:
+                os.makedirs(out)
+            except:
+                pass
+            for p in packages:
+                copyfile(p, '%s/%s' % (out, os.path.basename(devel_to_git(project, p))))
+        else:
+            for p in packages:
+                copyfile(p, '%s%s' % (args.output, os.path.basename(devel_to_git(project, p))))
 
-    command('git commit -a -m "Bump version"')
-    command('git tag -m "v%s" v%s' % (version, version))
+        if not args.keep:
+            target.cleanup()
 
-    version.to_devel()
-    set_version_in_wscript(version)
-    command('git commit -a -m "Bump version"')
-    command('git push')
-    command('git push --tags')
+    elif globals.command == 'release':
+        if args.minor is False and args.micro is False:
+            raise Error('you must specify --minor or --micro')
 
-    target.cleanup()
+        target = SourceTarget()
+        project.checkout(target)
 
-elif args.command == 'pot':
-    target = SourceTarget()
-    project.checkout(target)
+        version = project.version
+        version.to_release()
+        if args.minor:
+            version.bump_minor()
+        else:
+            version.bump_micro()
 
-    pots = project.cscript['make_pot'](target)
-    for p in pots:
-        copyfile(p, '%s/%s' % (args.output, os.path.basename(p)))
+        set_version_in_wscript(version)
+        append_version_to_changelog(version)
+        append_version_to_debian_changelog(version)
 
-    target.cleanup()
+        command('git commit -a -m "Bump version"')
+        command('git tag -m "v%s" v%s' % (version, version))
 
-elif args.command == 'changelog':
-    target = SourceTarget()
-    project.checkout(target)
+        version.to_devel()
+        set_version_in_wscript(version)
+        command('git commit -a -m "Bump version"')
+        command('git push')
+        command('git push --tags')
 
-    text = open('ChangeLog', 'r')
-    html = open('%s/changelog.html' % args.output, 'w')
-    versions = 8
-    
-    last = None
-    changes = []
-    
-    while 1:
-        l = text.readline()
-        if l == '':
-            break
-    
-        if len(l) > 0 and l[0] == "\t":
-            s = l.split()
-            if len(s) == 4 and s[1] == "Version" and s[3] == "released.":
-                v = Version(s[2])
-                if v.micro == 0:
-                    if last is not None and len(changes) > 0:
-                        print >>html,"<h2>Changes between version %s and %s</h2>" % (s[2], last)
-                        print >>html,"<ul>"
-                        for c in changes:
-                            print >>html,"<li>%s" % c
-                        print >>html,"</ul>"
-                    last = s[2]
-                    changes = []
-                    versions -= 1
-                    if versions < 0:
-                        break
-            else:
-                c = l.strip()
-                if len(c) > 0:
-                    if c[0] == '*':
-                        changes.append(c[2:])
-                    else:
-                        changes[-1] += " " + c
-
-    target.cleanup()
-
-elif args.command == 'manual':
-    target = SourceTarget()
-    project.checkout(target)
-
-    outs = project.cscript['make_manual'](target)
-    for o in outs:
-        if os.path.isfile(o):
-            copyfile(o, '%s/%s' % (args.output, os.path.basename(o)))
-        else:
-            copytree(o, '%s/%s' % (args.output, os.path.basename(o)))
+        target.cleanup()
 
-    target.cleanup()
+    elif globals.command == 'pot':
+        target = SourceTarget()
+        project.checkout(target)
 
-elif args.command == 'doxygen':
-    target = SourceTarget()
-    project.checkout(target)
+        pots = project.cscript['make_pot'](target)
+        for p in pots:
+            copyfile(p, '%s%s' % (args.output, os.path.basename(p)))
 
-    dirs = project.cscript['make_doxygen'](target)
-    if hasattr(dirs, 'strip') or (not hasattr(dirs, '__getitem__') and not hasattr(dirs, '__iter__')):
-        dirs = [dirs]
+        target.cleanup()
 
-    for d in dirs:
-        copytree(d, '%s/%s' % (args.output, 'doc'))
+    elif globals.command == 'changelog':
+        target = SourceTarget()
+        project.checkout(target)
+
+        text = open('ChangeLog', 'r')
+        html = tempfile.NamedTemporaryFile()
+        versions = 8
+
+        last = None
+        changes = []
+
+        while True:
+            l = text.readline()
+            if l == '':
+                break
+
+            if len(l) > 0 and l[0] == "\t":
+                s = l.split()
+                if len(s) == 4 and s[1] == "Version" and s[3] == "released.":
+                    v = Version(s[2])
+                    if v.micro == 0:
+                        if last is not None and len(changes) > 0:
+                            print >>html,"<h2>Changes between version %s and %s</h2>" % (s[2], last)
+                            print >>html,"<ul>"
+                            for c in changes:
+                                print >>html,"<li>%s" % c
+                            print >>html,"</ul>"
+                        last = s[2]
+                        changes = []
+                        versions -= 1
+                        if versions < 0:
+                            break
+                else:
+                    c = l.strip()
+                    if len(c) > 0:
+                        if c[0] == '*':
+                            changes.append(c[2:])
+                        else:
+                            changes[-1] += " " + c
+
+        copyfile(html.file, '%schangelog.html' % args.output)
+        html.close()
+        target.cleanup()
 
-    target.cleanup()
+    elif globals.command == 'manual':
+        target = SourceTarget()
+        project.checkout(target)
 
-elif args.command == 'latest':
-    target = SourceTarget()
-    project.checkout(target)
+        outs = project.cscript['make_manual'](target)
+        for o in outs:
+            if os.path.isfile(o):
+                copyfile(o, '%s%s' % (args.output, os.path.basename(o)))
+            else:
+                copytree(o, '%s%s' % (args.output, os.path.basename(o)))
 
-    f = command_and_read('git log --tags --simplify-by-decoration --pretty="%d"')
-    t = f.readline()
-    m = re.compile(".*\((.*)\).*").match(t)
-    latest = None
-    if m:
-        tags = m.group(1).split(', ')
-        for t in tags:
-            if len(t) > 0 and t[0] == 'v':
-                latest = t[1:]
+        target.cleanup()
 
-    print latest
-    target.cleanup()
+    elif globals.command == 'doxygen':
+        target = SourceTarget()
+        project.checkout(target)
 
-elif args.command == 'test':
-    if args.target is None:
-        raise Error('you must specify -t or --target')
+        dirs = project.cscript['make_doxygen'](target)
+        if hasattr(dirs, 'strip') or (not hasattr(dirs, '__getitem__') and not hasattr(dirs, '__iter__')):
+            dirs = [dirs]
+
+        for d in dirs:
+            copytree(d, '%s%s' % (args.output, 'doc'))
+
+        target.cleanup()
+
+    elif globals.command == 'latest':
+        target = SourceTarget()
+        project.checkout(target)
+
+        f = command_and_read('git log --tags --simplify-by-decoration --pretty="%d"')
+        latest = None
+        while latest is None:
+            t = f.readline()
+            m = re.compile(".*\((.*)\).*").match(t)
+            if m:
+                tags = m.group(1).split(', ')
+                for t in tags:
+                    s = t.split()
+                    if len(s) > 1:
+                        t = s[1]
+                    if len(t) > 0 and t[0] == 'v':
+                        v = Version(t[1:])
+                        if args.major is None or v.major == args.major:
+                            latest = v
+
+        print latest
+        target.cleanup()
+
+    elif globals.command == 'test':
+        if args.target is None:
+            raise Error('you must specify -t or --target')
+
+        target = None
+        try:
+            target = target_factory(args.target, args.debug, args.work)
+            target.test(project)
+        except Error as e:
+            if target is not None:
+                target.cleanup()
+            raise
 
-    target = None
-    try:
-        target = target_factory(args.target, args.debug, args.work)
-        target.test(project)
-    except Error as e:
         if target is not None:
             target.cleanup()
-        raise
-        
-    if target is not None:
-        target.cleanup()
 
-elif args.command == 'shell':
-    if args.target is None:
-        raise Error('you must specify -t or --target')
+    elif globals.command == 'shell':
+        if args.target is None:
+            raise Error('you must specify -t or --target')
 
-    target = target_factory(args.target, args.debug, args.work)
-    target.command('bash')
+        target = target_factory(args.target, args.debug, args.work)
+        target.command('bash')
+
+    elif globals.command == 'revision':
+
+        target = SourceTarget()
+        project.checkout(target)
+        print command_and_read('git rev-parse HEAD').readline().strip()[:7]
+        target.cleanup()
+
+    else:
+        raise Error('invalid command %s' % globals.command)
 
-else:
-    raise Error('invalid command %s' % args.command)
+try:
+    main()
+except Error as e:
+    print >>sys.stderr,'cdist: %s' % str(e)
+    sys.exit(1)