make using /tmp for the debian folder configurable (but it remains the default)

[auto-debuild.git] / auto_debuild.py

diff --git a/auto_debuild.py b/auto_debuild.py
index f1a3e84856e4edfe3e0340b1747749e50fd837cf..24720b9d683336185e9d832a7ed40a2e53c9b0b3 100755 (executable)
--- a/auto_debuild.py
+++ b/auto_debuild.py
@@ -16,7 +16,7 @@
 # along with this program; if not, write to the Free Software
 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
 
 # along with this program; if not, write to the Free Software
 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
 

-import os, shutil, stat, time, subprocess, sys, shlex
+import os, shutil, stat, time, subprocess, sys, shlex, tempfile

 from collections import OrderedDict
 
 # a dict with some useful additional getters which can convert types and handle one-element lists like their single member
 from collections import OrderedDict
 
 # a dict with some useful additional getters which can convert types and handle one-element lists like their single member


@@ -131,22 +131,28 @@ def cmakeRules(r, config):
        r.rules['auto_clean'] = [safeCall('rm', '-f', os.path.join(buildDir, 'CMakeCache.txt'))] # clean old cmake cache
 
 def automakeRules(r, config):
        r.rules['auto_clean'] = [safeCall('rm', '-f', os.path.join(buildDir, 'CMakeCache.txt'))] # clean old cmake cache
 
 def automakeRules(r, config):

+       buildDir = config.getstr('buildDir', 'build')
+       srcDir = os.getcwd()

        # "build" is what we are building *on*, and "host" is what we are building *for* (and GNU is weird...)
        # "build" is what we are building *on*, and "host" is what we are building *for* (and GNU is weird...)

-       # also, we don't really support cross-building... ;-) (to do, we'd have to write shell code that checks whether BUILD_GNU_TYPE
+       # also, we don't really support cross-building... ;-) (to do so, we'd have to write shell code that checks whether BUILD_GNU_TYPE

        #  and HOST_GNU_TYPE are equal, and if they are not, add a --host parameter)
        #  and HOST_GNU_TYPE are equal, and if they are not, add a --host parameter)

-       r.dh += ["--buildsystem=autoconf"]
+       r.dh += ["--buildsystem=autoconf", "--builddirectory="+config.getstr('buildDir')]

        r.rules['auto_configure'] = [
        r.rules['auto_configure'] = [

+               safeCall("mkdir", "-p", buildDir),
+               safeCall("cd", buildDir) + " && " +

                'BUILD_TYPE=$$(dpkg-architecture -qDEB_BUILD_GNU_TYPE) && ' + # doing the expansion beforehand ensures that we cancel if it fails
                'MULTIARCH=$$(dpkg-architecture -qDEB_HOST_MULTIARCH) && '+
                'BUILD_TYPE=$$(dpkg-architecture -qDEB_BUILD_GNU_TYPE) && ' + # doing the expansion beforehand ensures that we cancel if it fails
                'MULTIARCH=$$(dpkg-architecture -qDEB_HOST_MULTIARCH) && '+

-                       './configure --build=$$BUILD_TYPE '+ # do the escape manually here so we can use the variables (there's no user-controlled string in here anyway)
+               safeCall(srcDir+"/configure") +
+                       ' --build=$$BUILD_TYPE '+ # do the escape manually here so we can use the variables (there's no user-controlled string in here anyway)

                        '--prefix=/usr --includedir=/usr/include --mandir=/usr/share/man --infodir=/usr/share/info '+
                        '--libdir=/usr/lib/$$MULTIARCH --libexecdir=/usr/lib/$$MULTIARCH '+
                        safeCall('--docdir=/usr/share/doc/'+config['binaryName'], '--sysconfdir=/etc', '--localstatedir=/var', *config.get('automakeParameters', []))
        ]
                        '--prefix=/usr --includedir=/usr/include --mandir=/usr/share/man --infodir=/usr/share/info '+
                        '--libdir=/usr/lib/$$MULTIARCH --libexecdir=/usr/lib/$$MULTIARCH '+
                        safeCall('--docdir=/usr/share/doc/'+config['binaryName'], '--sysconfdir=/etc', '--localstatedir=/var', *config.get('automakeParameters', []))
        ]

-       r.rules['auto_clean'] = ['rm -f config.status'] # do not re-use old configuration (no need for escaping here, obviously)
+       r.rules['auto_clean'] = [safeCall("rm", "-f", "--", buildDir+"/config.status")] # do not re-use old configuration

 
 def makefileRules(r, config):
        r.dh += ["--buildsystem=makefile"]
 
 def makefileRules(r, config):
        r.dh += ["--buildsystem=makefile"]

