python-vm-builder lacks grub dependency

[Klaus Zimmermann]

The vmbuilder makes use of grub. However ubuntu-server-8.10-amd64 ships with lilo; grub get's not installed by
default. That leads to the error described in http://ubuntuforums.org/showthread.php?t=1056124 .
To remedy this, vmbuilder should imho depend on grub.

Description: Ubuntu 8.10
Release: 8.10

python-vm-builder:
  Installed: 0.9-0ubuntu3.1
  Candidate: 0.9-0ubuntu3.1
  Version table:
 *** 0.9-0ubuntu3.1 0
        500 http://de.archive.ubuntu.com intrepid-updates/universe Packages
        500 http://security.ubuntu.com intrepid-security/universe Packages
        100 /var/lib/dpkg/status
     0.9-0ubuntu3 0
        500 http://de.archive.ubuntu.com intrepid/universe Packages

[Chuck Short (zulcss)]

Still a problem on karmic.

chuck@cartman:~$ apt-cache depends python-vm-builder
python-vm-builder
 |Depends: kvm
  Depends: <qemu>
    qemu-kvm
  Depends: debootstrap
  Depends: parted
  Depends: kpartx
  Depends: ubuntu-keyring
  Depends: python
  Depends: python-central
  Depends: python-cheetah
  Depends: devscripts
  Recommends: python-libvirt

[Launchpad Janitor (janitor)]

This bug was fixed in the package vm-builder - 0.11.3-0ubuntu3

---------------
vm-builder (0.11.3-0ubuntu3) lucid; urgency=low

  * New patch, ssk-key-type-error, add parentheses as operator priority caused
    a TypeError in the format string of the user chown call in the
    install_authorized_keys() function; LP: #436835.
  * Add rsync dependency; LP: #314833.
  * Wrap build-deps and deps to get cleaner diffs.
  * Use Priority: optional.
  * Use launchpad.net/vmbuilder instead of code.l.n in copyright.
  * Downgrade devscripts dep to a Recommends as I believe this is only needed
    for rmadison in a Xen specific codepath, and it's rather heavy to install
    devscripts on servers; this needs to be confirmed though.
  * Depend on grub (legacy) as this is currently a hard requirement;
    LP: #331595.
  * Depend on qemu-kvm | qemu-kvm-extras instead of kvm | qemu as these are
    transitional packages nowadays.
  * New patch, ip-mask-typo, fixes typo in IP mask handling; LP: #313575; from
    r355.
  * Build-dep on python-all, not python-all-dev.
  * Bump up Standards-Version to 3.8.3.
  * Change Vcs-Bzr to XS-Upstream-Vcs-Bzr as the packaging is not kept in Bzr.
  * Add watch file.
  * Remove leftover VMBuilder/vcsversion.py.debdiff.
  * Move changes from 0.10-0ubuntu2 to sleep a little to leave a chance to
    loop devices to settle down to a new patch, settle-down-disk.
 -- Loic Minier <email address hidden> Sun, 17 Jan 2010 21:21:38 +0100

[Colin Watson (cjwatson)]

r348 upstream was supposed to remove this requirement. Please revert this - it's a serious bug for vm-builder to be making decisions about which boot loader is installed on the host system.

[Loïc Minier (lool)]

Ack; I reverted the grub dependency; I thought vm-builder was still hardcoding this requirement and wanted to reflect this in the deps while working on supporting grub2 and using the bootloader from the guest instead of the host; when I grepped for grub1, I thought that was still the case and I added the dependency erroneously; reverted now.