diff options
author | Neil Williams <codehelp@debian.org> | 2014-12-23 16:25:39 +0000 |
---|---|---|
committer | Neil Williams <codehelp@debian.org> | 2014-12-23 16:25:39 +0000 |
commit | b1545572859794a22a085f755988693b8d2d3377 (patch) | |
tree | 07a5ef381672d1d58fe738523620416deb701a36 /vmdebootstrap | |
parent | ec9b91e5f3382684c9b3f6157dd0cfb259de9850 (diff) | |
download | vmdebootstrap-b1545572859794a22a085f755988693b8d2d3377.tar.gz |
Fix package lists if no necessary packages exist
On foreign architectures, it is quite possible to
not have "necessary_packages", so append the user-
specified list and the necessary list separately.
Tidy up undefined local variables.
Diffstat (limited to 'vmdebootstrap')
-rwxr-xr-x | vmdebootstrap | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/vmdebootstrap b/vmdebootstrap index ab9486b..aae437a 100755 --- a/vmdebootstrap +++ b/vmdebootstrap @@ -318,6 +318,7 @@ class VmDebootstrap(cliapp.Application): # pylint: disable=too-many-public-meth self.runcmd(['install-mbr', self.settings['image']]) def setup_kpartx(self): + bootindex = None out = self.runcmd(['kpartx', '-avs', self.settings['image']]) if self.settings['bootsize']: bootindex = 0 @@ -326,7 +327,7 @@ class VmDebootstrap(cliapp.Application): # pylint: disable=too-many-public-meth else: rootindex = 0 parts = 1 - boot = None + boot = None devices = [line.split()[2] for line in out.splitlines() if line.startswith('add map ')] @@ -367,9 +368,12 @@ class VmDebootstrap(cliapp.Application): # pylint: disable=too-many-public-meth include.append('sudo') args = ['debootstrap', '--arch=%s' % self.settings['arch']] - if self.settings['package'] and len(necessary_packages) > 0: + if self.settings['package']: args.append( - '--include=%s' % ','.join(necessary_packages + include)) + '--include=%s' % ','.join(include)) + if len(necessary_packages) > 0: + args.append( + '--include=%s' % ','.join(necessary_packages)) if self.settings['foreign']: args.append('--foreign') if self.settings['variant']: |