Factor dependency writing into its own function and add support for Recommends

[auto-debuild.git] / auto_debuild.py

diff --git a/auto_debuild.py b/auto_debuild.py
index f8ff6419255c39784c9a9ff21254629b36b67440..9d7c0421d850ac8537dfd082f004c5d96a4ae2e8 100755 (executable)
--- a/auto_debuild.py
+++ b/auto_debuild.py
@@ -2,9 +2,10 @@
 import os, stat, time, subprocess, sys
 from collections import OrderedDict
 
 import os, stat, time, subprocess, sys
 from collections import OrderedDict
 

+# abstract representation of rules file

 class RulesFile:
        def __init__(self):
 class RulesFile:
        def __init__(self):

-               self.buildOptions = ''
+               self.env = []

                self.dh = []
                self.rules = OrderedDict()
        
                self.dh = []
                self.rules = OrderedDict()
        


@@ -12,43 +13,69 @@ class RulesFile:
                print >>f, "#!/usr/bin/make -f"
                print >>f, ""
                print >>f, "%:"
                print >>f, "#!/usr/bin/make -f"
                print >>f, ""
                print >>f, "%:"

-               print >>f, '\tDEB_BUILD_OPTIONS="'+self.buildOptions+'" dh $@',' '.join(self.dh)
+               print >>f, '\t'+' '.join(self.env)+' dh $@',' '.join(self.dh)

                for rule in self.rules:
                        print >>f, ""
                        print >>f, "override_dh_"+rule+":"
                        for line in self.rules[rule]:
                                print >>f, "\t"+line
 
                for rule in self.rules:
                        print >>f, ""
                        print >>f, "override_dh_"+rule+":"
                        for line in self.rules[rule]:
                                print >>f, "\t"+line
 

-def getArchitecture():
-       p = subprocess.Popen(['dpkg-architecture', '-qDEB_BUILD_ARCH'], stdout=subprocess.PIPE)
-       res = p.communicate()[0] # get only stdout
-       if p.returncode != 0: raise Exception("Querying dpkg for the architecture failed")
-       return res[0:len(res)-1] # chop of the \n at the end
-

 # build-system specific part of rules file
 def cmakeRules(config):
        r = RulesFile()
        r.dh += ["--buildsystem=cmake", "--builddirectory=build.dir"] # dh parameters: "build" is not a good idea, as that's also the name of a target...
        r.rules['auto_configure'] = [
                'mkdir -p build.dir',
 # build-system specific part of rules file
 def cmakeRules(config):
        r = RulesFile()
        r.dh += ["--buildsystem=cmake", "--builddirectory=build.dir"] # dh parameters: "build" is not a good idea, as that's also the name of a target...
        r.rules['auto_configure'] = [
                'mkdir -p build.dir',

-               "cd build.dir && cmake .. "+' '.join(['-DCMAKE_INSTALL_PREFIX=/usr', '-DCMAKE_BUILD_TYPE=Release'] + config.get('cmakeParameters', []))
+               "cd build.dir && cmake .. -DCMAKE_INSTALL_PREFIX=/usr "+' '.join(config.get('cmakeParameters', []))

        ]
        r.rules['auto_clean'] = ['rm -f build.dir/CMakeCache.txt'] # clean old cmake cache
        return r
 
        ]
        r.rules['auto_clean'] = ['rm -f build.dir/CMakeCache.txt'] # clean old cmake cache
        return r
 

-def writeDebList(list):
-       return ', '.join(list)
+def automakeRules(config):
+       r = RulesFile()
+       r.dh += ["--buildsystem=autoconf"]
+       r.rules['auto_configure'] = [
+               'BUILD_TYPE=$$(dpkg-architecture -qDEB_BUILD_GNU_TYPE) && ' + # doing the expansion beforehand ensures that we cancel if it fails
+               'MULTIARCH=$$(dpkg-architecture -qDEB_BUILD_MULTIARCH) && '+
+                       './configure --build=$$BUILD_TYPE ' +
+                       '--prefix=/usr --includedir=/usr/include --mandir=/usr/share/man --infodir=/usr/share/info ' +
+                       '--libdir=/usr/lib/$$MULTIARCH --libexecdir=/usr/lib/$$MULTIARCH '+
+                       '--sysconfdir=/etc --localstatedir=/var ' +
+                       ' '.join(config.get('automakeParameters', []))
+       ]
+       r.rules['auto_clean'] = ['rm -f config.status'] # do not re-use old configuration
+       return r
+
+# utility functions
+def commandInBuildEnv(config, command):
+       schroot = config.get('schroot')
+       if schroot is not None: command = ['schroot', '-c', schroot, '--'] + command
+       return command

 
 

