On 08/13/2015 11:10 PM, Saul Wold wrote:
On 08/12/2015 08:15 PM, Pengyu Ma wrote:
# ~/checkbashisms -f
/usr/lib64/intel-gpu-tools/intel-gpu-tools/check_drm_clients
possible bashism in
/usr/lib64/intel-gpu-tools/intel-gpu-tools/check_drm_clients line 3
(bash arrays, ${name[0|*|@]}):
SOURCE_DIR="$( dirname "${BASH_SOURCE[0]}" )"
possible bashism in
/usr/lib64/intel-gpu-tools/intel-gpu-tools/check_drm_clients line 3
($BASH_SOMETHING):
SOURCE_DIR="$( dirname "${BASH_SOURCE[0]}" )"

It seems there are some bash specific script.
Shall bash is rdepended in this package instead of remove bash?

Yes, if there are bashisms that can not be fixed easily then we should have a RDEPENDS = "bash" added to the recipe to ensure bash is installed for these tools.

OK, I will send V4 version.

Pengyu
Sau!

Pengyu

On 08/12/2015 11:33 PM, Burton, Ross wrote:

On 12 August 2015 at 16:17, Saul Wold <s...@linux.intel.com
<mailto:s...@linux.intel.com>> wrote:

    If you are running on a system with bash the /bin/sh will point to
    bash so there is no real change, the real test is to install these
    tools on a system without Bash and ensure they work.


Or run checkbashisms, or just read the scripts.

Ross



--
_______________________________________________
meta-intel mailing list
meta-intel@yoctoproject.org
https://lists.yoctoproject.org/listinfo/meta-intel

Reply via email to