+       r.rules['auto_configure'] = []

 
 def noneRules(r, config):
        r.dh += ["--buildsystem=makefile"] # makefile does the least possible harm
 
 def noneRules(r, config):
        r.dh += ["--buildsystem=makefile"] # makefile does the least possible harm


@@ -180,6 +186,15 @@ def writeDependency(f, name, list):
        if len(list):
                print >>f, name+": "+', '.join(list)
 
        if len(list):
                print >>f, name+": "+', '.join(list)
 

+def deleteDebianFolder():
+       if os.path.islink('debian'):
+               target = os.readlink('debian')
+               if os.path.exists(target):
+                       shutil.rmtree(target)
+               os.remove('debian')
+       else:
+               shutil.rmtree('debian')
+

 # actual work functions
 def createDebianFiles(config):
        if not isinstance(config, ConfigDict):
 # actual work functions
 def createDebianFiles(config):
        if not isinstance(config, ConfigDict):


@@ -197,21 +212,31 @@ def createDebianFiles(config):
        parallelJobs = config.getint('parallelJobs', 2)
        packageArchitecture = config.getstr('architecture', 'any')
        withPython2 = config.getbool('withPython2', False)
        parallelJobs = config.getint('parallelJobs', 2)
        packageArchitecture = config.getstr('architecture', 'any')
        withPython2 = config.getbool('withPython2', False)

+       withSIP = config.getbool('withSIP', False)

        withAutoreconf = config.getbool('withAutoreconf', False)
        # add some build dependencies (a bit hacky adding it to the build system...)
        withAutoreconf = config.getbool('withAutoreconf', False)
        # add some build dependencies (a bit hacky adding it to the build system...)

+       if withSIP:
+               withPython2 = True
+               buildSystem.buildDepends.append("python-sip")
+               buildSystem.binaryDepends.append("${sip:Depends}")

        if withPython2:
        if withPython2:

+               buildSystem.buildDepends.append("python")

                buildSystem.binaryDepends.append("${python:Depends}")
        if withAutoreconf:
                buildSystem.binaryDepends.append("${python:Depends}")
        if withAutoreconf:

-               buildSystem.binaryDepends.append("dh-autoreconf")
+               buildSystem.buildDepends.append("dh-autoreconf")

        # we return the list of files generated, so we need to know the architecture
        arch = getArchitecture(config)
        files = []
        # create folders
        # we return the list of files generated, so we need to know the architecture
        arch = getArchitecture(config)
        files = []
        # create folders

-       if os.path.exists('debian'): raise Exception('debian folder already exists?')
-       os.mkdir('debian')
-       os.mkdir('debian/source')
+       if os.path.exists('debian') or os.path.islink('debian'): raise Exception('debian folder already exists?')
+       if config.getbool('useTmp', True):
+               tempdir = tempfile.mkdtemp(prefix='auto-debuild-')
+               os.symlink(tempdir, 'debian')
+       else:
+               os.mkdir('debian')

        if not os.path.exists(debDir): os.makedirs(debDir)
        # source format file
        if not os.path.exists(debDir): os.makedirs(debDir)
        # source format file

+       os.mkdir('debian/source')

        with open('debian/source/format', 'w') as f:
                print >>f, "3.0 (native)"
        # compat file
        with open('debian/source/format', 'w') as f:
                print >>f, "3.0 (native)"
        # compat file


@@ -313,6 +338,7 @@ def createDebianFiles(config):
                r = RulesFile()
                r.rules['auto_test'] = []
                r.rules['auto_install'] = [safeCall('dh_auto_install', '--destdir=debian/'+binaryName)] # install everything into the binary package
                r = RulesFile()
                r.rules['auto_test'] = []
                r.rules['auto_install'] = [safeCall('dh_auto_install', '--destdir=debian/'+binaryName)] # install everything into the binary package

+               r.rules['auto_configure'] = ['dh_auto_configure'] # make sure there is an override (we may just append to it later)

                # patch rule file for build system: may only touch auto_* rules and the dh options
                buildSystem.ruleMaker(r, config)
                # global rules
                # patch rule file for build system: may only touch auto_* rules and the dh options
                buildSystem.ruleMaker(r, config)
                # global rules