+def getArchitecture(config):
+       cmd = commandInBuildEnv(config, ['dpkg-architecture', '-qDEB_BUILD_ARCH'])
+       p = subprocess.Popen(cmd, stdout=subprocess.PIPE)
+       res = p.communicate()[0] # get only stdout
+       if p.returncode != 0: raise Exception("Querying dpkg for the architecture failed")
+       return res[0:len(res)-1] # chop of the \n at the end
+
+def writeDependency(f, name, list):
+       if len(list):
+               print >>f, name+": "+', '.join(list)
+
+# actual work functions

 def createDebianFiles(config):
        sourceName = config['sourceName']
        binaryName = config.get('binaryName', sourceName+'-local')
        name = config.get('name', os.getlogin())
 def createDebianFiles(config):
        sourceName = config['sourceName']
        binaryName = config.get('binaryName', sourceName+'-local')
        name = config.get('name', os.getlogin())

-       email = config.get('email', os.getlogin()+'@'+os.uname()[1])
+       email = config.get('email', os.getlogin()+'@'+os.uname()[1]) # user@hostname

        debDir = os.path.expanduser(config['debDir'])
        buildSystem = config['buildSystem']
        version = config['version']
        debDir = os.path.expanduser(config['debDir'])
        buildSystem = config['buildSystem']
        version = config['version']

-       # we return the list of files generated
-       arch = getArchitecture()
+       dbgPackage = config.get('dbgPackage', False)
+       packageArchitecture = config.get('architecture', 'any')
+       # we return the list of files generated, so we need to know the architecture
+       arch = getArchitecture(config)

        files = []
        # source format file
        if not os.path.exists('debian/source'): os.mkdir('debian/source')
        files = []
        # source format file
        if not os.path.exists('debian/source'): os.mkdir('debian/source')


@@ -73,32 +100,46 @@ def createDebianFiles(config):
                print >>f, "Section:",config.get('section', 'misc')
                print >>f, "Priority: extra"
                print >>f, "Maintainer: %s <%s>" % (name, email)
                print >>f, "Section:",config.get('section', 'misc')
                print >>f, "Priority: extra"
                print >>f, "Maintainer: %s <%s>" % (name, email)

-               print >>f, "Build-Depends:",writeDebList(["debhelper (>= 9)"] + config.get('buildDepends', []))
+               writeDependency(f, 'Build-Depends', ["debhelper (>= 9)"] + config.get('buildDepends', []))

                print >>f, "Standards-Version: 3.9.3"
                print >>f, ""
                print >>f, "Package:",binaryName
                print >>f, "Standards-Version: 3.9.3"
                print >>f, ""
                print >>f, "Package:",binaryName

-               print >>f, "Architecture:",config.get('architecture', 'any')
-               print >>f, "Depends:",writeDebList(["${shlibs:Depends}", "${misc:Depends}"] + config.get('binaryDepends', []))
-               print >>f, "Provides:",writeDebList(config.get('binaryProvides', [sourceName]))
+               print >>f, "Architecture:",packageArchitecture
+               writeDependency(f, "Pre-Depends", ["${misc:Pre-Depends}"] + config.get('binaryPreDepends', []))
+               writeDependency(f, "Depends", ["${shlibs:Depends}", "${misc:Depends}"] + config.get('binaryDepends', []))
+               writeDependency(f, "Recommends", config.get('binaryRecommends', []))
+               writeDependency(f, "Provides", config.get('binaryProvides', [sourceName]))

                print >>f, "Description:",sourceName,"(auto-debuild)"
                print >>f, "Description:",sourceName,"(auto-debuild)"

-               print >>f, " Package auto-generated by auto-debuild"
+               print >>f, " Package auto-generated by auto-debuild."

                files.append(os.path.join(debDir, "%s_%s_%s.deb" % (binaryName, version, arch)))
                files.append(os.path.join(debDir, "%s_%s_%s.deb" % (binaryName, version, arch)))

+               if dbgPackage:
+                       print >>f, ""
+                       print >>f, "Package:",binaryName+"-dbg"
+                       print >>f, "Architecture:",packageArchitecture
+                       writeDependency(f, "Depends", ["${misc:Depends}", binaryName+" (= ${binary:Version})"])
+                       print >>f, "Description:",sourceName,"debug smbols (auto-debuild)"
+                       print >>f, " Package containing debug symbols for "+sourceName+", auto-generated by auto-debuild."
+                       files.append(os.path.join(debDir, "%s-dbg_%s_%s.deb" % (binaryName, version, arch)))

        # rules file: build system specific
        with open('debian/rules', 'w') as f:
        # rules file: build system specific
        with open('debian/rules', 'w') as f:

