Hi Naveen,

On Mon, 2019-05-13 at 14:49 +0800, Naveen Saini wrote:

Can you please add what it is in commit message?

> Signed-off-by: Lim, Raynald <[email protected]>
> Signed-off-by: Naveen Saini <[email protected]>
> ---
>  recipes-multimedia/itt/itt_17.0.bb | 28 ++++++++++++++++++++++++++++
>  1 file changed, 28 insertions(+)
>  create mode 100644 recipes-multimedia/itt/itt_17.0.bb
> 
> diff --git a/recipes-multimedia/itt/itt_17.0.bb b/recipes-
> multimedia/itt/itt_17.0.bb
> new file mode 100644
> index 00000000..f32bd8b7
> --- /dev/null
> +++ b/recipes-multimedia/itt/itt_17.0.bb
> @@ -0,0 +1,28 @@
> +SUMMARY = "IntelĀ® Single Event API"
> +DESCRIPTION = "IntelĀ® SEAPI is the translator of itt_notify calls
> into \
> +several OS specific and third party tracing formats. \
> +You can use it as memory/performance/whatever profiler."
> +
> +LICENSE = "BSD-3-Clause"
> +LIC_FILES_CHKSUM =
> "file://sea_itt_lib/Copyright.txt;md5=7d54dfc8860742fb06b9c5ad28f41fc
> d"
> +
> +SRC_URI = "\
> +     git://github.com/intel/IntelSEAPI.git;protocols=https \
> +"
> +SRCREV="36bff07521afffc0c0f7db79252338954ca7e6dd"
> +S = "${WORKDIR}/git"
> +
> +inherit pkgconfig
> +DEPENDS += "python cmake-native"
> +

Why is python2 needed?

> +do_compile() {
> +     python ${S}/buildall.py -i
> +}

Can this be changed to inherit cmake directly and pass these flags
here:

https://github.com/intel/IntelSEAPI/blob/master/buildall.py#L289

> +
> +do_install() {
> +     install -d -m 755 ${D}${libdir} ${D}${includedir}
> +     install -m 644 ${S}/bin/*.a ${D}${libdir}
> +     install -m 644 ${S}/ittnotify/include/ittnotify.h
> ${D}${includedir}
> +}
> +
> +ALLOW_EMPTY_${PN} = "1"
> -- 
> 2.17.0
> 

-- 
_______________________________________________
meta-intel mailing list
[email protected]
https://lists.yoctoproject.org/listinfo/meta-intel

Reply via email to