X-Git-Url: https://git.ralfj.de/auto-debuild.git/blobdiff_plain/a9d4d5c4a1be0311e53747b98dfd74cabcb8108d..07e101d3ec702bb92eb0efe2143147409f98a094:/auto_debuild.py?ds=inline

diff --git a/auto_debuild.py b/auto_debuild.py
index 61d7efc..94c7c14 100755
--- a/auto_debuild.py
+++ b/auto_debuild.py
@@ -1,8 +1,44 @@
 #!/usr/bin/python
-import os, shutil, stat, time, subprocess, sys
+# auto-debuild - Automatic Generation of Debian Packages
+# Copyright (C) 2012 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 os, shutil, stat, time, subprocess, sys, shlex, tempfile
 from collections import OrderedDict
 
-# some utility functions
+# a dict with some useful additional getters which can convert types and handle one-element lists like their single member
+class ConfigDict(dict):
+	def getstr(self, name, default = None):
+		if not name in self: return default
+		val = self[name]
+		if isinstance(val, list):
+			if len(val) != 1: raise Exception('%s is a list, but it should not' % name)
+			return val[0]
+		else:
+			return val
+	
+	def getint(self, name, default = None):
+		return int(self.getstr(name, default))
+	
+	def getbool(self, name, default = None):
+		val = self.getstr(name, default)
+		if isinstance(val, bool): return val # already a bool
+		return val.lower() in ('true', 'yes', 'on', '1')
+
+# create a safe-to-call shell command from the array
 def safeCall(*args):
 	res = ""
 	for arg in args:
@@ -11,6 +47,35 @@ def safeCall(*args):
 		res += "'"+arg+"'"
 	return res
 
+# Load a section-less config file: maps parameter names to space-separated lists of strings (with shell quotation)
+# Lines starting with spaces are continuation lines
+def loadConfigFile(file):
+	# read config file
+	linenr = 0
+	with open(file) as file:
+		result = ConfigDict()
+		curKey = None
+		for line in file:
+			linenr += 1
+			isCont = len(line) and line[0].isspace() # remember if we were a continuation line
+			if isCont and curKey is None:
+				raise Exception("Invalid config, line %d: Starting with continuation line" % linenr)
+			line = line.strip()
+			if not len(line) or line.startswith("#"): continue # skip empty and comment lines
+			try:
+				if isCont:
+					# continuation line
+					result[curKey] += shlex.split(line)
+				else:
+					# option line
+					pos = line.index("=") # will raise exception when substring is not found
+					curKey = line[:pos].strip()
+					result[curKey] = shlex.split(line[pos+1:]) # shlex.split also strips
+			except Exception:
+				raise Exception("Invalid config, line %d: Error parsing line (quoting issue?)" % linenr)
+	# add some convencience get functions
+	return result
+
 # representation of a build system
 class BuildSystem:
 	def __init__(self, ruleMaker, buildDepends = [], binaryDepends = []):
@@ -40,7 +105,7 @@ class RulesFile:
 		print >>f, ""
 		print >>f, ".PHONY: build" # there may be a directory called "build"
 		print >>f, ""
-		print >>f, "build %:" # need to mention "build" here again explicitly
+		print >>f, "build %:" # need to mention "build" here again explicitly so PHONY takes effect
 		# write proper dh call
 		dh = self.dh
 		if self.dhWith:
@@ -54,10 +119,9 @@ class RulesFile:
 				print >>f, "\t"+line
 
 # rule-makers
-def cmakeRules(config):
-	buildDir = config.get('buildDir', 'build')
+def cmakeRules(r, config):
+	buildDir = config.getstr('buildDir', 'build')
 	srcDir = os.getcwd()
-	r = RulesFile()
 	r.dh += ["--buildsystem=cmake", "--builddirectory="+buildDir] # dh parameters
 	r.rules['auto_configure'] = [
 		safeCall("mkdir", "-p", buildDir),
@@ -65,46 +129,49 @@ def cmakeRules(config):
 		  safeCall("cmake", srcDir, "-DCMAKE_INSTALL_PREFIX=/usr", *config.get('cmakeParameters', []))
 	]
 	r.rules['auto_clean'] = [safeCall('rm', '-f', os.path.join(buildDir, 'CMakeCache.txt'))] # clean old cmake cache
