Openembedded Core Discussions
 help / color / mirror / Atom feed
* [PATCH] piglit: ship files with correct permissions
@ 2014-03-25 12:06 Valentin Popa
  2014-03-25 14:34 ` Burton, Ross
  0 siblings, 1 reply; 2+ messages in thread
From: Valentin Popa @ 2014-03-25 12:06 UTC (permalink / raw)
  To: openembedded-core

Adress the warning
| warning: group pulse does not exist - using root

Piglit files from framework/ generated_tests/ tests/
and templates/ belong to xuser/pulse.
Don't keep the permissions while shipping the files,
they should be root/root.

[YOCTO #6028]

Signed-off-by: Valentin Popa <valentin.popa@intel.com>
---
 meta/recipes-graphics/piglit/piglit_git.bb | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/meta/recipes-graphics/piglit/piglit_git.bb b/meta/recipes-graphics/piglit/piglit_git.bb
index 8616764..7417cd5 100644
--- a/meta/recipes-graphics/piglit/piglit_git.bb
+++ b/meta/recipes-graphics/piglit/piglit_git.bb
@@ -33,11 +33,11 @@ do_install() {
 	install -d ${D}${libdir}/piglit/bin
 	install -m 755 ${S}/bin/* ${D}${libdir}/piglit/bin
 
-	cp -a lib/ ${D}${libdir}/piglit/
-	cp -a framework/ ${D}${libdir}/piglit/
-	cp -a generated_tests/ ${D}${libdir}/piglit/
-	cp -a tests/ ${D}${libdir}/piglit/
-	cp -a templates/ ${D}${libdir}/piglit/
+	cp -Pr lib/ ${D}${libdir}/piglit/
+	cp -Pr framework/ ${D}${libdir}/piglit/
+	cp -Pr generated_tests/ ${D}${libdir}/piglit/
+	cp -Pr tests/ ${D}${libdir}/piglit/
+	cp -Pr templates/ ${D}${libdir}/piglit/
 
 	sed -i -e 's|sys.path.append(.*)|sys.path.append("${libdir}/piglit")|' ${D}${bindir}/piglit-*.py
 	sed -i -e 's|^templatedir = .*$|templatedir = "${libdir}/piglit/templates"|' ${D}${bindir}/piglit-summary-html.py
-- 
1.8.3.2



^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] piglit: ship files with correct permissions
  2014-03-25 12:06 [PATCH] piglit: ship files with correct permissions Valentin Popa
@ 2014-03-25 14:34 ` Burton, Ross
  0 siblings, 0 replies; 2+ messages in thread
From: Burton, Ross @ 2014-03-25 14:34 UTC (permalink / raw)
  To: Valentin Popa; +Cc: OE-core

On 25 March 2014 12:06, Valentin Popa <valentin.popa@intel.com> wrote:
> Adress the warning
> | warning: group pulse does not exist - using root
>
> Piglit files from framework/ generated_tests/ tests/
> and templates/ belong to xuser/pulse.
> Don't keep the permissions while shipping the files,
> they should be root/root.

Reviewed-by: Ross Burton <ross.burton@intel.com>

Ross


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2014-03-25 14:34 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-03-25 12:06 [PATCH] piglit: ship files with correct permissions Valentin Popa
2014-03-25 14:34 ` Burton, Ross

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox