[meta-xilinx] [meta-xilinx-tools][PATCH] xsct-tc: append the local system's /usr/bin dir as Xvfb fallback

[Manjukumar Harthikote Matha]

Hi All,

Please note this patch is temporary till 2018.1 XSDK tool is released. We are trying to fix the hardcoded Xvfb path in XSDK script for 2018.1. Once the change is implemented, we can add Xvfb and xlsclients to HOSTTOOLS instead of this patch.

Thanks,
Manju

> -----Original Message-----
> From: Manjukumar Matha [mailto:manjukumar.harthikote-matha at xilinx.com]
> Sent: Wednesday, January 03, 2018 4:11 PM
> To: meta-xilinx at yoctoproject.org
> Cc: Jean-Francois Dagenais <jeff.dagenais at gmail.com>; Manjukumar Harthikote
> Matha <MANJUKUM at xilinx.com>
> Subject: [meta-xilinx-tools][PATCH] xsct-tc: append the local system's /usr/bin dir as
> Xvfb fallback
> 
> From: Jean-Francois Dagenais <jeff.dagenais at gmail.com>
> 
> See changes to README.md.
> 
> Essentially, we need to fallback to locally found Xvfb and xlsclients since the xserver-
> xorg recipe won't build without "x11" in DISTRO_FEATURE.
> 
> Signed-off-by: Jean-Francois Dagenais <jeff.dagenais at gmail.com>
> Signed-off-by: Manjukumar Matha <manjukumar.harthikote-matha at xilinx.com>
> ---
>  README.md               | 9 ++++++++-
>  classes/xsct-tc.bbclass | 1 +
>  2 files changed, 9 insertions(+), 1 deletion(-)
> 
> diff --git a/README.md b/README.md
> index 2acd4aa..057923a 100644
> --- a/README.md
> +++ b/README.md
> @@ -22,7 +22,14 @@ Dependencies
> 
>  This layer depends on:
> 
> -XSCT being installed somewhere the yocto build can access. You must define
> +* **If you don't have "x11" in your DISTRO_FEATURES, you'll need
> +`xlsclients` and `Xvfb` to satisfy the SDK's dependence on it. You may
> +install these by `apt-get install -y xvfb x11-utils` or the like. Note
> +that if your system distro doesn't install those in /usr/bin, you'll
> +need to provide links for them in /usr/bin since xsct (from the SDK
> +itself) not only depends on these but it actually assumes /usr/bin.
> +
> +* XSCT being installed somewhere the yocto build can access. You must
> +define
>  this path by specifying the XILINX_SDK_TOOLCHAIN variable globally, typically  in
> your local.conf (or site.conf).
> 
> diff --git a/classes/xsct-tc.bbclass b/classes/xsct-tc.bbclass index 937fbd2..f124919
> 100644
> --- a/classes/xsct-tc.bbclass
> +++ b/classes/xsct-tc.bbclass
> @@ -1,5 +1,6 @@
>  XSCT_PATH_ADD = "${XILINX_SDK_TOOLCHAIN}/bin:"
>  PATH =. "${XSCT_PATH_ADD}"
> +PATH_append = ":/usr/bin"
> 
>  def xsct_run(d):
>      import bb.process
> --
> 2.7.4