All the mail mirrored from lore.kernel.org
 help / color / mirror / Atom feed
* Re: [meta-oe][PATCH] sysdig: 0.1.101 -> 0.1.102
  2015-08-21 16:40 [meta-oe][PATCH] sysdig: 0.1.101 -> 0.1.102 Bian Naimeng
@ 2015-08-21 15:42 ` Khem Raj
  2015-08-25  0:44   ` Bian, Naimeng
  0 siblings, 1 reply; 6+ messages in thread
From: Khem Raj @ 2015-08-21 15:42 UTC (permalink / raw
  To: openembeded-devel

On Fri, Aug 21, 2015 at 9:40 AM, Bian Naimeng <biannm@cn.fujitsu.com> wrote:
> upgrade sysdig to 0.1.102, and using GIT source instead of archive.
>
> Signed-off-by: Bian Naimeng <biannm@cn.fujitsu.com>
> ---
>  .../01-donot-set-default-value-to-va_list.patch    | 22 --------------
>  meta-oe/recipes-extended/sysdig/sysdig_0.1.101.bb  | 34 ----------------------
>  meta-oe/recipes-extended/sysdig/sysdig_git.bb      | 32 ++++++++++++++++++++
>  3 files changed, 32 insertions(+), 56 deletions(-)
>  delete mode 100644 meta-oe/recipes-extended/sysdig/sysdig/01-donot-set-default-value-to-va_list.patch
>  delete mode 100644 meta-oe/recipes-extended/sysdig/sysdig_0.1.101.bb
>  create mode 100644 meta-oe/recipes-extended/sysdig/sysdig_git.bb
>
> diff --git a/meta-oe/recipes-extended/sysdig/sysdig/01-donot-set-default-value-to-va_list.patch b/meta-oe/recipes-extended/sysdig/sysdig/01-donot-set-default-value-to-va_list.patch
> deleted file mode 100644
> index 9cb6378..0000000
> --- a/meta-oe/recipes-extended/sysdig/sysdig/01-donot-set-default-value-to-va_list.patch
> +++ /dev/null

why is this patch dropped ?

> @@ -1,22 +0,0 @@
> -fix build error as following.
> -
> -  sysdig-0.1.101/userspace/libsinsp/ctext.h:376:50: error: could not convert '0' from 'int' to 'va_list {aka __va_list}'
> -  |    int8_t vprintf(const char*format, va_list ap = 0);
> -  |                                                   ^
> -  | make[2]: *** [userspace/libsinsp/CMakeFiles/sinsp.dir/ctext.cpp.o] Error 1
> -
> -
> -Signed-off-by: Bian Naimeng <biannm@cn.fujitsu.com>
> -
> -diff -Nurp sysdig-0.1.101.orig/userspace/libsinsp/ctext.h sysdig-0.1.101/userspace/libsinsp/ctext.h
> ---- sysdig-0.1.101.orig/userspace/libsinsp/ctext.h     2015-06-04 02:58:51.000000000 +0800
> -+++ sysdig-0.1.101/userspace/libsinsp/ctext.h  2015-07-02 09:18:14.154627357 +0800
> -@@ -373,7 +373,7 @@ class ctext
> -               // application to this library seamlessly.
> -               //
> -               int8_t printf(const char*format, ...);
> --              int8_t vprintf(const char*format, va_list ap = 0);
> -+              int8_t vprintf(const char*format, va_list ap);
> -
> -               //
> -               // nprintf is identical to the printf above EXCEPT for
> diff --git a/meta-oe/recipes-extended/sysdig/sysdig_0.1.101.bb b/meta-oe/recipes-extended/sysdig/sysdig_0.1.101.bb
> deleted file mode 100644
> index 789fd47..0000000
> --- a/meta-oe/recipes-extended/sysdig/sysdig_0.1.101.bb
> +++ /dev/null
> @@ -1,34 +0,0 @@
> -SUMMARY = "A New System Troubleshooting Tool Built for the Way You Work"
> -DESCRIPTION = "Sysdig is open source, system-level exploration: capture \
> -system state and activity from a running Linux instance, then save, \
> -filter and analyze."
> -HOMEPAGE = "http://www.sysdig.org/"
> -LICENSE = "GPLv2"
> -LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263"
> -
> -inherit cmake pkgconfig
> -
> -DEPENDS = "luajit zlib ncurses"
> -RDEPENDS_${PN} = "bash"
> -
> -
> -SRC_URI = "https://github.com/draios/sysdig/archive/${PV}.tar.gz \
> -           file://01-donot-set-default-value-to-va_list.patch \
> -          "
> -
> -SRC_URI[md5sum] = "5fe96a3a0fd98b2157a40cb29af41afc"
> -SRC_URI[sha256sum] = "6995e39be565514901b5cb587689ee2efbf8359293e4e597362382cccf0e9db6"
> -
> -DIR_ETC="/etc"
> -EXTRA_OECMAKE = ' -DUSE_BUNDLED_LUAJIT="OFF" \
> -                  -DUSE_BUNDLED_ZLIB="OFF" \
> -                  -DBUILD_DRIVER="OFF" \
> -                  -DUSE_BUNDLED_NCURSES="OFF" \
> -                  -DDIR_ETC="${DIR_ETC}" \
> -                '
> -
> -FILES_${PN} += " \
> -    ${DIR_ETC}/* \
> -    ${datadir}/zsh/* \
> -    ${prefix}/src/*  \
> -"
> diff --git a/meta-oe/recipes-extended/sysdig/sysdig_git.bb b/meta-oe/recipes-extended/sysdig/sysdig_git.bb
> new file mode 100644
> index 0000000..62aa433
> --- /dev/null
> +++ b/meta-oe/recipes-extended/sysdig/sysdig_git.bb
> @@ -0,0 +1,32 @@
> +SUMMARY = "A New System Troubleshooting Tool Built for the Way You Work"
> +DESCRIPTION = "Sysdig is open source, system-level exploration: capture \
> +system state and activity from a running Linux instance, then save, \
> +filter and analyze."
> +HOMEPAGE = "http://www.sysdig.org/"
> +LICENSE = "GPLv2"
> +LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263"
> +
> +inherit cmake pkgconfig
> +
> +DEPENDS = "luajit zlib ncurses"
> +RDEPENDS_${PN} = "bash"
> +
> +SRC_URI = "git://github.com/draios/sysdig.git;branch=master"
> +SRCREV = "b7394e29ced4f1a991af03c0381a5828abcbab7a"
> +PV = "0.1.102+git${SRCPV}"
> +
> +S = "${WORKDIR}/git"
> +
> +DIR_ETC="/etc"
> +EXTRA_OECMAKE = ' -DUSE_BUNDLED_LUAJIT="OFF" \
> +                  -DUSE_BUNDLED_ZLIB="OFF" \
> +                  -DBUILD_DRIVER="OFF" \
> +                  -DUSE_BUNDLED_NCURSES="OFF" \
> +                  -DDIR_ETC="${DIR_ETC}" \
> +                '
> +
> +FILES_${PN} += " \
> +    ${DIR_ETC}/* \
> +    ${datadir}/zsh/* \
> +    ${prefix}/src/*  \
> +"
> --
> 1.8.4.2
>
> --
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [meta-oe][PATCH] sysdig: 0.1.101 -> 0.1.102
@ 2015-08-21 16:40 Bian Naimeng
  2015-08-21 15:42 ` Khem Raj
  0 siblings, 1 reply; 6+ messages in thread
From: Bian Naimeng @ 2015-08-21 16:40 UTC (permalink / raw
  To: openembedded-devel

upgrade sysdig to 0.1.102, and using GIT source instead of archive.

Signed-off-by: Bian Naimeng <biannm@cn.fujitsu.com>
---
 .../01-donot-set-default-value-to-va_list.patch    | 22 --------------
 meta-oe/recipes-extended/sysdig/sysdig_0.1.101.bb  | 34 ----------------------
 meta-oe/recipes-extended/sysdig/sysdig_git.bb      | 32 ++++++++++++++++++++
 3 files changed, 32 insertions(+), 56 deletions(-)
 delete mode 100644 meta-oe/recipes-extended/sysdig/sysdig/01-donot-set-default-value-to-va_list.patch
 delete mode 100644 meta-oe/recipes-extended/sysdig/sysdig_0.1.101.bb
 create mode 100644 meta-oe/recipes-extended/sysdig/sysdig_git.bb

diff --git a/meta-oe/recipes-extended/sysdig/sysdig/01-donot-set-default-value-to-va_list.patch b/meta-oe/recipes-extended/sysdig/sysdig/01-donot-set-default-value-to-va_list.patch
deleted file mode 100644
index 9cb6378..0000000
--- a/meta-oe/recipes-extended/sysdig/sysdig/01-donot-set-default-value-to-va_list.patch
+++ /dev/null
@@ -1,22 +0,0 @@
-fix build error as following.
-
-  sysdig-0.1.101/userspace/libsinsp/ctext.h:376:50: error: could not convert '0' from 'int' to 'va_list {aka __va_list}'
-  |    int8_t vprintf(const char*format, va_list ap = 0);
-  |                                                   ^
-  | make[2]: *** [userspace/libsinsp/CMakeFiles/sinsp.dir/ctext.cpp.o] Error 1
-
-
-Signed-off-by: Bian Naimeng <biannm@cn.fujitsu.com>
-
-diff -Nurp sysdig-0.1.101.orig/userspace/libsinsp/ctext.h sysdig-0.1.101/userspace/libsinsp/ctext.h
---- sysdig-0.1.101.orig/userspace/libsinsp/ctext.h	2015-06-04 02:58:51.000000000 +0800
-+++ sysdig-0.1.101/userspace/libsinsp/ctext.h	2015-07-02 09:18:14.154627357 +0800
-@@ -373,7 +373,7 @@ class ctext
- 		// application to this library seamlessly.
- 		//
- 		int8_t printf(const char*format, ...);
--		int8_t vprintf(const char*format, va_list ap = 0);
-+		int8_t vprintf(const char*format, va_list ap);
- 
- 		//
- 		// nprintf is identical to the printf above EXCEPT for
diff --git a/meta-oe/recipes-extended/sysdig/sysdig_0.1.101.bb b/meta-oe/recipes-extended/sysdig/sysdig_0.1.101.bb
deleted file mode 100644
index 789fd47..0000000
--- a/meta-oe/recipes-extended/sysdig/sysdig_0.1.101.bb
+++ /dev/null
@@ -1,34 +0,0 @@
-SUMMARY = "A New System Troubleshooting Tool Built for the Way You Work"
-DESCRIPTION = "Sysdig is open source, system-level exploration: capture \
-system state and activity from a running Linux instance, then save, \
-filter and analyze."
-HOMEPAGE = "http://www.sysdig.org/"
-LICENSE = "GPLv2"
-LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263"
-
-inherit cmake pkgconfig
-
-DEPENDS = "luajit zlib ncurses"
-RDEPENDS_${PN} = "bash"
-
-
-SRC_URI = "https://github.com/draios/sysdig/archive/${PV}.tar.gz \
-           file://01-donot-set-default-value-to-va_list.patch \
-          "
-
-SRC_URI[md5sum] = "5fe96a3a0fd98b2157a40cb29af41afc"
-SRC_URI[sha256sum] = "6995e39be565514901b5cb587689ee2efbf8359293e4e597362382cccf0e9db6"
-
-DIR_ETC="/etc"
-EXTRA_OECMAKE = ' -DUSE_BUNDLED_LUAJIT="OFF" \
-                  -DUSE_BUNDLED_ZLIB="OFF" \
-                  -DBUILD_DRIVER="OFF" \
-                  -DUSE_BUNDLED_NCURSES="OFF" \
-                  -DDIR_ETC="${DIR_ETC}" \
-                '
-
-FILES_${PN} += " \
-    ${DIR_ETC}/* \
-    ${datadir}/zsh/* \ 
-    ${prefix}/src/*  \
-"
diff --git a/meta-oe/recipes-extended/sysdig/sysdig_git.bb b/meta-oe/recipes-extended/sysdig/sysdig_git.bb
new file mode 100644
index 0000000..62aa433
--- /dev/null
+++ b/meta-oe/recipes-extended/sysdig/sysdig_git.bb
@@ -0,0 +1,32 @@
+SUMMARY = "A New System Troubleshooting Tool Built for the Way You Work"
+DESCRIPTION = "Sysdig is open source, system-level exploration: capture \
+system state and activity from a running Linux instance, then save, \
+filter and analyze."
+HOMEPAGE = "http://www.sysdig.org/"
+LICENSE = "GPLv2"
+LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263"
+
+inherit cmake pkgconfig
+
+DEPENDS = "luajit zlib ncurses"
+RDEPENDS_${PN} = "bash"
+
+SRC_URI = "git://github.com/draios/sysdig.git;branch=master"
+SRCREV = "b7394e29ced4f1a991af03c0381a5828abcbab7a"
+PV = "0.1.102+git${SRCPV}"
+
+S = "${WORKDIR}/git"
+
+DIR_ETC="/etc"
+EXTRA_OECMAKE = ' -DUSE_BUNDLED_LUAJIT="OFF" \
+                  -DUSE_BUNDLED_ZLIB="OFF" \
+                  -DBUILD_DRIVER="OFF" \
+                  -DUSE_BUNDLED_NCURSES="OFF" \
+                  -DDIR_ETC="${DIR_ETC}" \
+                '
+
+FILES_${PN} += " \
+    ${DIR_ETC}/* \
+    ${datadir}/zsh/* \ 
+    ${prefix}/src/*  \
+"
-- 
1.8.4.2



^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [meta-oe][PATCH] sysdig: 0.1.101 -> 0.1.102
  2015-08-21 15:42 ` Khem Raj
@ 2015-08-25  0:44   ` Bian, Naimeng
  2015-08-25  1:00     ` Khem Raj
  0 siblings, 1 reply; 6+ messages in thread
From: Bian, Naimeng @ 2015-08-25  0:44 UTC (permalink / raw
  To: openembedded-devel@lists.openembedded.org



> -----Original Message-----
> From: openembedded-devel-bounces@lists.openembedded.org
> [mailto:openembedded-devel-bounces@lists.openembedded.org] On Behalf Of Khem
> Raj
> Sent: Friday, August 21, 2015 11:42 PM
> To: openembeded-devel
> Subject: Re: [oe] [meta-oe][PATCH] sysdig: 0.1.101 -> 0.1.102
> 
> On Fri, Aug 21, 2015 at 9:40 AM, Bian Naimeng <biannm@cn.fujitsu.com> wrote:
> > upgrade sysdig to 0.1.102, and using GIT source instead of archive.
> >
> > Signed-off-by: Bian Naimeng <biannm@cn.fujitsu.com>
> > ---
> >  .../01-donot-set-default-value-to-va_list.patch    | 22 --------------
> >  meta-oe/recipes-extended/sysdig/sysdig_0.1.101.bb  | 34
> ----------------------
> >  meta-oe/recipes-extended/sysdig/sysdig_git.bb      | 32
> ++++++++++++++++++++
> >  3 files changed, 32 insertions(+), 56 deletions(-)  delete mode
> > 100644
> > meta-oe/recipes-extended/sysdig/sysdig/01-donot-set-default-value-to-v
> > a_list.patch  delete mode 100644
> > meta-oe/recipes-extended/sysdig/sysdig_0.1.101.bb
> >  create mode 100644 meta-oe/recipes-extended/sysdig/sysdig_git.bb
> >
> > diff --git
> > a/meta-oe/recipes-extended/sysdig/sysdig/01-donot-set-default-value-to
> > -va_list.patch
> > b/meta-oe/recipes-extended/sysdig/sysdig/01-donot-set-default-value-to
> > -va_list.patch
> > deleted file mode 100644
> > index 9cb6378..0000000
> > ---
> > a/meta-oe/recipes-extended/sysdig/sysdig/01-donot-set-default-value-to
> > -va_list.patch
> > +++ /dev/null
> 
> why is this patch dropped ?
> 

Thanks for your review.

Because this bug had been fixed at upstream before 0.1.102 released.
Obtain detail from here 
https://github.com/draios/sysdig/commit/3cf4db08314b8559405198fb998eab421fec4b7d

Thanks
 Bian

> > @@ -1,22 +0,0 @@
> > -fix build error as following.
> > -
> > -  sysdig-0.1.101/userspace/libsinsp/ctext.h:376:50: error: could not
> convert '0' from 'int' to 'va_list {aka __va_list}'
> > -  |    int8_t vprintf(const char*format, va_list ap = 0);
> > -  |                                                   ^
> > -  | make[2]: ***
> > [userspace/libsinsp/CMakeFiles/sinsp.dir/ctext.cpp.o] Error 1
> > -
> > -
> > -Signed-off-by: Bian Naimeng <biannm@cn.fujitsu.com>
> > -
> > -diff -Nurp sysdig-0.1.101.orig/userspace/libsinsp/ctext.h
> sysdig-0.1.101/userspace/libsinsp/ctext.h
> > ---- sysdig-0.1.101.orig/userspace/libsinsp/ctext.h     2015-06-04
> 02:58:51.000000000 +0800
> > -+++ sysdig-0.1.101/userspace/libsinsp/ctext.h  2015-07-02
> > -+++ 09:18:14.154627357 +0800
> > -@@ -373,7 +373,7 @@ class ctext
> > -               // application to this library seamlessly.
> > -               //
> > -               int8_t printf(const char*format, ...);
> > --              int8_t vprintf(const char*format, va_list ap = 0);
> > -+              int8_t vprintf(const char*format, va_list ap);
> > -
> > -               //
> > -               // nprintf is identical to the printf above EXCEPT for
> > diff --git a/meta-oe/recipes-extended/sysdig/sysdig_0.1.101.bb
> > b/meta-oe/recipes-extended/sysdig/sysdig_0.1.101.bb
> > deleted file mode 100644
> > index 789fd47..0000000
> > --- a/meta-oe/recipes-extended/sysdig/sysdig_0.1.101.bb
> > +++ /dev/null
> > @@ -1,34 +0,0 @@
> > -SUMMARY = "A New System Troubleshooting Tool Built for the Way You Work"
> > -DESCRIPTION = "Sysdig is open source, system-level exploration:
> > capture \ -system state and activity from a running Linux instance,
> > then save, \ -filter and analyze."
> > -HOMEPAGE = "http://www.sysdig.org/"
> > -LICENSE = "GPLv2"
> > -LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263"
> > -
> > -inherit cmake pkgconfig
> > -
> > -DEPENDS = "luajit zlib ncurses"
> > -RDEPENDS_${PN} = "bash"
> > -
> > -
> > -SRC_URI = "https://github.com/draios/sysdig/archive/${PV}.tar.gz \
> > -           file://01-donot-set-default-value-to-va_list.patch \
> > -          "
> > -
> > -SRC_URI[md5sum] = "5fe96a3a0fd98b2157a40cb29af41afc"
> > -SRC_URI[sha256sum] =
> "6995e39be565514901b5cb587689ee2efbf8359293e4e597362382cccf0e9db6"
> > -
> > -DIR_ETC="/etc"
> > -EXTRA_OECMAKE = ' -DUSE_BUNDLED_LUAJIT="OFF" \
> > -                  -DUSE_BUNDLED_ZLIB="OFF" \
> > -                  -DBUILD_DRIVER="OFF" \
> > -                  -DUSE_BUNDLED_NCURSES="OFF" \
> > -                  -DDIR_ETC="${DIR_ETC}" \
> > -                '
> > -
> > -FILES_${PN} += " \
> > -    ${DIR_ETC}/* \
> > -    ${datadir}/zsh/* \
> > -    ${prefix}/src/*  \
> > -"
> > diff --git a/meta-oe/recipes-extended/sysdig/sysdig_git.bb
> > b/meta-oe/recipes-extended/sysdig/sysdig_git.bb
> > new file mode 100644
> > index 0000000..62aa433
> > --- /dev/null
> > +++ b/meta-oe/recipes-extended/sysdig/sysdig_git.bb
> > @@ -0,0 +1,32 @@
> > +SUMMARY = "A New System Troubleshooting Tool Built for the Way You Work"
> > +DESCRIPTION = "Sysdig is open source, system-level exploration:
> > +capture \ system state and activity from a running Linux instance,
> > +then save, \ filter and analyze."
> > +HOMEPAGE = "http://www.sysdig.org/"
> > +LICENSE = "GPLv2"
> > +LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263"
> > +
> > +inherit cmake pkgconfig
> > +
> > +DEPENDS = "luajit zlib ncurses"
> > +RDEPENDS_${PN} = "bash"
> > +
> > +SRC_URI = "git://github.com/draios/sysdig.git;branch=master"
> > +SRCREV = "b7394e29ced4f1a991af03c0381a5828abcbab7a"
> > +PV = "0.1.102+git${SRCPV}"
> > +
> > +S = "${WORKDIR}/git"
> > +
> > +DIR_ETC="/etc"
> > +EXTRA_OECMAKE = ' -DUSE_BUNDLED_LUAJIT="OFF" \
> > +                  -DUSE_BUNDLED_ZLIB="OFF" \
> > +                  -DBUILD_DRIVER="OFF" \
> > +                  -DUSE_BUNDLED_NCURSES="OFF" \
> > +                  -DDIR_ETC="${DIR_ETC}" \
> > +                '
> > +
> > +FILES_${PN} += " \
> > +    ${DIR_ETC}/* \
> > +    ${datadir}/zsh/* \
> > +    ${prefix}/src/*  \
> > +"
> > --
> > 1.8.4.2
> >
> > --
> > _______________________________________________
> > 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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [meta-oe][PATCH] sysdig: 0.1.101 -> 0.1.102
  2015-08-25  0:44   ` Bian, Naimeng
@ 2015-08-25  1:00     ` Khem Raj
  2015-08-25  1:12       ` Bian, Naimeng
  0 siblings, 1 reply; 6+ messages in thread
From: Khem Raj @ 2015-08-25  1:00 UTC (permalink / raw
  To: openembedded-devel

[-- Attachment #1: Type: text/plain, Size: 1945 bytes --]


> On Aug 24, 2015, at 5:44 PM, Bian, Naimeng <biannm@cn.fujitsu.com> wrote:
> 
> 
> 
>> -----Original Message-----
>> From: openembedded-devel-bounces@lists.openembedded.org
>> [mailto:openembedded-devel-bounces@lists.openembedded.org] On Behalf Of Khem
>> Raj
>> Sent: Friday, August 21, 2015 11:42 PM
>> To: openembeded-devel
>> Subject: Re: [oe] [meta-oe][PATCH] sysdig: 0.1.101 -> 0.1.102
>> 
>> On Fri, Aug 21, 2015 at 9:40 AM, Bian Naimeng <biannm@cn.fujitsu.com> wrote:
>>> upgrade sysdig to 0.1.102, and using GIT source instead of archive.
>>> 
>>> Signed-off-by: Bian Naimeng <biannm@cn.fujitsu.com>
>>> ---
>>> .../01-donot-set-default-value-to-va_list.patch    | 22 --------------
>>> meta-oe/recipes-extended/sysdig/sysdig_0.1.101.bb  | 34
>> ----------------------
>>> meta-oe/recipes-extended/sysdig/sysdig_git.bb      | 32
>> ++++++++++++++++++++
>>> 3 files changed, 32 insertions(+), 56 deletions(-)  delete mode
>>> 100644
>>> meta-oe/recipes-extended/sysdig/sysdig/01-donot-set-default-value-to-v
>>> a_list.patch  delete mode 100644
>>> meta-oe/recipes-extended/sysdig/sysdig_0.1.101.bb
>>> create mode 100644 meta-oe/recipes-extended/sysdig/sysdig_git.bb
>>> 
>>> diff --git
>>> a/meta-oe/recipes-extended/sysdig/sysdig/01-donot-set-default-value-to
>>> -va_list.patch
>>> b/meta-oe/recipes-extended/sysdig/sysdig/01-donot-set-default-value-to
>>> -va_list.patch
>>> deleted file mode 100644
>>> index 9cb6378..0000000
>>> ---
>>> a/meta-oe/recipes-extended/sysdig/sysdig/01-donot-set-default-value-to
>>> -va_list.patch
>>> +++ /dev/null
>> 
>> why is this patch dropped ?
>> 
> 
> Thanks for your review.
> 
> Because this bug had been fixed at upstream before 0.1.102 released.
> Obtain detail from here
> https://github.com/draios/sysdig/commit/3cf4db08314b8559405198fb998eab421fec4b7d
> 

OK it would be nice to have this kind of information in commit message



[-- Attachment #2: Message signed with OpenPGP using GPGMail --]
[-- Type: application/pgp-signature, Size: 211 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [meta-oe][PATCH] sysdig: 0.1.101 -> 0.1.102
  2015-08-25  1:00     ` Khem Raj
@ 2015-08-25  1:12       ` Bian, Naimeng
  2015-08-25  1:20         ` Khem Raj
  0 siblings, 1 reply; 6+ messages in thread
From: Bian, Naimeng @ 2015-08-25  1:12 UTC (permalink / raw
  To: openembedded-devel@lists.openembedded.org



> -----Original Message-----
> From: openembedded-devel-bounces@lists.openembedded.org
> [mailto:openembedded-devel-bounces@lists.openembedded.org] On Behalf Of Khem
> Raj
> Sent: Tuesday, August 25, 2015 9:00 AM
> To: openembedded-devel@lists.openembedded.org
> Subject: Re: [oe] [meta-oe][PATCH] sysdig: 0.1.101 -> 0.1.102
> 
> 
> > On Aug 24, 2015, at 5:44 PM, Bian, Naimeng <biannm@cn.fujitsu.com> wrote:
> >
> >
> >
> >> -----Original Message-----
> >> From: openembedded-devel-bounces@lists.openembedded.org
> >> [mailto:openembedded-devel-bounces@lists.openembedded.org] On Behalf
> >> Of Khem Raj
> >> Sent: Friday, August 21, 2015 11:42 PM
> >> To: openembeded-devel
> >> Subject: Re: [oe] [meta-oe][PATCH] sysdig: 0.1.101 -> 0.1.102
> >>
> >> On Fri, Aug 21, 2015 at 9:40 AM, Bian Naimeng <biannm@cn.fujitsu.com> wrote:
> >>> upgrade sysdig to 0.1.102, and using GIT source instead of archive.
> >>>
> >>> Signed-off-by: Bian Naimeng <biannm@cn.fujitsu.com>
> >>> ---
> >>> .../01-donot-set-default-value-to-va_list.patch    | 22 --------------
> >>> meta-oe/recipes-extended/sysdig/sysdig_0.1.101.bb  | 34
> >> ----------------------
> >>> meta-oe/recipes-extended/sysdig/sysdig_git.bb      | 32
> >> ++++++++++++++++++++
> >>> 3 files changed, 32 insertions(+), 56 deletions(-)  delete mode
> >>> 100644
> >>> meta-oe/recipes-extended/sysdig/sysdig/01-donot-set-default-value-to
> >>> -v
> >>> a_list.patch  delete mode 100644
> >>> meta-oe/recipes-extended/sysdig/sysdig_0.1.101.bb
> >>> create mode 100644 meta-oe/recipes-extended/sysdig/sysdig_git.bb
> >>>
> >>> diff --git
> >>> a/meta-oe/recipes-extended/sysdig/sysdig/01-donot-set-default-value-
> >>> to
> >>> -va_list.patch
> >>> b/meta-oe/recipes-extended/sysdig/sysdig/01-donot-set-default-value-
> >>> to
> >>> -va_list.patch
> >>> deleted file mode 100644
> >>> index 9cb6378..0000000
> >>> ---
> >>> a/meta-oe/recipes-extended/sysdig/sysdig/01-donot-set-default-value-
> >>> to
> >>> -va_list.patch
> >>> +++ /dev/null
> >>
> >> why is this patch dropped ?
> >>
> >
> > Thanks for your review.
> >
> > Because this bug had been fixed at upstream before 0.1.102 released.
> > Obtain detail from here
> > https://github.com/draios/sysdig/commit/3cf4db08314b8559405198fb998eab
> > 421fec4b7d
> >
> 
> OK it would be nice to have this kind of information in commit message
> 

I got it, will be noted at next time.

Thanks
 Bian

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [meta-oe][PATCH] sysdig: 0.1.101 -> 0.1.102
  2015-08-25  1:12       ` Bian, Naimeng
@ 2015-08-25  1:20         ` Khem Raj
  0 siblings, 0 replies; 6+ messages in thread
From: Khem Raj @ 2015-08-25  1:20 UTC (permalink / raw
  To: openembedded-devel

[-- Attachment #1: Type: text/plain, Size: 224 bytes --]


> On Aug 24, 2015, at 6:12 PM, Bian, Naimeng <biannm@cn.fujitsu.com> wrote:
> 
> I got it, will be noted at next time.

You can git commit —amend this message and send a V2 as well if this has not yet been applied.

[-- Attachment #2: Message signed with OpenPGP using GPGMail --]
[-- Type: application/pgp-signature, Size: 211 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2015-08-25  1:20 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-08-21 16:40 [meta-oe][PATCH] sysdig: 0.1.101 -> 0.1.102 Bian Naimeng
2015-08-21 15:42 ` Khem Raj
2015-08-25  0:44   ` Bian, Naimeng
2015-08-25  1:00     ` Khem Raj
2015-08-25  1:12       ` Bian, Naimeng
2015-08-25  1:20         ` Khem Raj

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.