[meta-freescale] [meta-fsl-arm][PATCH] imx-gpu-viv: correct file ownership

[sujith h]

Hi Chris,

On Thu, Apr 21, 2016 at 8:16 PM, Larson, Chris <Chris_Larson at mentor.com>
wrote:

> There's no need for this to use a postfunc, just integrate it into
> do_install. Use of the postfunc was an artifact of the fact that it was a
> bbappend.
>

Sure,  I have created second version of the patch based on the review
comment.


> -Chris
> ________________________________________
> From: Sujith Haridasan [sujith.h at gmail.com]
> Sent: Thursday, April 21, 2016 6:04 AM
> To: meta-freescale at yoctoproject.org
> Cc: Sujith H; Larson, Chris; Haridasan, Sujith
> Subject: [meta-freescale] [meta-fsl-arm][PATCH] imx-gpu-viv: correct file
> ownership
>
> From: Sujith H <sujith.h at gmail.com>
>
> This recipe copies binary files with `cp`, including `cp -a`, so the file
> ownership can end up being the same as the build user. Work around this by
> chown'ing everything to root:root.
>
> This avoids a package_qa failure.
>
> Signed-off-by: Sujith H <sujith.h at gmail.com>
> Signed-off-by: Christopher Larson <chris_larson at mentor.com>
> Signed-off-by: Sujith Haridasan <Sujith_Haridasan at mentor.com>
> ---
>  recipes-graphics/imx-gpu-viv/imx-gpu-viv.inc | 5 +++++
>  1 file changed, 5 insertions(+)
>
> diff --git a/recipes-graphics/imx-gpu-viv/imx-gpu-viv.inc
> b/recipes-graphics/imx-gpu-viv/imx-gpu-viv.inc
> index ce00724..151ae9b 100644
> --- a/recipes-graphics/imx-gpu-viv/imx-gpu-viv.inc
> +++ b/recipes-graphics/imx-gpu-viv/imx-gpu-viv.inc
> @@ -211,6 +211,11 @@ do_install () {
>      find ${D}${includedir} -type f -exec chmod 644 {} \;
>  }
>
> +do_install[postfuncs] += "fixup_perms"
> +fixup_perms () {
> +    chown -R root:root "${D}"
> +}
> +
>  ALLOW_EMPTY_${PN} = "1"
>
>  FILES_libclc-mx6 = "${libdir}/libCLC${SOLIBS}"
> --
> 1.9.1
>
>


-- 
സുജിത് ഹരിദാസന്
Bangalore
<Project>Contributor to KDE project
<Project>Contributor to Yocto project
http://fci.wikia.com/wiki/Anti-DRM-Campaign
<Blog> http://sujithh.info
C-x C-c
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.yoctoproject.org/pipermail/meta-freescale/attachments/20160421/b295d804/attachment.html>