[PATCH] kernel-devsrc: Fix backward compaibilty

[PATCH] kernel-devsrc: Fix backward compaibilty

[Jian Liu]

kernel-dev is replaced by kernel-devsrc, which breaks
backward compaibilty.
 -- add alias for kernel-devsrc
 -- kernel-devsrc provides the kernel-dev.
 -- remove kernel-dev from the sub-packages of linux-windriver

Signed-off-by: Jian Liu <jian.liu@windriver.com>

diff --git a/meta/classes/kernel.bbclass b/meta/classes/kernel.bbclass
index dee6f7d..84ab078 100644
--- a/meta/classes/kernel.bbclass
+++ b/meta/classes/kernel.bbclass
@@ -371,11 +371,10 @@ EXPORT_FUNCTIONS do_compile do_install do_configure
 
 # kernel-base becomes kernel-${KERNEL_VERSION}
 # kernel-image becomes kernel-image-${KERNEL_VERISON}
-PACKAGES = "kernel kernel-base kernel-vmlinux kernel-image kernel-dev kernel-modules"
+PACKAGES = "kernel kernel-base kernel-vmlinux kernel-image kernel-modules"
 FILES_${PN} = ""
 FILES_kernel-base = "/lib/modules/${KERNEL_VERSION}/modules.order /lib/modules/${KERNEL_VERSION}/modules.builtin"
 FILES_kernel-image = "/boot/${KERNEL_IMAGETYPE}*"
-FILES_kernel-dev = "/boot/System.map* /boot/Module.symvers* /boot/config* ${KERNEL_SRC_PATH} /lib/modules/${KERNEL_VERSION}/build"
 FILES_kernel-vmlinux = "/boot/vmlinux*"
 FILES_kernel-modules = ""
 RDEPENDS_kernel = "kernel-base"
diff --git a/meta/conf/layer.conf b/meta/conf/layer.conf
index 7100ed7..07f8fbe 100644
--- a/meta/conf/layer.conf
+++ b/meta/conf/layer.conf
@@ -16,6 +16,8 @@ BBLAYERS_LAYERINDEX_NAME_core = "openembedded-core"
 # Set a variable to get to the top of the metadata location
 COREBASE = '${@os.path.normpath("${LAYERDIR}/../")}'
 
+PREFERRED_PROVIDER_virtual/kernel-dev = "kernel-devsrc"
+
 # opkg-utils is for update-alternatives :(
 SIGGEN_EXCLUDERECIPES_ABISAFE += " \
   sysvinit-inittab \
diff --git a/meta/recipes-kernel/linux/kernel-devsrc.bb b/meta/recipes-kernel/linux/kernel-devsrc.bb
index 196c8c7..fa641f3 100644
--- a/meta/recipes-kernel/linux/kernel-devsrc.bb
+++ b/meta/recipes-kernel/linux/kernel-devsrc.bb
@@ -74,3 +74,9 @@ do_install[lockfiles] = "${TMPDIR}/kernel-scripts.lock"
 PACKAGES = "kernel-devsrc"
 FILES_${PN} = "${KERNEL_SRC_PATH}"
 RDEPENDS_${PN} = "bc"
+
+# For backwards compatibility after rename
+RPROVIDES_${PN} = "kernel-dev"
+RREPLACES_${PN} = "kernel-dev"
+RCONFLICTS_${PN} = "kernel-dev"
+PROVIDES += "kernel-dev"

Re: [PATCH] kernel-devsrc: Fix backward compaibilty

[Richard Purdie]

On Wed, 2016-01-20 at 13:27 +0800, Jian Liu wrote:
> kernel-dev is replaced by kernel-devsrc

No, its not.

> , which breaks
> backward compaibilty.
>  -- add alias for kernel-devsrc
>  -- kernel-devsrc provides the kernel-dev.
>  -- remove kernel-dev from the sub-packages of linux-windriver

kernel-dev is useful for things like profiling tools where you need
things like system.Map or the kernel configuration without source code.

If you need the "full" source, you can use kernel-devsrc.

I'm not sure we want to remove kernel-dev given it does have a use?

Cheers,

Richard

Re: [PATCH] kernel-devsrc: Fix backward compaibilty

[Jian Liu]

On 2016年01月20日 15:52, Richard Purdie wrote:
> On Wed, 2016-01-20 at 13:27 +0800, Jian Liu wrote:
>> kernel-dev is replaced by kernel-devsrc
> No, its not.
>
>> , which breaks
>> backward compaibilty.
>>   -- add alias for kernel-devsrc
>>   -- kernel-devsrc provides the kernel-dev.
>>   -- remove kernel-dev from the sub-packages of linux-windriver
> kernel-dev is useful for things like profiling tools where you need
> things like system.Map or the kernel configuration without source code.
>
> If you need the "full" source, you can use kernel-devsrc.
>
> I'm not sure we want to remove kernel-dev given it does have a use?
Ok.  I see.

Thanks Richard.

>
> Cheers,
>
> Richard
>

Re: [PATCH] kernel-devsrc: Fix backward compaibilty

[Bruce Ashfield]

[-- Attachment #1: Type: text/plain, Size: 1524 bytes --]

On Wed, Jan 20, 2016 at 2:52 AM, Richard Purdie <
richard.purdie@linuxfoundation.org> wrote:

> On Wed, 2016-01-20 at 13:27 +0800, Jian Liu wrote:
> > kernel-dev is replaced by kernel-devsrc
>
> No, its not.
>
> > , which breaks
> > backward compaibilty.
> >  -- add alias for kernel-devsrc
> >  -- kernel-devsrc provides the kernel-dev.
> >  -- remove kernel-dev from the sub-packages of linux-windriver
>
> kernel-dev is useful for things like profiling tools where you need
> things like system.Map or the kernel configuration without source code.
>
> If you need the "full" source, you can use kernel-devsrc.
>
> I'm not sure we want to remove kernel-dev given it does have a use?
>

And I have an open bugzilla to validate (and likely fix)
kernel-dev/kernel-devsrc
against out of tree module builds, ensure that they can both the be
installed
on the target at the same time, and also so document how they map to the
packaging breakdown of kernel source in other distros.

Strangely, I was just hacking on that yesterday while working with Saul on
udev
problems.

So for the short term, leaving everything as-is makes the most sense to me.

Bruce


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



-- 
"Thou shalt not follow the NULL pointer, for chaos and madness await thee
at its end"

[-- Attachment #2: Type: text/html, Size: 2482 bytes --]