-	return r
 
-def automakeRules(config):
+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...)
-	# 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)
-	r = RulesFile()
-	r.dh += ["--buildsystem=autoconf"]
+	r.dh += ["--buildsystem=autoconf", "--builddirectory="+config.getstr('buildDir')]
 	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) && '+
-			'./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 '+
-			'--sysconfdir=/etc --localstatedir=/var '+
-			safeCall(*config.get('automakeParameters', []))
+			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)
-	return r
+	r.rules['auto_clean'] = [safeCall("rm", "-f", "--", buildDir+"/config.status")] # do not re-use old configuration
 
-def pythonRules(config):
-	r = RulesFile()
-	r.dh += ["--buildsystem=python_distutils"]
-	r.dhWith.add('python2')
-	r.rules['auto_clean'] = [ # clean properly
-		'dh_auto_clean',
-		'rm -rf build'
-	]
+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
+	r.rules['auto_configure'] = []
+	r.rules['auto_build'] = []
+	r.rules['auto_clean'] = []
 	return r
 
 # build systems
 buildSystems = {
 	'cmake': BuildSystem(cmakeRules, ["cmake"]),
-	'automake': BuildSystem(automakeRules, ["automake"]),
-	'python': BuildSystem(pythonRules, ["python-setuptools"], ["${python:Depends}"]),
+	'automake': BuildSystem(automakeRules),
+	'makefile': BuildSystem(makefileRules),
+	'none': BuildSystem(noneRules),
 }
 
 # utility functions
 def commandInBuildEnv(config, command):
-	schroot = config.get('schroot')
+	schroot = config.getstr('schroot')
 	if schroot is not None: command = ['schroot', '-c', schroot, '--'] + command
 	return command
 
@@ -119,28 +186,52 @@ def writeDependency(f, name, 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):
-	sourceName = config['sourceName']
-	binaryName = config.get('binaryName', sourceName+'-local')
-	name = config.get('name', os.getenv('USER')) # os.getlogin() fails in minimal chroots
-	email = config.get('email', os.getenv('USER')+'@'+os.uname()[1]) # user@hostname
-	debDir = os.path.expanduser(config['debDir'])
-	buildSystem = buildSystems[config['buildSystem']] # get the data immediately
-	version = config['version']
-	dbgPackage = config.get('dbgPackage', False)
-	parallelJobs = int(config.get('parallelJobs', 2))
-	packageArchitecture = config.get('architecture', 'any')
-	withPython2 = config.get('withPython2', False)
+	if not isinstance(config, ConfigDict):
+		config = ConfigDict(config)
+	sourceName = config.getstr('sourceName')
+	binaryName = config.getstr('binaryName', sourceName+'-local')
+	config['binaryName'] = binaryName # make it usable by build systems
+	name = config.getstr('name', os.getenv('USER')) # os.getlogin() fails in minimal chroots
+	email = config.getstr('email', os.getenv('USER')+'@'+os.uname()[1]) # user@hostname
+	debDir = os.path.expanduser(config.getstr('debDir'))
+	buildSystem = buildSystems[config.getstr('buildSystem')] # get the data immediately
+	version = config.getstr('version') # version name excluding epoch (used for filenames)
+	fullVersion = str(config.getint('epoch'))+':'+version if 'epoch' in config else version # version name including epoch
+	dbgPackage = config.getbool('dbgPackage', 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...)
+	if withSIP:
+		withPython2 = True
+		buildSystem.buildDepends.append("python-sip")
+		buildSystem.binaryDepends.append("${sip:Depends}")
 	if withPython2:
