* [meta-oe][PATCH] fluentbit: add new recipe @ 2016-03-31 6:42 Eduardo Silva 2016-04-01 13:38 ` Martin Jansa 0 siblings, 1 reply; 10+ messages in thread From: Eduardo Silva @ 2016-03-31 6:42 UTC (permalink / raw) To: openembedded-devel Fluent Bit is an open source data collector for Embedded Linux, this patch adds the recipe for version 0.7.2. Project web site: http://fluentbit.io Documentation : http://fluentbit.io/documentation/ Signed-off-by: Eduardo Silva <eduardo@monkey.io> --- meta-oe/recipes-extended/fluentbit/fluentbit_0.7.2.bb | 15 +++++++++++++++ 1 file changed, 15 insertions(+) create mode 100644 meta-oe/recipes-extended/fluentbit/fluentbit_0.7.2.bb diff --git a/meta-oe/recipes-extended/fluentbit/fluentbit_0.7.2.bb b/meta-oe/recipes-extended/fluentbit/fluentbit_0.7.2.bb new file mode 100644 index 0000000..fd38e84 --- /dev/null +++ b/meta-oe/recipes-extended/fluentbit/fluentbit_0.7.2.bb @@ -0,0 +1,15 @@ +SUMMARY = "Fast data collector for Embedded Linux" +HOMEPAGE = "http://fluentbit.io" +BUGTRACKER = "https://github.com/fluent/fluent-bit/issues" + +SRC_URI = "http://fluentbit.io/releases/0.7/fluent-bit-${PV}.tar.gz" +SRC_URI[md5sum] = "6df9d676e1d2d782a243e655e144e8ae" +SRC_URI[sha256sum] = "695b56ce378f56855c9554f88f5d8e4f7b11ba7691284903870f779912af4ebe" +S = "${WORKDIR}/fluent-bit-${PV}" + +LICENSE = "Apache-2.0" +LIC_FILES_CHKSUM = "file://${S}/LICENSE;md5=2ee41112a44fe7014dce33e26468ba93" + +RDEPENDS_${PN} = "zlib" + +inherit cmake -- 2.5.0 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [meta-oe][PATCH] fluentbit: add new recipe 2016-03-31 6:42 [meta-oe][PATCH] fluentbit: add new recipe Eduardo Silva @ 2016-04-01 13:38 ` Martin Jansa 2016-04-01 16:53 ` Eduardo Silva 0 siblings, 1 reply; 10+ messages in thread From: Martin Jansa @ 2016-04-01 13:38 UTC (permalink / raw) To: openembedded-devel [-- Attachment #1: Type: text/plain, Size: 2015 bytes --] On Thu, Mar 31, 2016 at 12:42:52AM -0600, Eduardo Silva wrote: > Fluent Bit is an open source data collector for Embedded Linux, this > patch adds the recipe for version 0.7.2. > > Project web site: http://fluentbit.io > Documentation : http://fluentbit.io/documentation/ > > Signed-off-by: Eduardo Silva <eduardo@monkey.io> > --- > meta-oe/recipes-extended/fluentbit/fluentbit_0.7.2.bb | 15 +++++++++++++++ > 1 file changed, 15 insertions(+) > create mode 100644 meta-oe/recipes-extended/fluentbit/fluentbit_0.7.2.bb > > diff --git a/meta-oe/recipes-extended/fluentbit/fluentbit_0.7.2.bb b/meta-oe/recipes-extended/fluentbit/fluentbit_0.7.2.bb > new file mode 100644 > index 0000000..fd38e84 > --- /dev/null > +++ b/meta-oe/recipes-extended/fluentbit/fluentbit_0.7.2.bb > @@ -0,0 +1,15 @@ > +SUMMARY = "Fast data collector for Embedded Linux" > +HOMEPAGE = "http://fluentbit.io" > +BUGTRACKER = "https://github.com/fluent/fluent-bit/issues" > + > +SRC_URI = "http://fluentbit.io/releases/0.7/fluent-bit-${PV}.tar.gz" > +SRC_URI[md5sum] = "6df9d676e1d2d782a243e655e144e8ae" > +SRC_URI[sha256sum] = "695b56ce378f56855c9554f88f5d8e4f7b11ba7691284903870f779912af4ebe" > +S = "${WORKDIR}/fluent-bit-${PV}" > + > +LICENSE = "Apache-2.0" > +LIC_FILES_CHKSUM = "file://${S}/LICENSE;md5=2ee41112a44fe7014dce33e26468ba93" LIC_FILES_CHKSUM is relative to ${S}, so LIC_FILES_CHKSUM = "file://LICENSE.. should work fine > + > +RDEPENDS_${PN} = "zlib" If it does link with libz then the runtime dependency should be added automatically by shlibs code in package.bbclass if it's not the case can you add comment explaining why it needs to be added explicitly? > + > +inherit cmake > -- > 2.5.0 > > -- > _______________________________________________ > Openembedded-devel mailing list > Openembedded-devel@lists.openembedded.org > http://lists.openembedded.org/mailman/listinfo/openembedded-devel -- Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 188 bytes --] ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [meta-oe][PATCH] fluentbit: add new recipe 2016-04-01 13:38 ` Martin Jansa @ 2016-04-01 16:53 ` Eduardo Silva 2016-04-01 16:57 ` Martin Jansa 0 siblings, 1 reply; 10+ messages in thread From: Eduardo Silva @ 2016-04-01 16:53 UTC (permalink / raw) To: openembedded-devel > +LIC_FILES_CHKSUM = "file://${S}/LICENSE;md5=2ee41112a44fe7014dce33e26468ba93" > > LIC_FILES_CHKSUM is relative to ${S}, so LIC_FILES_CHKSUM = > "file://LICENSE.. should work fine > > Thanks, I have updated my local copy > > + > > +RDEPENDS_${PN} = "zlib" > > If it does link with libz then the runtime dependency should be added > automatically by shlibs code in package.bbclass if it's not the case > can you add comment explaining why it needs to be added explicitly? > > I added that because I got a QA warning message: WARNING: QA Issue: fluentbit-dev rdepends on zlib, but it isn't a build dependency? [build-deps] WARNING: QA Issue: fluentbit rdepends on zlib, but it isn't a build dependency? [build-deps] what's your suggestion in this case ? best > > + > > +inherit cmake > > -- > > 2.5.0 > > > > -- > > _______________________________________________ > > Openembedded-devel mailing list > > Openembedded-devel@lists.openembedded.org > > http://lists.openembedded.org/mailman/listinfo/openembedded-devel > > -- > Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com > > -- > _______________________________________________ > Openembedded-devel mailing list > Openembedded-devel@lists.openembedded.org > http://lists.openembedded.org/mailman/listinfo/openembedded-devel > > -- Eduardo Silva Monkey Software ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [meta-oe][PATCH] fluentbit: add new recipe 2016-04-01 16:53 ` Eduardo Silva @ 2016-04-01 16:57 ` Martin Jansa 2016-04-01 17:01 ` Eduardo Silva 0 siblings, 1 reply; 10+ messages in thread From: Martin Jansa @ 2016-04-01 16:57 UTC (permalink / raw) To: openembedded-devel Then it's probably build time dependency as the warning says, not runtime dependency like you did. Read what the component build does, if it checks for zlib in configure then you really need to add it to DEPENDS not RDEPENDS_${PN}. On Fri, Apr 1, 2016 at 6:53 PM, Eduardo Silva <eduardo@monkey.io> wrote: > > +LIC_FILES_CHKSUM = > "file://${S}/LICENSE;md5=2ee41112a44fe7014dce33e26468ba93" > > > > > LIC_FILES_CHKSUM is relative to ${S}, so LIC_FILES_CHKSUM = > > "file://LICENSE.. should work fine > > > > > Thanks, I have updated my local copy > > > > > + > > > +RDEPENDS_${PN} = "zlib" > > > > If it does link with libz then the runtime dependency should be added > > automatically by shlibs code in package.bbclass if it's not the case > > can you add comment explaining why it needs to be added explicitly? > > > > > I added that because I got a QA warning message: > > WARNING: QA Issue: fluentbit-dev rdepends on zlib, but it isn't a build > dependency? [build-deps] > WARNING: QA Issue: fluentbit rdepends on zlib, but it isn't a build > dependency? [build-deps] > > what's your suggestion in this case ? > > best > > > > > > + > > > +inherit cmake > > > -- > > > 2.5.0 > > > > > > -- > > > _______________________________________________ > > > Openembedded-devel mailing list > > > Openembedded-devel@lists.openembedded.org > > > http://lists.openembedded.org/mailman/listinfo/openembedded-devel > > > > -- > > Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com > > > > -- > > _______________________________________________ > > Openembedded-devel mailing list > > Openembedded-devel@lists.openembedded.org > > http://lists.openembedded.org/mailman/listinfo/openembedded-devel > > > > > > > -- > Eduardo Silva > Monkey Software > -- > _______________________________________________ > Openembedded-devel mailing list > Openembedded-devel@lists.openembedded.org > http://lists.openembedded.org/mailman/listinfo/openembedded-devel > ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [meta-oe][PATCH] fluentbit: add new recipe 2016-04-01 16:57 ` Martin Jansa @ 2016-04-01 17:01 ` Eduardo Silva 2016-04-01 17:20 ` Eduardo Silva 0 siblings, 1 reply; 10+ messages in thread From: Eduardo Silva @ 2016-04-01 17:01 UTC (permalink / raw) To: openembedded-devel Oh, sorry I misunderstood. The program requires zlib to run, so it's a build (headers) and runtime dependency. On Fri, Apr 1, 2016 at 10:57 AM, Martin Jansa <martin.jansa@gmail.com> wrote: > Then it's probably build time dependency as the warning says, not runtime > dependency like you did. > > Read what the component build does, if it checks for zlib in configure then > you really need to add it to DEPENDS not RDEPENDS_${PN}. > > On Fri, Apr 1, 2016 at 6:53 PM, Eduardo Silva <eduardo@monkey.io> wrote: > > > > +LIC_FILES_CHKSUM = > > "file://${S}/LICENSE;md5=2ee41112a44fe7014dce33e26468ba93" > > > > > > > > LIC_FILES_CHKSUM is relative to ${S}, so LIC_FILES_CHKSUM = > > > "file://LICENSE.. should work fine > > > > > > > > Thanks, I have updated my local copy > > > > > > > > + > > > > +RDEPENDS_${PN} = "zlib" > > > > > > If it does link with libz then the runtime dependency should be added > > > automatically by shlibs code in package.bbclass if it's not the case > > > can you add comment explaining why it needs to be added explicitly? > > > > > > > > I added that because I got a QA warning message: > > > > WARNING: QA Issue: fluentbit-dev rdepends on zlib, but it isn't a build > > dependency? [build-deps] > > WARNING: QA Issue: fluentbit rdepends on zlib, but it isn't a build > > dependency? [build-deps] > > > > what's your suggestion in this case ? > > > > best > > > > > > > > > > + > > > > +inherit cmake > > > > -- > > > > 2.5.0 > > > > > > > > -- > > > > _______________________________________________ > > > > Openembedded-devel mailing list > > > > Openembedded-devel@lists.openembedded.org > > > > http://lists.openembedded.org/mailman/listinfo/openembedded-devel > > > > > > -- > > > Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com > > > > > > -- > > > _______________________________________________ > > > Openembedded-devel mailing list > > > Openembedded-devel@lists.openembedded.org > > > http://lists.openembedded.org/mailman/listinfo/openembedded-devel > > > > > > > > > > > > -- > > Eduardo Silva > > Monkey Software > > -- > > _______________________________________________ > > Openembedded-devel mailing list > > Openembedded-devel@lists.openembedded.org > > http://lists.openembedded.org/mailman/listinfo/openembedded-devel > > > -- > _______________________________________________ > Openembedded-devel mailing list > Openembedded-devel@lists.openembedded.org > http://lists.openembedded.org/mailman/listinfo/openembedded-devel > -- Eduardo Silva Monkey Software ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [meta-oe][PATCH] fluentbit: add new recipe 2016-04-01 17:01 ` Eduardo Silva @ 2016-04-01 17:20 ` Eduardo Silva 2016-04-02 18:17 ` Martin Jansa 0 siblings, 1 reply; 10+ messages in thread From: Eduardo Silva @ 2016-04-01 17:20 UTC (permalink / raw) To: openembedded-devel on this case should be "RDEPENDS_${PN} = "zlib" OK ? On Fri, Apr 1, 2016 at 11:01 AM, Eduardo Silva <eduardo@monkey.io> wrote: > Oh, sorry I misunderstood. The program requires zlib to run, so it's a > build (headers) and runtime dependency. > > On Fri, Apr 1, 2016 at 10:57 AM, Martin Jansa <martin.jansa@gmail.com> > wrote: > >> Then it's probably build time dependency as the warning says, not runtime >> dependency like you did. >> >> Read what the component build does, if it checks for zlib in configure >> then >> you really need to add it to DEPENDS not RDEPENDS_${PN}. >> >> On Fri, Apr 1, 2016 at 6:53 PM, Eduardo Silva <eduardo@monkey.io> wrote: >> >> > > +LIC_FILES_CHKSUM = >> > "file://${S}/LICENSE;md5=2ee41112a44fe7014dce33e26468ba93" >> > >> > > >> > > LIC_FILES_CHKSUM is relative to ${S}, so LIC_FILES_CHKSUM = >> > > "file://LICENSE.. should work fine >> > > >> > > >> > Thanks, I have updated my local copy >> > >> > >> > > > + >> > > > +RDEPENDS_${PN} = "zlib" >> > > >> > > If it does link with libz then the runtime dependency should be added >> > > automatically by shlibs code in package.bbclass if it's not the case >> > > can you add comment explaining why it needs to be added explicitly? >> > > >> > > >> > I added that because I got a QA warning message: >> > >> > WARNING: QA Issue: fluentbit-dev rdepends on zlib, but it isn't a build >> > dependency? [build-deps] >> > WARNING: QA Issue: fluentbit rdepends on zlib, but it isn't a build >> > dependency? [build-deps] >> > >> > what's your suggestion in this case ? >> > >> > best >> > >> > >> > >> > > > + >> > > > +inherit cmake >> > > > -- >> > > > 2.5.0 >> > > > >> > > > -- >> > > > _______________________________________________ >> > > > Openembedded-devel mailing list >> > > > Openembedded-devel@lists.openembedded.org >> > > > http://lists.openembedded.org/mailman/listinfo/openembedded-devel >> > > >> > > -- >> > > Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com >> > > >> > > -- >> > > _______________________________________________ >> > > Openembedded-devel mailing list >> > > Openembedded-devel@lists.openembedded.org >> > > http://lists.openembedded.org/mailman/listinfo/openembedded-devel >> > > >> > > >> > >> > >> > -- >> > Eduardo Silva >> > Monkey Software >> > -- >> > _______________________________________________ >> > Openembedded-devel mailing list >> > Openembedded-devel@lists.openembedded.org >> > http://lists.openembedded.org/mailman/listinfo/openembedded-devel >> > >> -- >> _______________________________________________ >> Openembedded-devel mailing list >> Openembedded-devel@lists.openembedded.org >> http://lists.openembedded.org/mailman/listinfo/openembedded-devel >> > > > > -- > Eduardo Silva > Monkey Software > -- Eduardo Silva Monkey Software ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [meta-oe][PATCH] fluentbit: add new recipe 2016-04-01 17:20 ` Eduardo Silva @ 2016-04-02 18:17 ` Martin Jansa 2016-04-04 20:46 ` Paul Eggleton 0 siblings, 1 reply; 10+ messages in thread From: Martin Jansa @ 2016-04-02 18:17 UTC (permalink / raw) To: openembedded-devel [-- Attachment #1: Type: text/plain, Size: 3430 bytes --] On Fri, Apr 01, 2016 at 11:20:49AM -0600, Eduardo Silva wrote: > on this case should be "RDEPENDS_${PN} = "zlib" OK ? NO > On Fri, Apr 1, 2016 at 11:01 AM, Eduardo Silva <eduardo@monkey.io> wrote: > > > Oh, sorry I misunderstood. The program requires zlib to run, so it's a > > build (headers) and runtime dependency. > > > > On Fri, Apr 1, 2016 at 10:57 AM, Martin Jansa <martin.jansa@gmail.com> > > wrote: > > > >> Then it's probably build time dependency as the warning says, not runtime > >> dependency like you did. > >> > >> Read what the component build does, if it checks for zlib in configure > >> then > >> you really need to add it to DEPENDS not RDEPENDS_${PN}. > >> > >> On Fri, Apr 1, 2016 at 6:53 PM, Eduardo Silva <eduardo@monkey.io> wrote: > >> > >> > > +LIC_FILES_CHKSUM = > >> > "file://${S}/LICENSE;md5=2ee41112a44fe7014dce33e26468ba93" > >> > > >> > > > >> > > LIC_FILES_CHKSUM is relative to ${S}, so LIC_FILES_CHKSUM = > >> > > "file://LICENSE.. should work fine > >> > > > >> > > > >> > Thanks, I have updated my local copy > >> > > >> > > >> > > > + > >> > > > +RDEPENDS_${PN} = "zlib" > >> > > > >> > > If it does link with libz then the runtime dependency should be added > >> > > automatically by shlibs code in package.bbclass if it's not the case > >> > > can you add comment explaining why it needs to be added explicitly? > >> > > > >> > > > >> > I added that because I got a QA warning message: > >> > > >> > WARNING: QA Issue: fluentbit-dev rdepends on zlib, but it isn't a build > >> > dependency? [build-deps] > >> > WARNING: QA Issue: fluentbit rdepends on zlib, but it isn't a build > >> > dependency? [build-deps] > >> > > >> > what's your suggestion in this case ? > >> > > >> > best > >> > > >> > > >> > > >> > > > + > >> > > > +inherit cmake > >> > > > -- > >> > > > 2.5.0 > >> > > > > >> > > > -- > >> > > > _______________________________________________ > >> > > > Openembedded-devel mailing list > >> > > > Openembedded-devel@lists.openembedded.org > >> > > > http://lists.openembedded.org/mailman/listinfo/openembedded-devel > >> > > > >> > > -- > >> > > Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com > >> > > > >> > > -- > >> > > _______________________________________________ > >> > > Openembedded-devel mailing list > >> > > Openembedded-devel@lists.openembedded.org > >> > > http://lists.openembedded.org/mailman/listinfo/openembedded-devel > >> > > > >> > > > >> > > >> > > >> > -- > >> > Eduardo Silva > >> > Monkey Software > >> > -- > >> > _______________________________________________ > >> > Openembedded-devel mailing list > >> > Openembedded-devel@lists.openembedded.org > >> > http://lists.openembedded.org/mailman/listinfo/openembedded-devel > >> > > >> -- > >> _______________________________________________ > >> Openembedded-devel mailing list > >> Openembedded-devel@lists.openembedded.org > >> http://lists.openembedded.org/mailman/listinfo/openembedded-devel > >> > > > > > > > > -- > > Eduardo Silva > > Monkey Software > > > > > > -- > Eduardo Silva > Monkey Software > -- > _______________________________________________ > Openembedded-devel mailing list > Openembedded-devel@lists.openembedded.org > http://lists.openembedded.org/mailman/listinfo/openembedded-devel -- Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 188 bytes --] ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [meta-oe][PATCH] fluentbit: add new recipe 2016-04-02 18:17 ` Martin Jansa @ 2016-04-04 20:46 ` Paul Eggleton 2016-04-04 20:48 ` Eduardo Silva 0 siblings, 1 reply; 10+ messages in thread From: Paul Eggleton @ 2016-04-04 20:46 UTC (permalink / raw) To: Eduardo Silva; +Cc: openembedded-devel On Sat, 02 Apr 2016 20:17:14 Martin Jansa wrote: > On Fri, Apr 01, 2016 at 11:20:49AM -0600, Eduardo Silva wrote: > > on this case should be "RDEPENDS_${PN} = "zlib" OK ? > > NO It's build-time so you want zlib in DEPENDS. A runtime dependency will get added automatically (since binaries packaged by the recipe will be linking against libz.so). Cheers, Paul -- Paul Eggleton Intel Open Source Technology Centre ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [meta-oe][PATCH] fluentbit: add new recipe 2016-04-04 20:46 ` Paul Eggleton @ 2016-04-04 20:48 ` Eduardo Silva 2016-04-06 10:41 ` Felipe Ferreri Tonello 0 siblings, 1 reply; 10+ messages in thread From: Eduardo Silva @ 2016-04-04 20:48 UTC (permalink / raw) To: Paul Eggleton; +Cc: openembedded-devel Thanks. Already sent a Patch v2. On Apr 4, 2016 1:46 PM, "Paul Eggleton" <paul.eggleton@linux.intel.com> wrote: > On Sat, 02 Apr 2016 20:17:14 Martin Jansa wrote: > > On Fri, Apr 01, 2016 at 11:20:49AM -0600, Eduardo Silva wrote: > > > on this case should be "RDEPENDS_${PN} = "zlib" OK ? > > > > NO > > It's build-time so you want zlib in DEPENDS. A runtime dependency will get > added automatically (since binaries packaged by the recipe will be linking > against libz.so). > > Cheers, > Paul > > -- > > Paul Eggleton > Intel Open Source Technology Centre > ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [meta-oe][PATCH] fluentbit: add new recipe 2016-04-04 20:48 ` Eduardo Silva @ 2016-04-06 10:41 ` Felipe Ferreri Tonello 0 siblings, 0 replies; 10+ messages in thread From: Felipe Ferreri Tonello @ 2016-04-06 10:41 UTC (permalink / raw) To: openembedded-devel, eduardo [-- Attachment #1: Type: text/plain, Size: 707 bytes --] Hi Eduardo, On 04/04/16 21:48, Eduardo Silva wrote: > Thanks. Already sent a Patch v2. Please, avoid top-posting. > On Apr 4, 2016 1:46 PM, "Paul Eggleton" <paul.eggleton@linux.intel.com> > wrote: > >> On Sat, 02 Apr 2016 20:17:14 Martin Jansa wrote: >>> On Fri, Apr 01, 2016 at 11:20:49AM -0600, Eduardo Silva wrote: >>>> on this case should be "RDEPENDS_${PN} = "zlib" OK ? >>> >>> NO >> >> It's build-time so you want zlib in DEPENDS. A runtime dependency will get >> added automatically (since binaries packaged by the recipe will be linking >> against libz.so). >> >> Cheers, >> Paul >> >> -- >> >> Paul Eggleton >> Intel Open Source Technology Centre >> -- Felipe [-- Attachment #2: 0x92698E6A.asc --] [-- Type: application/pgp-keys, Size: 7310 bytes --] ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2016-04-06 10:38 UTC | newest] Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2016-03-31 6:42 [meta-oe][PATCH] fluentbit: add new recipe Eduardo Silva 2016-04-01 13:38 ` Martin Jansa 2016-04-01 16:53 ` Eduardo Silva 2016-04-01 16:57 ` Martin Jansa 2016-04-01 17:01 ` Eduardo Silva 2016-04-01 17:20 ` Eduardo Silva 2016-04-02 18:17 ` Martin Jansa 2016-04-04 20:46 ` Paul Eggleton 2016-04-04 20:48 ` Eduardo Silva 2016-04-06 10:41 ` Felipe Ferreri Tonello
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.