Add dependency to package virtualbox-3.0 and/or virtualbox-3.1

Bug #524704 reported by permanuno
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
TestDrive
Fix Released
Low
Dustin Kirkland 
testdrive (Ubuntu)
Fix Released
Low
Dustin Kirkland 

Bug Description

I installed testdrive 1.21-0ubuntu1~ppa3 on a machine with non-free virtualbox-3.1. Since I didn't have any of the VM programs in the dependency list [qemu-kvm | kvm (>= 1:84+dfsg-0ubuntu12.4) | virtualbox-ose (>= 3.0.8)], APT installed kvm and all it's required depencencies. In ~/.testdriverc I added 'VIRT = "virtualbox"', and tried to installed Ubuntu Desktop via testdrive. When the new virtual machine tried to boot, I got an error from VirtualBox about a corrupted DKMS module as a result of the kvm installation (sorry for not having the exact error message). Additionally, none of my other virtual machines would boot. Luckily, after uninstalling testdrive + dependecies, and reinstalling virtualbox-3.1, my VMs started working again.

Looking at the testdrive script it seems like the program will work with virtualbox-ose 3.0 & 3.1, so it will probably work with the non-free version. Could 'virtualbox-3.0' and 'virtualbox-3.1' be added as dependencies to testdrive so that it won't try to install other VM software? Those are the exact package names from 'apt-cache search virtualbox'.

Tags: patch
permanuno (permanuno)
description: updated
summary: - Add dependency to package virtualbox-3.0 or virtualbox-3.1
+ Add dependency to package virtualbox-3.0 and/or virtualbox-3.1
Revision history for this message
permanuno (permanuno) wrote :

This is my first time submitting a patch, so please pardon if I've done something wrong.

The attached bzr patch to 'debian/control' should add virtualbox-3.0 and/or virtualbox-3.1 as dependencies. I don't know how to bundle the checked-out files into a deb file so I wasn't able to test thoroughly. However, I applied the same change to a downloaded testdrive deb file that I unpacked, modified, then repacked, and it installed without problems.

Revision history for this message
Dustin Kirkland  (kirkland) wrote :

Hi,

Thanks for the report.

I have never seen the non-free version of virtualbox.

Can you post the output of your:

dpkg -l | grep virtualbox

Thanks!

:-Dustin

Changed in testdrive:
status: New → Fix Committed
importance: Undecided → Low
assignee: nobody → Dustin Kirkland (kirkland)
Changed in testdrive (Ubuntu):
status: New → Fix Committed
importance: Undecided → Low
assignee: nobody → Dustin Kirkland (kirkland)
Revision history for this message
permanuno (permanuno) wrote :

Hi Dustin,

Here's the output:

$ dpkg -l | grep virtualbox
ii virtualbox-3.1 3.1.4-57640_Ubuntu_jaunty Sun VirtualBox

There's also a 'virtualbox-3.0' in the virtualbox.org repository: http://download.virtualbox.org/virtualbox/debian

- Sergio

tags: added: patch
Revision history for this message
Dustin Kirkland  (kirkland) wrote :

Cool, thanks.

This should be fixed in the next release.

Changed in testdrive:
status: Fix Committed → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package testdrive - 1.22-0ubuntu1

---------------
testdrive (1.22-0ubuntu1) lucid; urgency=low

  * testdrive:
    - fix run() return value checking, LP: #525438
    - more graceful error if ISO url 404's, LP: #491600
    - remove unnecessary warning when running VB, LP: #486884
    - offer to launch usb-creator after running VM, LP: #507537
    - add a -v|--version option, LP: #526548
    - unconditional print message about cleaning cache, LP: #526546
    - fix typo, LP: #526570
    - clean up output and prompt at the end of a desktop run
  * testdrive-select-iso, doc/testdrive-select-iso.1, debian/install,
    debian/rules:
    - move this file into testdrive, now that testdrive is in python too
  * testdriverc: quote sample CACHE value, LP: #524241
  * debian/control:
    - allow for non-free virtualbox in package dependencies, LP: #524704
    - drop suggests of testdrivegtk package, LP: #526490
 -- Dustin Kirkland <email address hidden> Thu, 18 Feb 2010 22:31:34 -0600

Changed in testdrive (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.