Patchwork image: Add kmod-native to DEPENDS

login
register
mail settings
Submitter Phil Blundell
Date Jan. 20, 2014, 3:31 p.m.
Message ID <1390231901.2441.5.camel@e130.pbcl.net>
Download mbox | patch
Permalink /patch/65279/
State New
Headers show

Comments

Phil Blundell - Jan. 20, 2014, 3:31 p.m.
We will run depmod so we need it to be in the sysroot.  (We do depend on
depmodwrapper-cross already but this doesn't have any dependency on
depmod itself.)

SIgned-off-by: Phil Blundell <pb@pbcl.net>
---
 meta/classes/image.bbclass | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Paul Eggleton - Jan. 20, 2014, 3:35 p.m.
Hi Phil,

On Monday 20 January 2014 15:31:41 Phil Blundell wrote:
> We will run depmod so we need it to be in the sysroot.  (We do depend on
> depmodwrapper-cross already but this doesn't have any dependency on
> depmod itself.)

Perhaps I'm not grasping the subtleties but shouldn't the dependency be added 
to depmodwrapper-cross rather than here?

Cheers,
Paul

Patch

diff --git a/meta/classes/image.bbclass b/meta/classes/image.bbclass
index a954b69..1c8ad28 100644
--- a/meta/classes/image.bbclass
+++ b/meta/classes/image.bbclass
@@ -10,7 +10,7 @@  inherit gzipnative
 
 LICENSE = "MIT"
 PACKAGES = ""
-DEPENDS += "${MLPREFIX}qemuwrapper-cross ${MLPREFIX}depmodwrapper-cross"
+DEPENDS += "${MLPREFIX}qemuwrapper-cross ${MLPREFIX}depmodwrapper-cross kmod-native"
 RDEPENDS += "${PACKAGE_INSTALL} ${LINGUAS_INSTALL}"
 RRECOMMENDS += "${PACKAGE_INSTALL_ATTEMPTONLY}"