| Message ID | 20250311143744.102671-1-afd@ti.com |
|---|---|
| State | Superseded |
| Delegated to: | Ryan Eatmon |
| Headers | show |
| Series | [meta-arago,master/scarthgap] meta-arago-extras: Add recipe for NNShark | expand |
Hello Andrew, On 11/03/25 20:07, Andrew Davis via lists.yoctoproject.org wrote: > Signed-off-by: Andrew Davis <afd@ti.com> > --- > .../recipes-multimedia/nnshark/nnshark_git.bb | 38 +++++++++++++++++++ > 1 file changed, 38 insertions(+) > create mode 100644 meta-arago-extras/recipes-multimedia/nnshark/nnshark_git.bb > > diff --git a/meta-arago-extras/recipes-multimedia/nnshark/nnshark_git.bb b/meta-arago-extras/recipes-multimedia/nnshark/nnshark_git.bb > new file mode 100644 > index 00000000..1a3d8ad5 > --- /dev/null > +++ b/meta-arago-extras/recipes-multimedia/nnshark/nnshark_git.bb > @@ -0,0 +1,38 @@ > +SUMMARY = "A GStreamer NNstreamer pipelines real-time profiling plugin" nitpick, the below sounds much better? SUMMARY = "A GStreamer plugin to support real-time performance profiling of NNStreamer pipelines." > +HOMEPAGE = "https://github.com/nnstreamer/nnshark" > + > +LICENSE = "GPL-2.0-only & LGPL-2.1-only" > +LIC_FILES_CHKSUM = "file://COPYING;md5=e1caa368743492879002ad032445fa97 \ > + file://COPYING.LESSER;md5=66c40c88533cd228b5b85936709801c8" > + > +SRC_URI = "git://github.com/nnstreamer/nnshark.git;protocol=https;branch=master;name=nnshark \ > + git://github.com/GStreamer/common.git;protocol=https;branch=master;name=common;subdir=${S}/common \ > +" any specific reasons to not go with something like below, SRC_URI = "gitsm://github.com/nnstreamer/nnshark.git;protocol=https;branch=master \ " SRCREV = "bcee91b23a98c981b87051fe1b2ad545991c4aa2" > + > +SRCREV_FORMAT="default" > +SRCREV_nnshark="bcee91b23a98c981b87051fe1b2ad545991c4aa2" > +SRCREV_common="b64f03f6090245624608beb5d2fff335e23a01c0" > + > +S = "${WORKDIR}/git" probably master might need a separate patch for UNPACKDIR stuff? -- Chirag > + > +DEPENDS = "\ > + gtk-doc-native \ > + gstreamer1.0 \ > + gstreamer1.0-plugins-base \ > +" > + > +inherit pkgconfig autotools-brokensep > + > +EXTRA_OECONF = " \ > + --disable-graphviz \ > + --disable-gtk-doc \ > +" > + > +do_configure:prepend() { > + sh autogen.sh --noconfigure > +} > + > +FILES:${PN} += "\ > + ${libdir}/gstreamer-1.0/libgstsharktracers.so \ > + ${libdir}/gstreamer-1.0/libgstsharktracers.la \ > +"
On Tue, Mar 11, 2025 at 09:37:44AM -0500, Andrew Davis via lists.yoctoproject.org wrote: > Signed-off-by: Andrew Davis <afd@ti.com> > --- > .../recipes-multimedia/nnshark/nnshark_git.bb | 38 +++++++++++++++++++ > 1 file changed, 38 insertions(+) > create mode 100644 meta-arago-extras/recipes-multimedia/nnshark/nnshark_git.bb > > diff --git a/meta-arago-extras/recipes-multimedia/nnshark/nnshark_git.bb b/meta-arago-extras/recipes-multimedia/nnshark/nnshark_git.bb > new file mode 100644 > index 00000000..1a3d8ad5 > --- /dev/null > +++ b/meta-arago-extras/recipes-multimedia/nnshark/nnshark_git.bb > @@ -0,0 +1,38 @@ > +SUMMARY = "A GStreamer NNstreamer pipelines real-time profiling plugin" > +HOMEPAGE = "https://github.com/nnstreamer/nnshark" > + > +LICENSE = "GPL-2.0-only & LGPL-2.1-only" > +LIC_FILES_CHKSUM = "file://COPYING;md5=e1caa368743492879002ad032445fa97 \ > + file://COPYING.LESSER;md5=66c40c88533cd228b5b85936709801c8" > + > +SRC_URI = "git://github.com/nnstreamer/nnshark.git;protocol=https;branch=master;name=nnshark \ > + git://github.com/GStreamer/common.git;protocol=https;branch=master;name=common;subdir=${S}/common \ > +" > + > +SRCREV_FORMAT="default" > +SRCREV_nnshark="bcee91b23a98c981b87051fe1b2ad545991c4aa2" > +SRCREV_common="b64f03f6090245624608beb5d2fff335e23a01c0" You might want to fix these ^ https://lists.yoctoproject.org/g/meta-ti/message/18319 > +S = "${WORKDIR}/git" > + > +DEPENDS = "\ > + gtk-doc-native \ > + gstreamer1.0 \ > + gstreamer1.0-plugins-base \ > +" > + > +inherit pkgconfig autotools-brokensep > + > +EXTRA_OECONF = " \ > + --disable-graphviz \ > + --disable-gtk-doc \ > +" > + > +do_configure:prepend() { > + sh autogen.sh --noconfigure > +} > + > +FILES:${PN} += "\ > + ${libdir}/gstreamer-1.0/libgstsharktracers.so \ > + ${libdir}/gstreamer-1.0/libgstsharktracers.la \ > +" > -- > 2.39.2
On Tue, Mar 11, 2025 at 09:28:44PM +0530, Chirag Shilwant via lists.yoctoproject.org wrote: > Hello Andrew, > > On 11/03/25 20:07, Andrew Davis via lists.yoctoproject.org wrote: > >Signed-off-by: Andrew Davis <afd@ti.com> > >--- > > .../recipes-multimedia/nnshark/nnshark_git.bb | 38 +++++++++++++++++++ > > 1 file changed, 38 insertions(+) > > create mode 100644 meta-arago-extras/recipes-multimedia/nnshark/nnshark_git.bb > > > >diff --git a/meta-arago-extras/recipes-multimedia/nnshark/nnshark_git.bb b/meta-arago-extras/recipes-multimedia/nnshark/nnshark_git.bb > >new file mode 100644 > >index 00000000..1a3d8ad5 > >--- /dev/null > >+++ b/meta-arago-extras/recipes-multimedia/nnshark/nnshark_git.bb > >@@ -0,0 +1,38 @@ > >+SUMMARY = "A GStreamer NNstreamer pipelines real-time profiling plugin" > > > nitpick, the below sounds much better? > > SUMMARY = "A GStreamer plugin to support real-time performance > profiling of NNStreamer pipelines." This is 86+ characters long, which is too long for the SUMMARY (72 max). Also, no dots at the end of the SUMMARY, only in the DESCRIPTION, which happens automatically if not set explicitly. > >+HOMEPAGE = "https://github.com/nnstreamer/nnshark" > >+ > >+LICENSE = "GPL-2.0-only & LGPL-2.1-only" > >+LIC_FILES_CHKSUM = "file://COPYING;md5=e1caa368743492879002ad032445fa97 \ > >+ file://COPYING.LESSER;md5=66c40c88533cd228b5b85936709801c8" > >+ > >+SRC_URI = "git://github.com/nnstreamer/nnshark.git;protocol=https;branch=master;name=nnshark \ > >+ git://github.com/GStreamer/common.git;protocol=https;branch=master;name=common;subdir=${S}/common \ > >+" > > any specific reasons to not go with something like below, > > SRC_URI = "gitsm://github.com/nnstreamer/nnshark.git;protocol=https;branch=master > \ > " > > SRCREV = "bcee91b23a98c981b87051fe1b2ad545991c4aa2" Unless you want to control each SRCREV separately. > >+SRCREV_FORMAT="default" > >+SRCREV_nnshark="bcee91b23a98c981b87051fe1b2ad545991c4aa2" > >+SRCREV_common="b64f03f6090245624608beb5d2fff335e23a01c0" > >+ > >+S = "${WORKDIR}/git" > > probably master might need a separate patch for UNPACKDIR stuff? No changes for git based recipes needed. > >+ > >+DEPENDS = "\ > >+ gtk-doc-native \ > >+ gstreamer1.0 \ > >+ gstreamer1.0-plugins-base \ > >+" > >+ > >+inherit pkgconfig autotools-brokensep > >+ > >+EXTRA_OECONF = " \ > >+ --disable-graphviz \ > >+ --disable-gtk-doc \ > >+" > >+ > >+do_configure:prepend() { > >+ sh autogen.sh --noconfigure > >+} > >+ > >+FILES:${PN} += "\ > >+ ${libdir}/gstreamer-1.0/libgstsharktracers.so \ > >+ ${libdir}/gstreamer-1.0/libgstsharktracers.la \ > >+"
On 3/11/25 11:26 AM, Denys Dmytriyenko wrote: > On Tue, Mar 11, 2025 at 09:28:44PM +0530, Chirag Shilwant via lists.yoctoproject.org wrote: >> Hello Andrew, >> >> On 11/03/25 20:07, Andrew Davis via lists.yoctoproject.org wrote: >>> Signed-off-by: Andrew Davis <afd@ti.com> >>> --- >>> .../recipes-multimedia/nnshark/nnshark_git.bb | 38 +++++++++++++++++++ >>> 1 file changed, 38 insertions(+) >>> create mode 100644 meta-arago-extras/recipes-multimedia/nnshark/nnshark_git.bb >>> >>> diff --git a/meta-arago-extras/recipes-multimedia/nnshark/nnshark_git.bb b/meta-arago-extras/recipes-multimedia/nnshark/nnshark_git.bb >>> new file mode 100644 >>> index 00000000..1a3d8ad5 >>> --- /dev/null >>> +++ b/meta-arago-extras/recipes-multimedia/nnshark/nnshark_git.bb >>> @@ -0,0 +1,38 @@ >>> +SUMMARY = "A GStreamer NNstreamer pipelines real-time profiling plugin" >> >> >> nitpick, the below sounds much better? >> >> SUMMARY = "A GStreamer plugin to support real-time performance >> profiling of NNStreamer pipelines." > > This is 86+ characters long, which is too long for the SUMMARY (72 max). Also, > no dots at the end of the SUMMARY, only in the DESCRIPTION, which happens > automatically if not set explicitly. > > >>> +HOMEPAGE = "https://github.com/nnstreamer/nnshark" >>> + >>> +LICENSE = "GPL-2.0-only & LGPL-2.1-only" >>> +LIC_FILES_CHKSUM = "file://COPYING;md5=e1caa368743492879002ad032445fa97 \ >>> + file://COPYING.LESSER;md5=66c40c88533cd228b5b85936709801c8" >>> + >>> +SRC_URI = "git://github.com/nnstreamer/nnshark.git;protocol=https;branch=master;name=nnshark \ >>> + git://github.com/GStreamer/common.git;protocol=https;branch=master;name=common;subdir=${S}/common \ >>> +" >> >> any specific reasons to not go with something like below, >> >> SRC_URI = "gitsm://github.com/nnstreamer/nnshark.git;protocol=https;branch=master >> \ >> " >> >> SRCREV = "bcee91b23a98c981b87051fe1b2ad545991c4aa2" > > Unless you want to control each SRCREV separately. > Right, that is the reason I do this, the version of "common" when pulled as a submodule is not available, we have the exact same issue for TI GST plugins and we patch around[0]. This way of pulling "common" manually with SRC_URI seemed better. Andrew [0] https://git.yoctoproject.org/meta-arago/tree/meta-arago-extras/recipes-multimedia/gst-plugins-ti/gstreamer1.0-plugins-vpe/0001-Switch-submodule-common-to-github.patch > >>> +SRCREV_FORMAT="default" >>> +SRCREV_nnshark="bcee91b23a98c981b87051fe1b2ad545991c4aa2" >>> +SRCREV_common="b64f03f6090245624608beb5d2fff335e23a01c0" >>> + >>> +S = "${WORKDIR}/git" >> >> probably master might need a separate patch for UNPACKDIR stuff? > > No changes for git based recipes needed. > > >>> + >>> +DEPENDS = "\ >>> + gtk-doc-native \ >>> + gstreamer1.0 \ >>> + gstreamer1.0-plugins-base \ >>> +" >>> + >>> +inherit pkgconfig autotools-brokensep >>> + >>> +EXTRA_OECONF = " \ >>> + --disable-graphviz \ >>> + --disable-gtk-doc \ >>> +" >>> + >>> +do_configure:prepend() { >>> + sh autogen.sh --noconfigure >>> +} >>> + >>> +FILES:${PN} += "\ >>> + ${libdir}/gstreamer-1.0/libgstsharktracers.so \ >>> + ${libdir}/gstreamer-1.0/libgstsharktracers.la \ >>> +"
diff --git a/meta-arago-extras/recipes-multimedia/nnshark/nnshark_git.bb b/meta-arago-extras/recipes-multimedia/nnshark/nnshark_git.bb new file mode 100644 index 00000000..1a3d8ad5 --- /dev/null +++ b/meta-arago-extras/recipes-multimedia/nnshark/nnshark_git.bb @@ -0,0 +1,38 @@ +SUMMARY = "A GStreamer NNstreamer pipelines real-time profiling plugin" +HOMEPAGE = "https://github.com/nnstreamer/nnshark" + +LICENSE = "GPL-2.0-only & LGPL-2.1-only" +LIC_FILES_CHKSUM = "file://COPYING;md5=e1caa368743492879002ad032445fa97 \ + file://COPYING.LESSER;md5=66c40c88533cd228b5b85936709801c8" + +SRC_URI = "git://github.com/nnstreamer/nnshark.git;protocol=https;branch=master;name=nnshark \ + git://github.com/GStreamer/common.git;protocol=https;branch=master;name=common;subdir=${S}/common \ +" + +SRCREV_FORMAT="default" +SRCREV_nnshark="bcee91b23a98c981b87051fe1b2ad545991c4aa2" +SRCREV_common="b64f03f6090245624608beb5d2fff335e23a01c0" + +S = "${WORKDIR}/git" + +DEPENDS = "\ + gtk-doc-native \ + gstreamer1.0 \ + gstreamer1.0-plugins-base \ +" + +inherit pkgconfig autotools-brokensep + +EXTRA_OECONF = " \ + --disable-graphviz \ + --disable-gtk-doc \ +" + +do_configure:prepend() { + sh autogen.sh --noconfigure +} + +FILES:${PN} += "\ + ${libdir}/gstreamer-1.0/libgstsharktracers.so \ + ${libdir}/gstreamer-1.0/libgstsharktracers.la \ +"
Signed-off-by: Andrew Davis <afd@ti.com> --- .../recipes-multimedia/nnshark/nnshark_git.bb | 38 +++++++++++++++++++ 1 file changed, 38 insertions(+) create mode 100644 meta-arago-extras/recipes-multimedia/nnshark/nnshark_git.bb