From e6d3676b591a8b020b2dbd8b7b2c2e40c5446608 Mon Sep 17 00:00:00 2001 From: California Sullivan Date: Wed, 23 Aug 2017 12:56:10 -0700 Subject: lms: use sh instead of bash in notifyDesktop.sh A recent update to OE-core revealed the missing runtime dependency: ERROR: lms8-8.0.0-7-r0 do_package_qa: QA Issue: /usr/bin/notifyDesktop.sh contained in package lms8 requires /bin/bash, but no providers found in RDEPENDS_lms8? [file-rdeps] But it turns out that notifyDesktop doesn't actually require bash: $ checkbashisms.pl notifyDesktop.sh could not find any possible bashisms in bash script notifyDesktop.sh So just change the shebang line to /bin/sh. Signed-off-by: California Sullivan CC: mikko.ylinen@linux.intel.com CC: anand.vastrad@intel.com Signed-off-by: Saul Wold --- .../amt/lms/0004-Intel-AMT-ME-real-time-notification-infra.patch | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/common/recipes-bsp/amt/lms/0004-Intel-AMT-ME-real-time-notification-infra.patch b/common/recipes-bsp/amt/lms/0004-Intel-AMT-ME-real-time-notification-infra.patch index 6866705a..04f584ae 100644 --- a/common/recipes-bsp/amt/lms/0004-Intel-AMT-ME-real-time-notification-infra.patch +++ b/common/recipes-bsp/amt/lms/0004-Intel-AMT-ME-real-time-notification-infra.patch @@ -1672,7 +1672,7 @@ index 0000000..06bfd97 --- /dev/null +++ b/src/notifyDesktop.sh @@ -0,0 +1,10 @@ -+#!/bin/bash ++#!/bin/sh +# get list of current users +for x in $(who | sort -u -k1,1 | awk '{print $1}'); do + # send message to anyone on display 0 -- cgit v1.2.3-54-g00ecf