@@ -325,12 +351,13 @@ def createDebianFiles(config):
                if withPython2:
                        r.dhWith.add('python2')
                        r.rules['python2'] = ['dh_python2 --no-guessing-versions --no-shebang-rewrite']
                if withPython2:
                        r.dhWith.add('python2')
                        r.rules['python2'] = ['dh_python2 --no-guessing-versions --no-shebang-rewrite']

+                       if withSIP:
+                               r.rules['python2'].append(safeCall('dh_sip', '-p'+binaryName))

                if withAutoreconf:
                        r.dhWith.add('autoreconf')
                r.rules['builddeb'] = [safeCall('dh_builddeb', "--destdir="+debDir)] # passing this gobally to dh results in weird problems (like stuff being installed there, and not in the package...)
                # installation rule
                if 'binarySkipFiles' in config:
                if withAutoreconf:
                        r.dhWith.add('autoreconf')
                r.rules['builddeb'] = [safeCall('dh_builddeb', "--destdir="+debDir)] # passing this gobally to dh results in weird problems (like stuff being installed there, and not in the package...)
                # installation rule
                if 'binarySkipFiles' in config:

-                       if not 'auto_install' in r.rules: r.rules['auto_install'] = ['dh_auto_install'] # make sure there is an override

                        r.rules['auto_install'].append(safeCall('cd', 'debian/'+binaryName) + " && " + safeCall('rm', *config.get('binarySkipFiles')))
                # debug packages
                if dbgPackage:
                        r.rules['auto_install'].append(safeCall('cd', 'debian/'+binaryName) + " && " + safeCall('rm', *config.get('binarySkipFiles')))
                # debug packages
                if dbgPackage:


@@ -339,7 +366,6 @@ def createDebianFiles(config):
                r.rules['installdocs'] = [safeCall('dh_installdocs', '--link-doc='+binaryName)]
                # wait after configuration?
                if config.getbool('waitAfterConfig', False):
                r.rules['installdocs'] = [safeCall('dh_installdocs', '--link-doc='+binaryName)]
                # wait after configuration?
                if config.getbool('waitAfterConfig', False):

-                       if not 'auto_configure' in r.rules: r.rules['auto_configure'] = ['dh_auto_configure'] # make sure there is an override

                        r.rules['auto_configure'].append("@"+safeCall('read', '-p', 'Configuration done. Hit "Enter" to build the package. ', 'DUMMY_VAR')) # if we run in dash, we need to tell it which variable to use for the result...
                # dump it to a file
                r.write(f)
                        r.rules['auto_configure'].append("@"+safeCall('read', '-p', 'Configuration done. Hit "Enter" to build the package. ', 'DUMMY_VAR')) # if we run in dash, we need to tell it which variable to use for the result...
                # dump it to a file
                r.write(f)


@@ -354,7 +380,7 @@ def buildDebianPackage(config):
        commands = ['dpkg-checkbuilddeps', 'debian/rules clean', 'debian/rules build', 'fakeroot debian/rules binary']
        command = ['bash', '-c', ' && '.join(commands)] # make it all one command, so we don't have to open and close the chroot too often
        subprocess.check_call(commandInBuildEnv(config, command))
        commands = ['dpkg-checkbuilddeps', 'debian/rules clean', 'debian/rules build', 'fakeroot debian/rules binary']
        command = ['bash', '-c', ' && '.join(commands)] # make it all one command, so we don't have to open and close the chroot too often
        subprocess.check_call(commandInBuildEnv(config, command))

-       shutil.rmtree('debian') # cleanup: the debian folder only contains what we just created
+       deleteDebianFolder() # cleanup: the debian folder only contains what we just created

 
 ###################################################################
 # if we are called directly as script
 
 ###################################################################
 # if we are called directly as script


@@ -363,10 +389,10 @@ if __name__ == "__main__":
                # get config
                config = loadConfigFile('auto-debuild.conf')
                # generate debian files
                # get config
                config = loadConfigFile('auto-debuild.conf')
                # generate debian files

-               if os.path.exists('debian'):
-                       if raw_input("A debian folder already exists, to you want to remove it (y/N)? ").lower() != "y":
+               if os.path.exists('debian') or os.path.islink('debian'):
+                       if raw_input("A debian folder already exists, do you want to remove it and whatever it links to (y/N)? ").lower() != "y":

                                sys.exit(1)
                                sys.exit(1)

-                       shutil.rmtree('debian')
+                       deleteDebianFolder()

                files = createDebianFiles(config)
                # check if a file is overwritten
                for file in files:
                files = createDebianFiles(config)
                # check if a file is overwritten
                for file in files: