Merge lp:~laney/livecd-rootfs/omap4-install-reverted-stack into lp:livecd-rootfs

Proposed by Iain Lane
Status: Merged
Merged at revision: 780
Proposed branch: lp:~laney/livecd-rootfs/omap4-install-reverted-stack
Merge into: lp:livecd-rootfs
Diff against target: 35 lines (+16/-2)
1 file modified
live-build/auto/config (+16/-2)
To merge this branch: bzr merge lp:~laney/livecd-rootfs/omap4-install-reverted-stack
Reviewer Review Type Date Requested Status
Colin Watson Approve
Oliver Grawert Pending
Ubuntu Core Development Team Pending
Review via email: mp+175046@code.launchpad.net

Description of the change

On armhf+omap4, stop using the ubuntu-desktop task and use the metapackage instead

The task pulls in the X stack which the metapackage does not. We can take advantage of this to install the omap-revert X stack that's needed for these images.

I'm wondering if I should do either or both of

  - Collapse the case into a single case "$ARCH+$SUBARCH" statement since it only cases for armhf+omap4
  - Combine the add_package call with the existing place it's used further on in the script

Let me know what you think.

To post a comment you must log in.
Revision history for this message
Colin Watson (cjwatson) wrote :

I would be inclined to do the first but not the second. Otherwise this looks good to me.

review: Approve
780. By Iain Lane

Collapse case statement into one for $ARCH+$SUBARCH since it only checks for armhf+omap4

Revision history for this message
Iain Lane (laney) wrote :

> I would be inclined to do the first but not the second. Otherwise this looks
> good to me.

Cheers, done. I'll upload now.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'live-build/auto/config'
2--- live-build/auto/config 2013-07-12 09:05:35 +0000
3+++ live-build/auto/config 2013-07-16 16:16:30 +0000
4@@ -172,7 +172,21 @@
5
6 case $PROJECT in
7 ubuntu|ubuntu-dvd)
8- add_task install minimal standard ubuntu-desktop
9+ add_task install minimal standard
10+ case "$ARCH+$SUBARCH" in
11+ armhf+omap4)
12+ # Don't use the task as this drags in the regular X
13+ # stack (the metapackage doesn't). We want the
14+ # "reverted" one.
15+ add_package install ubuntu-desktop \
16+ xserver-xorg-core-omap-revert \
17+ xserver-xorg-video-omap-revert \
18+ xserver-xorg-input-evdev-omap-revert
19+ ;;
20+ *)
21+ add_task install ubuntu-desktop
22+ ;;
23+ esac
24 LIVE_TASK='ubuntu-live'
25 ;;
26
27@@ -316,7 +330,7 @@
28 add_package install u-boot-linaro-omap4-panda u-boot-tools
29 ;;
30 *)
31- add_package install u-boot-linaro-omap4-panda u-boot-tools pvr-omap4 xserver-xorg-video-omap-revert xserver-xorg-input-evdev-omap-revert
32+ add_package install u-boot-linaro-omap4-panda u-boot-tools pvr-omap4
33 ;;
34 esac
35 BINARY_REMOVE_LINUX=false

Subscribers

People subscribed via source and target branches