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

[Manjukumar Harthikote Matha]

Applied

Thanks,
Manju

> -----Original Message-----
> From: Manjukumar Harthikote Matha
> Sent: Wednesday, January 03, 2018 5:53 PM
> To: Manjukumar Harthikote Matha <MANJUKUM at xilinx.com>; meta-
> xilinx at yoctoproject.org
> Cc: Jean-Francois Dagenais <jeff.dagenais at gmail.com>
> Subject: RE: [meta-xilinx-tools][PATCH] xsct-tc: append the local system's /usr/bin
> dir as Xvfb fallback
> 
> 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