[PATCH v2] kernel.bbclass: fix emit_depmod_pkgdata() workdir

[PATCH v2] kernel.bbclass: fix emit_depmod_pkgdata() workdir

[Ioan-Adrian Ratiu]

bitbake commit 67a7b8b02 "build: don't use $B as the default cwd for
functions" breaks the assumption that this function is running under
${B}. This causes build failures because System.map is under ${B}.
Fix this by using an absolute path instead of relying on cwd.

Signed-off-by: Ioan-Adrian Ratiu <adrian.ratiu@ni.com>
---
 meta/classes/kernel.bbclass | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/classes/kernel.bbclass b/meta/classes/kernel.bbclass
index 062a0df..f3e1bee 100644
--- a/meta/classes/kernel.bbclass
+++ b/meta/classes/kernel.bbclass
@@ -359,7 +359,7 @@ emit_depmod_pkgdata() {
 	# Stash data for depmod
 	install -d ${PKGDESTWORK}/${KERNEL_PACKAGE_NAME}-depmod/
 	echo "${KERNEL_VERSION}" > ${PKGDESTWORK}/${KERNEL_PACKAGE_NAME}-depmod/${KERNEL_PACKAGE_NAME}-abiversion
-	cp System.map ${PKGDESTWORK}/${KERNEL_PACKAGE_NAME}-depmod/System.map-${KERNEL_VERSION}
+	cp ${B}/System.map ${PKGDESTWORK}/${KERNEL_PACKAGE_NAME}-depmod/System.map-${KERNEL_VERSION}
 }
 
 PACKAGEFUNCS += "emit_depmod_pkgdata"
-- 
2.9.2

Re: [PATCH v2] kernel.bbclass: fix emit_depmod_pkgdata() workdir

[Burton, Ross]

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

On 11 August 2016 at 10:39, Ioan-Adrian Ratiu <adrian.ratiu@ni.com> wrote:

> bitbake commit 67a7b8b02 "build: don't use $B as the default cwd for
> functions" breaks the assumption that this function is running under
> ${B}. This causes build failures because System.map is under ${B}.
> Fix this by using an absolute path instead of relying on cwd.
>

This failed to apply and I noticed that I fixed this myself in July...

Ross

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

Re: [PATCH v2] kernel.bbclass: fix emit_depmod_pkgdata() workdir

[Ioan-Adrian Ratiu]

On Thu, 11 Aug 2016, "Burton, Ross" <ross.burton@intel.com> wrote:
> On 11 August 2016 at 10:39, Ioan-Adrian Ratiu <adrian.ratiu@ni.com> wrote:
>
>> bitbake commit 67a7b8b02 "build: don't use $B as the default cwd for
>> functions" breaks the assumption that this function is running under
>> ${B}. This causes build failures because System.map is under ${B}.
>> Fix this by using an absolute path instead of relying on cwd.
>>
>
> This failed to apply and I noticed that I fixed this myself in July...

What commit id is the fix on the master branch? I can't find it..

>
> Ross

Re: [PATCH v2] kernel.bbclass: fix emit_depmod_pkgdata() workdir

[Ioan-Adrian Ratiu]

On Thu, 11 Aug 2016, Ioan-Adrian Ratiu <adrian.ratiu@ni.com> wrote:
> On Thu, 11 Aug 2016, "Burton, Ross" <ross.burton@intel.com> wrote:
>> On 11 August 2016 at 10:39, Ioan-Adrian Ratiu <adrian.ratiu@ni.com> wrote:
>>
>>> bitbake commit 67a7b8b02 "build: don't use $B as the default cwd for
>>> functions" breaks the assumption that this function is running under
>>> ${B}. This causes build failures because System.map is under ${B}.
>>> Fix this by using an absolute path instead of relying on cwd.
>>>
>>
>> This failed to apply and I noticed that I fixed this myself in July...
>
> What commit id is the fix on the master branch? I can't find it..
>

I managed to find it d596286bc4cf1977 thanks!

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