-               # get rule file for build system
+               # get rule file for build system: may only touch auto_config and auto_clean rules and the dh options

                if buildSystem == 'cmake':
                        r = cmakeRules(config)
                if buildSystem == 'cmake':
                        r = cmakeRules(config)

+               elif buildSystem == 'automake':
+                       r = automakeRules(config)

                else:
                        raise Exception("Invalid build system "+buildSystem)
                # global rules
                else:
                        raise Exception("Invalid build system "+buildSystem)
                # global rules

-               r.buildOptions = "parallel=2"
+               r.env += ["DEB_BUILD_OPTIONS='parallel=2'"]
+               if not dbgPackage: r.env += ["DEB_CFLAGS_APPEND='-g0'", "DEB_CXXFLAGS_APPEND='-g0'"] # disable debug information

                r.dh += ['--parallel']
                r.dh += ['--parallel']

-               if not 'builddeb' in r.rules:
-                       r.rules['builddeb'] = ['dh_builddeb --destdir='+debDir] # passing this gobally to dh results in weird problems (like stuff being installed there, and not in the package...)
-               else:
-                       print "WARNING: Build system messes with global rules"
-               if not 'auto_test' in r.rules: # unless the build system has something in mind for this
-                       r.rules['auto_test'] = []
+               r.rules['builddeb'] = ['dh_builddeb --destdir='+debDir] # passing this gobally to dh results in weird problems (like stuff being installed there, and not in the package...)
+               r.rules['auto_test'] = []
+               r.rules['auto_install'] = ['dh_auto_install --destdir=debian/'+binaryName] # install everything into the binary package
+               # for debug packages
+               if dbgPackage:
+                       r.rules['strip'] = ['dh_strip --dbg-package='+binaryName+"-dbg"] # put debug files in appropriate package
+                       r.rules['installdocs'] = ['dh_installdocs --link-doc='+binaryName] # make the doc folder of the dbg package a symlink

                # dump it to a file
                r.write(f)
        mode = os.stat('debian/rules').st_mode
                # dump it to a file
                r.write(f)
        mode = os.stat('debian/rules').st_mode


@@ -106,24 +147,32 @@ def createDebianFiles(config):
        # return list of files affected
        return files
 
        # return list of files affected
        return files
 

-def buildDebianPackage():
-       subprocess.check_call(['dpkg-checkbuilddeps'])
-       subprocess.check_call(['debian/rules', 'clean'])
-       subprocess.check_call(['debian/rules', 'build'])
-       subprocess.check_call(['fakeroot', 'debian/rules', 'binary'])
-       subprocess.check_call(['debian/rules', 'clean'])
+def buildDebianPackage(config):
+       commands = ['dpkg-checkbuilddeps', 'debian/rules clean', 'debian/rules build', 'fakeroot debian/rules binary', 'debian/rules clean']
+       command = ['bash', '-c', ' && '.join(commands)]
+       subprocess.check_call(commandInBuildEnv(config, command))
+
+# all at once
+def createAndInstall(config, overwriteCheck = False):
+       # generate debian files
+       files = createDebianFiles(config)
+       # check if a file is overwritten
+       if overwriteCheck:
+               for file in files:
+                       if os.path.exists(file):
+                               if raw_input("Do you want to overwrite %s (y/N)? " % file).lower() != "y":
+                                       sys.exit(1)
+       # run compilation
+       buildDebianPackage(config)
+       # install files
+       print "Installing created deb files..."
+       subprocess.check_call(['sudo', 'dpkg', '--install'] + files)

 
 # if we are called directly as script
 if __name__ == "__main__":
 
 # if we are called directly as script
 if __name__ == "__main__":

-       # generate debian files
+       # get config

        import imp
        import imp

-       config = imp.load_source('config', 'debian/auto-debuild.conf')
+       config = imp.load_source('config', 'debian/auto-debuild.conf').__dict__

        os.remove('debian/auto-debuild.confc')
        os.remove('debian/auto-debuild.confc')

-       files = createDebianFiles(config.__dict__)
-       # check if a file is overwritten
-       for file in files:
-               if os.path.exists(file):
-                       if raw_input("Do you want to overwrite %s (y/N)? " % file).lower() != "y":
-                               sys.exit(1)
-       # run compilation
-       buildDebianPackage()
+       # and go for it
+       createAndInstall(config, overwriteCheck=True)