[meta-intel] [PATCH] lms8: Add bash to RDEPENDS

Mikko Ylinen mikko.ylinen at linux.intel.com
Mon Aug 21 11:37:55 PDT 2017


Hi,

On 19/08/17 03:48, California Sullivan wrote:
> An 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]
>

notifyDesktop.sh has no dependency to bash [1] so how about just
change the shebang line:

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 6866705..04f584a 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

[1] $ checkbashisms.pl notifyDesktop.sh
could not find any possible bashisms in bash script notifyDesktop.sh

-- Mikko


More information about the meta-intel mailing list