-		buildSystem.binaryDepends.append("${python:Depends}") # HACK, but it works: make sure dependencies on binary are added
+		buildSystem.buildDepends.append("python")
+		buildSystem.binaryDepends.append("${python:Depends}")
+	if withAutoreconf:
+		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
-	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?')
+	tempdir = tempfile.mkdtemp(prefix='auto-debuild-')
+	os.mkdir(tempdir+'/source')
+	os.symlink(tempdir, 'debian')
 	if not os.path.exists(debDir): os.makedirs(debDir)
 	# source format file
 	with open('debian/source/format', 'w') as f:
@@ -153,39 +244,59 @@ def createDebianFiles(config):
 		print >>f, "Auto-generated by auto-debuild, not suited for distribution"
 	# changelog file
 	with open('debian/changelog', 'w') as f:
-		print >>f, sourceName,"("+version+")","UNRELEASED; urgency=low"
+		print >>f, sourceName,"("+fullVersion+")","UNRELEASED; urgency=low"
 		print >>f, ""
 		print >>f, "  * Auto-generated by auto-debuild"
 		print >>f, ""
 		print >>f, " --",name,"<"+email+">  "+time.strftime('%a, %d %b %Y %H:%M:%S %z')
 	# control file
 	with open('debian/control', 'w') as f:
+		# source package
 		print >>f, "Source:",sourceName
-		print >>f, "Section:",config.get('section', 'misc')
+		print >>f, "Section:",config.getstr('section', 'misc')
 		print >>f, "Priority: extra"
 		print >>f, "Maintainer: %s <%s>" % (name, email)
 		writeDependency(f, 'Build-Depends', ["debhelper (>= 9)"] + buildSystem.buildDepends + config.get('buildDepends', []))
 		print >>f, "Standards-Version: 3.9.3"
+		# main binary package
 		print >>f, ""
 		print >>f, "Package:",binaryName
 		print >>f, "Architecture:",packageArchitecture
+		if 'binaryMultiArch' in config:
+			print >>f, "Multi-Arch:",config.getstr('binaryMultiArch')
 		writeDependency(f, "Pre-Depends", ["${misc:Pre-Depends}"] + config.get('binaryPreDepends', []))
 		writeDependency(f, "Depends", ["${shlibs:Depends}", "${misc:Depends}"] + buildSystem.binaryDepends +
 			config.get('binaryDepends', []))
 		writeDependency(f, "Recommends", config.get('binaryRecommends', []))
 		writeDependency(f, "Provides", config.get('binaryProvides', [sourceName]))
 		writeDependency(f, "Conflicts", config.get('binaryConflicts', []))
+		writeDependency(f, "Breaks", config.get('binaryBreaks', []) + config.get('binaryBreaksReplaces', []))
+		writeDependency(f, "Replaces", config.get('binaryReplaces', []) + config.get('binaryBreaksReplaces', []))
 		print >>f, "Description:",sourceName,"(auto-debuild)"
 		print >>f, " Package auto-generated by auto-debuild."
 		files.append(os.path.join(debDir, "%s_%s_%s.deb" % (binaryName, version, arch)))
+		# debug package
 		if dbgPackage:
 			print >>f, ""
 			print >>f, "Package:",binaryName+"-dbg"
+			print >>f, "Section: debug"
+			print >>f, "Priority: extra"
 			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)))
+		# shim packages
+		for shim in config.get('binaryShims', []):
+			print >>f, ""
+			print >>f, "Package:",shim
+			print >>f, "Section:",config.getstr('section', 'misc')
+			print >>f, "Priority: extra"
+			print >>f, "Architecture:",packageArchitecture
+			writeDependency(f, "Depends", ["${misc:Depends}", binaryName+" (= ${binary:Version})"])
+			print >>f, "Description:",sourceName,"shim for",shim,"(auto-debuild)"
+			print >>f, " Package pretending to be "+shim+", auto-generated by auto-debuild."
+			files.append(os.path.join(debDir, "%s_%s_%s.deb" % (shim, version, arch)))
 	# install file
 	with open('debian/'+binaryName+'.install', 'w') as f:
 		for line in config.get('binaryInstallFiles', []):
@@ -198,9 +309,9 @@ def createDebianFiles(config):
 			print >>f, "#!/bin/sh"
 			print >>f, "set -e"
 			print >>f, 'if [ "$1" = "configure" ]; then'
