On 8/14/13 1:10 PM, Darren Hart wrote:
On Wed, 2013-08-14 at 09:41 -0700, Yang Shi wrote:
[YOCTO #4286]

The package runtime mapping rename issue is already fixed by
commit 0bc564af07c1bae8112f834a60aea3b72af7de13, the "kernel-module-" workaround
is not necessary anymore for out-of-tree module package, so revert that commit.

This reverts commit 71aafc214fe407b7620e747c11dfb8326c867b1c.


Is runtime mapping rename the right solution to this? I don't have any
experience with it, I'm not objecting, just asking the question. Is
runtime mapping rename a catchall for things that don't do the right
thing in the first place, or is it the preferred mechanism?

There we two patches that went into the main oe-core that make this change no longer relevant. The first is a change 51928b6b5ca0a46a9dcd754483a19af58b95fa18, by Martin Jansa. This automatically adds the kernel-module- prefix to kernel module packages.

The second was the recent commit 0bc564af07c1bae8112f834a60aea3b72af7de13 that allowed this change to work in a multilib environment. To keep the example being as 'clean' as possible, we want to revert the workaround as it is no longer needed.

My uninformed initial reaction is that it makes more sense to be
explicit with this sort of thing so people can know what to expect as
the output from recipes like this without having to track down what the
runtime mapping rename will do to the package output.

The initial bug and feature request was that users creating packages that provide kernel-modules shouldn't have to know they need 'kernel-module-' in the name. (Without that there are numerous QA and other checks that fail, some in ways that make it hard to figure out what went wrong.)

As things stand now, I can see from the hello-mod recipe that the
package will be called kernel-module-hello-mod, if we drop this, I would
expect it to be hello-mod if I didn't have a deep knowledge of this
rename mechanism...

Perhaps the need then is to document that the inherit of the modules bbclass will automatically name module packages w/ kernel-module- as required by the oe-core build environment?

My 0.02 USD.

Signed-off-by: Yang Shi <yang....@windriver.com>
---
  .../recipes-kernel/hello-mod/hello-mod_0.1.bb      |    8 --------
  1 files changed, 0 insertions(+), 8 deletions(-)

diff --git a/meta-skeleton/recipes-kernel/hello-mod/hello-mod_0.1.bb 
b/meta-skeleton/recipes-kernel/hello-mod/hello-mod_0.1.bb
index 621f700..64a6dda 100644
--- a/meta-skeleton/recipes-kernel/hello-mod/hello-mod_0.1.bb
+++ b/meta-skeleton/recipes-kernel/hello-mod/hello-mod_0.1.bb
@@ -13,11 +13,3 @@ SRC_URI = "file://Makefile \
            "

  S = "${WORKDIR}"
-
-# Kernel module packages MUST begin with 'kernel-module-', otherwise
-# multilib image generation can fail.
-#
-# The following line is only necessary if the recipe name does not begin
-# with kernel-module-.
-#
-PKG_${PN} = "kernel-module-${PN}"


_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Reply via email to