Merge lp:~tom-gall/linaro-seeds/linaro-alip-on-a-diet into lp:~linaro-maintainers/linaro-seeds/linaro.maverick

Proposed by Tom Gall
Status: Merged
Merged at revision: 38
Proposed branch: lp:~tom-gall/linaro-seeds/linaro-alip-on-a-diet
Merge into: lp:~linaro-maintainers/linaro-seeds/linaro.maverick
Diff against target: 28 lines (+1/-7)
1 file modified
linaro-alip (+1/-7)
To merge this branch: bzr merge lp:~tom-gall/linaro-seeds/linaro-alip-on-a-diet
Reviewer Review Type Date Requested Status
Jamie Bennett (community) Approve
Review via email: mp+37624@code.launchpad.net

This proposal supersedes a proposal from 2010-10-05.

Description of the change

Hopefully bzr handles this right, this should merge a subset of changes that looks like some other folks were a bit quicker to submit than I :-)

remove evince, evolution-data-server, screensaver-default-images, xscreensaver, xscreensaver-data, xscreensaver-gl

replace gnome-terminal with xfce4-terminal

To post a comment you must log in.
Revision history for this message
Tom Gall (tom-gall) wrote : Posted in a previous version of this proposal

From the diff looks like it's mostly right.

One question, why was jockey-gtk removed? I would think we would want that.

ibus-m17n appears that is a miss-re-add and I will remove and resubmit.

Revision history for this message
Jamie Bennett (jamiebennett) wrote : Posted in a previous version of this proposal

Any hardware dependant device drivers should be shipped with the hwpack on this image making jockey redundant.

Please resubmit with the conflicts from Steve's patch resolved.

review: Needs Fixing
Revision history for this message
Steve Langasek (vorlon) wrote : Posted in a previous version of this proposal

On Tue, Oct 05, 2010 at 02:44:26PM -0000, Tom Gall wrote:
> Tom Gall has proposed merging
> lp:~tom-gall/linaro-seeds/linaro-alip-on-a-diet into lp:linaro-seeds.

> Requested reviews:
> Linaro Maintainers (linaro-maintainers)

> Hopefully bzr handles this right, this should merge a subset of changes
> that looks like some other folks were a bit quicker to submit than I :-)

> remove evince, evolution-data-server, screensaver-default-images,
> xscreensaver, xscreensaver-data, xscreensaver-gl

evolution-data-server is explicitly listed as a component of the GNOME
Mobile Platform at <http://www.gnome.org/mobile>. Maybe it doesn't require
direct seeding, but that just means this change should be a no-op at best if
something else pulls it in. If this actually removes e-d-s from the image,
I think we should check with ARM that this is what they want.

--
Steve Langasek Give me a lever long enough and a Free OS
Debian Developer to set it on, and I can move the world.
Ubuntu Developer http://www.debian.org/
<email address hidden> <email address hidden>

Revision history for this message
Steve Langasek (vorlon) wrote : Posted in a previous version of this proposal

On Tue, Oct 05, 2010 at 02:59:46PM -0000, Tom Gall wrote:
> Hopefully bzr handles this right, this should merge a subset of changes
> that looks like some other folks were a bit quicker to submit than I :-)

> remove evince, evolution-data-server, screensaver-default-images,
> xscreensaver, xscreensaver-data, xscreensaver-gl

evolution-data-server is explicitly listed as a component of the GNOME
Mobile Platform at <http://www.gnome.org/mobile>. Maybe it doesn't require
direct seeding, but that just means this change should be a no-op at best if
something else pulls it in. If this actually removes e-d-s from the image,
I think we should check with ARM that this is what they want.

--
Steve Langasek Give me a lever long enough and a Free OS
Debian Developer to set it on, and I can move the world.
Ubuntu Developer http://www.debian.org/
<email address hidden> <email address hidden>

Revision history for this message
Tom Gall (tom-gall) wrote :

Right, evolution-data-server doesn't need need explicit seeding. It's still present in my local validation test images.

Revision history for this message
Jamie Bennett (jamiebennett) wrote :

Approving but you need to check with ARM whether e-d-s is required for the images. If it is I would like to see this explicit in the seed, if not we need to investigate how to remove it fully.

review: Approve
38. By Jamie Bennett

Merge a patch from Tom Gall to trim down the ALIP image

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'linaro-alip'
2--- linaro-alip 2010-10-05 07:33:32 +0000
3+++ linaro-alip 2010-10-05 15:38:00 +0000
4@@ -9,7 +9,6 @@
5
6 * consolekit
7 * doc-base
8- * evolution-data-server
9 * expat
10 * flash-kernel
11 * gdebi
12@@ -42,15 +41,10 @@
13
14 # and recommends
15
16- * (evince)
17 * (firefox)
18- * (gnome-terminal)
19 * (linaro-alip-default-settings)
20 * (notify-osd)
21 * (onboard)
22- * (screensaver-default-images)
23 * (vinagre)
24 * (xcursor-themes)
25- * (xscreensaver)
26- * (xscreensaver-data)
27- * (xscreensaver-gl)
28+ * (xfce4-terminal)

Subscribers

People subscribed via source and target branches

to all changes: