Merge lp:~asac/linaro-seeds/lp670711 into lp:~linaro-maintainers/linaro-seeds/linaro.maverick

Proposed by Alexander Sack
Status: Rejected
Rejected by: Loïc Minier
Proposed branch: lp:~asac/linaro-seeds/lp670711
Merge into: lp:~linaro-maintainers/linaro-seeds/linaro.maverick
Diff against target: 10 lines (+3/-0)
1 file modified
linaro-headless (+3/-0)
To merge this branch: bzr merge lp:~asac/linaro-seeds/lp670711
Reviewer Review Type Date Requested Status
Steve Langasek (community) Disapprove
Review via email: mp+40107@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Steve Langasek (vorlon) wrote :

Per Loïc, these packages belong in the hwpack, not in the head images. I don't want to see us flip-flopping on this - we need to come to some kind of consensus about what the right way is to solve this.

review: Disapprove
Revision history for this message
Alexander Sack (asac) wrote :

we have to wait for loic then, because i think there is a miscommunication along the line here.

Revision history for this message
Loïc Minier (lool) wrote :

I'm not sure whether Steve is referring to bootloaders/kernels being pulled in the rootfs or not.

See discussion in bug #669591 for the rationale being not using dependencies from the kernel package to bootloaders.

The choice of a kernel and a bootloader are independent and are also independent of the choice of the software you want to run with it; I don't want the kernel to depend on the bootloader, I don't want headless to depend on a board specific package.

Now if we assume all Linaro boards are going to use u-boot, it would seem that it kind of makes sense to pull in the stuff like flash-kernel and uboot-mkimage at the seed level, but thinking about it a bit, it is not too nice.

Some use cases for not installing these packages: I don't want for flash-kernel to run when I install a kernel update in a NFS root, I don't want flash-kernel to run when creating an image on an armel buildd. There are ways to prevent that from happening, but if we can avoid flash-kernel in the first place, we should.

I'm not sure if there is any disadvantage in putting that in the hwpacks instead?

Unmerged revisions

29. By Alexander Sack

add devio, flash-kernel, uboot-mkimage as recommends to linaro-headless seed (see lp:670711)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'linaro-headless'
2--- linaro-headless 2010-08-24 01:13:03 +0000
3+++ linaro-headless 2010-11-04 15:31:27 +0000
4@@ -12,4 +12,7 @@
5 == Linaro Headless ==
6
7 * (chessy-overlay-minimal) # ships branding and auto login serial/console feature
8+ * (flash-kernel) # see lp:670711
9+ * (devio) # see lp:670711
10+ * (uboot-mkimage) # see lp:670711
11

Subscribers

People subscribed via source and target branches