Merge lp://qastaging/~lool/linaro-image-tools/really-automatic-apt into lp://qastaging/linaro-image-tools/11.11

Proposed by Loïc Minier
Status: Merged
Merged at revision: 303
Proposed branch: lp://qastaging/~lool/linaro-image-tools/really-automatic-apt
Merge into: lp://qastaging/linaro-image-tools/11.11
Diff against target: 26 lines (+3/-2)
2 files modified
linaro_image_tools/media_create/tests/test_media_create.py (+1/-1)
linaro_image_tools/media_create/utils.py (+2/-1)
To merge this branch: bzr merge lp://qastaging/~lool/linaro-image-tools/really-automatic-apt
Reviewer Review Type Date Requested Status
Guilherme Salgado (community) Approve
Review via email: mp+54686@code.qastaging.launchpad.net

Description of the change

Pass --yes to apt-get when automatically installing packages for missing commands, this is needed when more than one package would get installed; bug #741638.

I tested this in the same chroot as for the bug report, after purging the packages which had been locally installed and got:

Installing required command qemu-arm-static from package qemu-kvm-extras-static
Reading package lists... Done
Building dependency tree
Reading state information... Done
The following extra packages will be installed:
  qemu-user-static
Recommended packages:
  binfmt-support debootstrap
The following NEW packages will be installed:
  qemu-kvm-extras-static qemu-user-static
0 upgraded, 2 newly installed, 0 to remove and 47 not upgraded.
Need to get 0B/11.5MB of archives.
After this operation, 30.3MB of additional disk space will be used.
WARNING: The following packages cannot be authenticated!
  qemu-user-static qemu-kvm-extras-static
Authentication warning overridden.
Selecting previously deselected package qemu-user-static.
(Reading database ... 19008 files and directories currently installed.)
Unpacking qemu-user-static (from .../qemu-user-static_0.14.50-2011.03-1-0ubuntu1~ppa10.10.1_amd64.deb) ...
[...]

This branch will likely conflict with other pending mps, but it should be trivial to resolve.

To post a comment you must log in.
Revision history for this message
Guilherme Salgado (salgado) wrote :

Looks good to me

 review approve

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches