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

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

[Ankit Navik]

Add generic recipe for OpenCL API headers.

Suggested-by: Burton, Ross <ross.burton@intel.com>
Signed-off-by: Ankit Navik <ankit.tarot@gmail.com>
---
 .../recipes-core/opencl-headers/opencl-headers_git.bb   | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)
 create mode 100644 meta-oe/recipes-core/opencl-headers/opencl-headers_git.bb

diff --git a/meta-oe/recipes-core/opencl-headers/opencl-headers_git.bb b/meta-oe/recipes-core/opencl-headers/opencl-headers_git.bb
new file mode 100644
index 0000000..818fd31
--- /dev/null
+++ b/meta-oe/recipes-core/opencl-headers/opencl-headers_git.bb
@@ -0,0 +1,17 @@
+SUMMARY  = "OpenCL API Headers"
+DESCRIPTION = "OpenCL compute API headers from Khronos Group"
+LICENSE  = "Khronos"
+LIC_FILES_CHKSUM = "file://LICENSE;md5=dcefc90f4c3c689ec0c2489064e7273b"
+SECTION = "base"
+
+S = "${WORKDIR}/git"
+SRCREV = "40c5d226c7c0706f0176884e9b94b3886679c983"
+SRC_URI = "git://github.com/KhronosGroup/OpenCL-Headers.git"
+
+do_configure[noexec] = "1"
+do_compile[noexec] = "1"
+
+do_install () {
+	install -d ${D}${includedir}/CL/
+	install -m 0644 -D -t ${D}${includedir}/CL ${S}/CL/*.h
+}
-- 
1.9.1

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

[Burton, Ross]

On Tue, 16 Oct 2018 at 05:43, Ankit Navik <ankit.tarot@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

Ross

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

[Ankit Navik]

Hi Ross
On Tue, Oct 16, 2018 at 5:01 PM Burton, Ross <ross.burton@intel.com> wrote:
>
> On Tue, 16 Oct 2018 at 05:43, Ankit Navik <ankit.tarot@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.

Thanks, Ankit
>
> Ross

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

[Khem Raj]

On Tue, Oct 16, 2018 at 5:37 AM Ankit Navik <ankit.tarot@gmail.com> wrote:
>
> Hi Ross
> On Tue, Oct 16, 2018 at 5:01 PM Burton, Ross <ross.burton@intel.com> wrote:
> >
> > On Tue, 16 Oct 2018 at 05:43, Ankit Navik <ankit.tarot@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


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

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

[Ankit Navik]

Hi Raj,
On Tue, Oct 16, 2018 at 7:17 PM Khem Raj <raj.khem@gmail.com> wrote:
>
> On Tue, Oct 16, 2018 at 5:37 AM Ankit Navik <ankit.tarot@gmail.com> wrote:
> >
> > Hi Ross
> > On Tue, Oct 16, 2018 at 5:01 PM Burton, Ross <ross.burton@intel.com> wrote:
> > >
> > > On Tue, 16 Oct 2018 at 05:43, Ankit Navik <ankit.tarot@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.
Regards, Ankit
>
>
> >
> > Thanks, Ankit
> > >
> > > Ross
> > --
> > _______________________________________________
> > Openembedded-devel mailing list
> > Openembedded-devel@lists.openembedded.org
> > http://lists.openembedded.org/mailman/listinfo/openembedded-devel

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

[Khem Raj]

On Tue, Oct 16, 2018 at 7:06 AM Ankit Navik <ankit.tarot@gmail.com> wrote:
>
> Hi Raj,
> On Tue, Oct 16, 2018 at 7:17 PM Khem Raj <raj.khem@gmail.com> wrote:
> >
> > On Tue, Oct 16, 2018 at 5:37 AM Ankit Navik <ankit.tarot@gmail.com> wrote:
> > >
> > > Hi Ross
> > > On Tue, Oct 16, 2018 at 5:01 PM Burton, Ross <ross.burton@intel.com> wrote:
> > > >
> > > > On Tue, 16 Oct 2018 at 05:43, Ankit Navik <ankit.tarot@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@lists.openembedded.org
> > > http://lists.openembedded.org/mailman/listinfo/openembedded-devel

[PATCH v4] opencl-headers: Initial recipe for OpenCL headers

[Ankit Navik]

Add generic recipe for OpenCL API headers.

Suggested-by: Burton, Ross <ross.burton@intel.com>
Signed-off-by: Ankit Navik <ankit.tarot@gmail.com>
---
 .../recipes-core/opencl-headers/opencl-headers_git.bb   | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)
 create mode 100644 meta-oe/recipes-core/opencl-headers/opencl-headers_git.bb

diff --git a/meta-oe/recipes-core/opencl-headers/opencl-headers_git.bb b/meta-oe/recipes-core/opencl-headers/opencl-headers_git.bb
new file mode 100644
index 0000000..dec1bea
--- /dev/null
+++ b/meta-oe/recipes-core/opencl-headers/opencl-headers_git.bb
@@ -0,0 +1,17 @@
+SUMMARY  = "OpenCL API Headers"
+DESCRIPTION = "OpenCL compute API headers from Khronos Group"
+LICENSE  = "Khronos"
+LIC_FILES_CHKSUM = "file://LICENSE;md5=dcefc90f4c3c689ec0c2489064e7273b"
+SECTION = "base"
+
+S = "${WORKDIR}/git"
+SRCREV = "40c5d226c7c0706f0176884e9b94b3886679c983"
+SRC_URI = "git://github.com/KhronosGroup/OpenCL-Headers.git"
+
+do_configure[noexec] = "1"
+do_compile[noexec] = "1"
+
+do_install () {
+	install -d ${D}${includedir}/CL/
+	install -m 0644 ${S}/CL/*.h ${D}${includedir}/CL
+}
-- 
1.9.1