[oe] [PATCH v3] opencl-headers: Initial recipe for OpenCL headers

[Khem Raj]

On Tue, Oct 16, 2018 at 7:06 AM Ankit Navik <ankit.tarot at gmail.com> wrote:
>
> Hi Raj,
> On Tue, Oct 16, 2018 at 7:17 PM Khem Raj <raj.khem at gmail.com> wrote:
> >
> > On Tue, Oct 16, 2018 at 5:37 AM Ankit Navik <ankit.tarot at gmail.com> wrote:
> > >
> > > Hi Ross
> > > On Tue, Oct 16, 2018 at 5:01 PM Burton, Ross <ross.burton at intel.com> wrote:
> > > >
> > > > On Tue, 16 Oct 2018 at 05:43, Ankit Navik <ankit.tarot at gmail.com> wrote:
> > > >
> > > > > +do_install () {
> > > > > +       install -d ${D}${includedir}/CL/
> > > > > +       install -m 0644 -D -t ${D}${includedir}/CL ${S}/CL/*.h
> > > > > +}
> > > >
> > > > -D to install is 'create directories' which you've already done in the
> > > > previous line, so that can be removed.
> > > >
> > > > If you're removing -D then a more idiomatic form would be:
> > > >
> > > > +       install -d ${D}${includedir}/CL/
> > > > +       install -m 0644 ${S}/CL/*.h ${D}${includedir}/CL
> > > Opps, missed -D.
> > > Cool. I will use your suggestion.
> >
> > you just need single line there in do_install
> >
> >  install -m 0644 -D -t ${D}${includedir}/CL ${S}/CL/*.h
> Single line solution is not working as I said in earlier email, it
> throws an error as No such file and directory.

yes it wont now I see that its using wildcards. So send the final
patch with Ross's suggestions and we can move on

> Regards, Ankit
> >
> >
> > >
> > > Thanks, Ankit
> > > >
> > > > Ross
> > > --
> > > _______________________________________________
> > > Openembedded-devel mailing list
> > > Openembedded-devel at lists.openembedded.org
> > > http://lists.openembedded.org/mailman/listinfo/openembedded-devel