diff options
author | Robert Yang <liezhi.yang@windriver.com> | 2012-05-29 22:53:06 +0800 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2012-05-30 12:04:45 +0100 |
commit | e40995e569289598a1d9d71e19734402f2b54718 (patch) | |
tree | 108328e272a149da0e27dec0e0f0bebe602b80b8 /meta/recipes-extended/cups | |
parent | e4c35790d6dc23a0933f188f52fa4434784e1d98 (diff) | |
download | poky-e40995e569289598a1d9d71e19734402f2b54718.tar.gz |
meta: replace os.system with subprocess.call
Replace os.system with subprocess.call since the older function would
fail (more or less) silently if the executed program cannot be found
More info:
http://docs.python.org/library/subprocess.html#subprocess-replacements
[YOCTO #2454]
(From OE-Core rev: a07d03cc6f67c88feb9813ae7deb6e4a93552dfe)
Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/recipes-extended/cups')
-rw-r--r-- | meta/recipes-extended/cups/cups14.inc | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/meta/recipes-extended/cups/cups14.inc b/meta/recipes-extended/cups/cups14.inc index 7c1968220d..85d8ca232f 100644 --- a/meta/recipes-extended/cups/cups14.inc +++ b/meta/recipes-extended/cups/cups14.inc | |||
@@ -57,9 +57,10 @@ fakeroot do_install () { | |||
57 | } | 57 | } |
58 | 58 | ||
59 | python do_package_append() { | 59 | python do_package_append() { |
60 | import subprocess | ||
60 | # Change permissions back the way they were, they probably had a reason... | 61 | # Change permissions back the way they were, they probably had a reason... |
61 | workdir = d.getVar('WORKDIR', True) | 62 | workdir = d.getVar('WORKDIR', True) |
62 | os.system('chmod 0511 %s/install/cups/var/run/cups/certs' % workdir) | 63 | subprocess.call('chmod 0511 %s/install/cups/var/run/cups/certs' % workdir, shell=True) |
63 | } | 64 | } |
64 | 65 | ||
65 | PACKAGES =+ "${PN}-lib ${PN}-libimage" | 66 | PACKAGES =+ "${PN}-lib ${PN}-libimage" |