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

[Manjukumar Harthikote Matha]

Hi Jeff,

> -----Original Message-----
> From: Jean-Francois Dagenais [mailto:jeff.dagenais at gmail.com]
> Sent: Thursday, July 06, 2017 10:17 AM
> To: Manjukumar Harthikote Matha <MANJUKUM at xilinx.com>; Sai Pavan Boddu
> <saipava at xilinx.com>; Cyril Chemparathy <cyrilc at xilinx.com>
> Cc: git <git at xilinx.com>; michal.simek at xilinx.com; meta-xilinx at yoctoproject.org;
> Jean-Francois Dagenais <jeff.dagenais at gmail.com>
> Subject: [PATCH 3/7] xsct-tc: append the local system's /usr/bin dir as Xvfb fallback
>
> 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>
> ---
>  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"


I will check with team here on this requirement. I am not sure if it is a right idea to include /usr/bin in the PATH variable as of now. Maybe there is a way to append this in local.conf or set it in your local PATH

Thanks
Manju

>
>  def xsct_run(d):
>      import bb.process
> --
> 2.1.4



This email and any attachments are intended for the sole use of the named recipient(s) and contain(s) confidential information that may be proprietary, privileged or copyrighted under applicable law. If you are not the intended recipient, do not read, copy, or forward this email message or any attachments. Delete this email message and any attachments immediately.