-			for alternative in config['alternatives']:
-				print >>f, safeCall('update-alternatives', '--install', alternative['link'], alternative['name'], alternative['target'],
-					str(alternative['priority']))
+			for alternative in config.get('alternatives'):
+				alternative = shlex.split(alternative)
+				print >>f, safeCall('update-alternatives', '--install', alternative[0], alternative[1], alternative[2], alternative[3])
 			print >>f, 'fi'
 			print >>f, ''
 			print >>f, '#DEBHELPER#'
@@ -210,8 +321,9 @@ def createDebianFiles(config):
 			print >>f, "#!/bin/sh"
 			print >>f, "set -e"
 			print >>f, 'if [ "$1" = "remove" ]; then'
-			for alternative in config['alternatives']:
-				print >>f, safeCall('update-alternatives', '--remove', alternative['name'], alternative['target'])
+			for alternative in config.get('alternatives'):
+				alternative = shlex.split(alternative)
+				print >>f, safeCall('update-alternatives', '--remove', alternative[1], alternative[2])
 			print >>f, 'fi'
 			print >>f, ''
 			print >>f, '#DEBHELPER#'
@@ -219,8 +331,13 @@ def createDebianFiles(config):
 			print >>f, 'exit 0'
 	# rules file: build system specific
 	with open('debian/rules', 'w') as f:
-		# get rule file for build system: may only touch auto_config and auto_clean rules and the dh options
-		r = buildSystem.ruleMaker(config)
+		# pre-fill rule file with our global defaults
+		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
 		r.env["DEB_BUILD_OPTIONS"] = 'parallel='+str(parallelJobs)
 		if not dbgPackage:
@@ -230,20 +347,22 @@ def createDebianFiles(config):
 		r.dh += ['--parallel']
 		if withPython2:
 			r.dhWith.add('python2')
-			r.rules['python2'] = ['dh_python2 --no-guessing-versions']
+			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...)
-		r.rules['auto_test'] = []
 		# installation rule
-		r.rules['auto_install'] = [safeCall('dh_auto_install', '--destdir=debian/'+binaryName)] # install everything into the binary package
 		if 'binarySkipFiles' in config:
-			r.rules['auto_install'].append(safeCall('cd', 'debian/'+binaryName) + " && " +
-				safeCall('rm', *config['binarySkipFiles']))
-		# for debug packages
+			r.rules['auto_install'].append(safeCall('cd', 'debian/'+binaryName) + " && " + safeCall('rm', *config.get('binarySkipFiles')))
+		# debug packages
 		if dbgPackage:
 			r.rules['strip'] = [safeCall('dh_strip', '--dbg-package='+binaryName+"-dbg")] # put debug files in appropriate package
-			r.rules['installdocs'] = [safeCall('dh_installdocs', '--link-doc='+binaryName)] # make the doc folder of the dbg package a symlink
+		# make the doc folder of the other packages a symlink (dbg, shims)
+		r.rules['installdocs'] = [safeCall('dh_installdocs', '--link-doc='+binaryName)]
 		# wait after configuration?
-		if config.get('waitAfterConfig', False):
+		if config.getbool('waitAfterConfig', False):
 			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)
@@ -253,24 +372,24 @@ def createDebianFiles(config):
 	return files
 
 def buildDebianPackage(config):
-	commands = ['dpkg-checkbuilddeps', 'debian/rules clean', 'debian/rules build', 'fakeroot debian/rules binary', 'debian/rules clean']
+	if not isinstance(config, ConfigDict):
+		config = ConfigDict(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))
-	shutil.rmtree('debian') # it 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 __name__ == "__main__":
 	try:
-		import imp
 		# get config
-		config = imp.load_source('config', 'auto-debuild.conf').__dict__
-		os.remove('auto-debuild.confc')
+		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)
-			shutil.rmtree('debian')
+			deleteDebianFolder()
 		files = createDebianFiles(config)
 		# check if a file is overwritten
 		for file in files: