* [meta-networking][PATCH] v3] Wireshark: build fix and recipe update
@ 2014-07-21 22:20 Armin Kuster
2014-07-23 13:56 ` Martin Jansa
0 siblings, 1 reply; 5+ messages in thread
From: Armin Kuster @ 2014-07-21 22:20 UTC (permalink / raw)
To: openembedded-devel
Backported Arm build fix
Added gtk3 support
Cleaned up PACKAGECONFIG options
Signed-off-by: Armin Kuster <akuster@mvista.com>
---
.../files/va_list-can-t-be-NULL-on-ARM.patch | 98 ++++++++++++++++++++++
.../wireshark/wireshark_1.12.0-rc2.bb | 5 +-
2 files changed, 102 insertions(+), 1 deletion(-)
create mode 100644 meta-networking/recipes-support/wireshark/files/va_list-can-t-be-NULL-on-ARM.patch
diff --git a/meta-networking/recipes-support/wireshark/files/va_list-can-t-be-NULL-on-ARM.patch b/meta-networking/recipes-support/wireshark/files/va_list-can-t-be-NULL-on-ARM.patch
new file mode 100644
index 0000000..301c381
--- /dev/null
+++ b/meta-networking/recipes-support/wireshark/files/va_list-can-t-be-NULL-on-ARM.patch
@@ -0,0 +1,98 @@
+From 320c4f0d705a3ed94f710fb4b7b3eef897ff7bc4 Mon Sep 17 00:00:00 2001
+From: Steev Klimaszewski <threeway@gmail.com>
+Date: Thu, 19 Jun 2014 16:54:57 -0500
+Subject: [PATCH] va_list can't be NULL on ARM.
+
+Bug: 10209
+Change-Id: Ibd63a530450b7d2d4ec244e91c77caa731ba63aa
+Signed-off-by: Steev Klimaszewski <threeway@gmail.com>
+Signed-off-by: Balint Reczey <balint@balintreczey.hu>
+Reviewed-on: https://code.wireshark.org/review/2464
+Reviewed-by: Evan Huus <eapache@gmail.com>
+Reviewed-by: Michael Mann <mmann78@netscape.net>
+
+The patch was imported from the wireshark git server
+ (https://code.wireshark.org/review/p/wireshark.git) as of commit id
+ 320c4f0d705a3ed94f710fb4b7b3eef897ff7bc4.
+
+Signed-off-by: Armin Kuster <akuster@mvista.com>
+
+---
+ epan/expert.c | 36 +++++++++++++++++++++++++++++-------
+ 1 file changed, 29 insertions(+), 7 deletions(-)
+
+diff --git a/epan/expert.c b/epan/expert.c
+index 46be838..a69566d 100644
+--- a/epan/expert.c
++++ b/epan/expert.c
+@@ -381,15 +381,26 @@ expert_set_info_vformat(packet_info *pinfo, proto_item *pi, int group, int sever
+ tap_queue_packet(expert_tap, pinfo, ei);
+ }
+
+-void
+-expert_add_info(packet_info *pinfo, proto_item *pi, expert_field *expindex)
++/* Helper function for expert_add_info() to work around compiler's special needs on ARM*/
++static inline void
++expert_add_info_internal(packet_info *pinfo, proto_item *pi, expert_field *expindex, ...)
+ {
++ /* the va_list is ignored */
++ va_list unused;
+ expert_field_info* eiinfo;
+
+ /* Look up the item */
+ EXPERT_REGISTRAR_GET_NTH(expindex->ei, eiinfo);
+
+- expert_set_info_vformat(pinfo, pi, eiinfo->group, eiinfo->severity, *eiinfo->hf_info.p_id, FALSE, eiinfo->summary, NULL);
++ va_start(unused, expindex);
++ expert_set_info_vformat(pinfo, pi, eiinfo->group, eiinfo->severity, *eiinfo->hf_info.p_id, FALSE, eiinfo->summary, unused);
++ va_end(unused);
++}
++
++void
++expert_add_info(packet_info *pinfo, proto_item *pi, expert_field *expindex)
++{
++ expert_add_info_internal(pinfo, pi, expindex);
+ }
+
+ void
+@@ -406,22 +417,33 @@ expert_add_info_format(packet_info *pinfo, proto_item *pi, expert_field *expinde
+ va_end(ap);
+ }
+
+-proto_item *
+-proto_tree_add_expert(proto_tree *tree, packet_info *pinfo, expert_field* expindex,
+- tvbuff_t *tvb, gint start, gint length)
++/* Helper function for expert_add_expert() to work around compiler's special needs on ARM*/
++static inline proto_item *
++proto_tree_add_expert_internal(proto_tree *tree, packet_info *pinfo, expert_field* expindex,
++ tvbuff_t *tvb, gint start, gint length, ...)
+ {
+ expert_field_info* eiinfo;
+ proto_item *ti;
++ va_list unused;
+
+ /* Look up the item */
+ EXPERT_REGISTRAR_GET_NTH(expindex->ei, eiinfo);
+
+ ti = proto_tree_add_text(tree, tvb, start, length, "%s", eiinfo->summary);
+- expert_set_info_vformat(pinfo, ti, eiinfo->group, eiinfo->severity, *eiinfo->hf_info.p_id, FALSE, eiinfo->summary, NULL);
++ va_start(unused, length);
++ expert_set_info_vformat(pinfo, ti, eiinfo->group, eiinfo->severity, *eiinfo->hf_info.p_id, FALSE, eiinfo->summary, unused);
++ va_end(unused);
+ return ti;
+ }
+
+ proto_item *
++proto_tree_add_expert(proto_tree *tree, packet_info *pinfo, expert_field* expindex,
++ tvbuff_t *tvb, gint start, gint length)
++{
++ return proto_tree_add_expert_internal(tree, pinfo, expindex, tvb, start, length);
++}
++
++proto_item *
+ proto_tree_add_expert_format(proto_tree *tree, packet_info *pinfo, expert_field* expindex,
+ tvbuff_t *tvb, gint start, gint length, const char *format, ...)
+ {
+--
+1.9.1
+
diff --git a/meta-networking/recipes-support/wireshark/wireshark_1.12.0-rc2.bb b/meta-networking/recipes-support/wireshark/wireshark_1.12.0-rc2.bb
index 66716c3..47e72f4 100644
--- a/meta-networking/recipes-support/wireshark/wireshark_1.12.0-rc2.bb
+++ b/meta-networking/recipes-support/wireshark/wireshark_1.12.0-rc2.bb
@@ -14,6 +14,7 @@ PACKAGECONFIG += " ${@bb.utils.contains("IMAGE_FEATURES", "x11-base", "gtk2 grap
PACKAGECONFIG += " ${@bb.utils.contains("DISTRO_FEATURES", "ipv6", "ipv6", "", d)}"
PACKAGECONFIG[gtk2] = "--with-gtk2=yes,--with-gtk2=no, gtk+"
+PACKAGECONFIG[gtk3] = "--with-gtk3=yes, --with-gtk3=no, gtk+3"
PACKAGECONFIG[graphics] = "--enable-wireshark, --disable-wireshark,"
PACKAGECONFIG[ipv6] = "--enable-ipv6,--disable-ipv6,"
PACKAGECONFIG[gnutls] = "--with-gnutls=yes,--with-gnutls=no, gnutls"
@@ -22,7 +23,9 @@ PACKAGECONFIG[gcrypt] = "--with-gcrypt=yes,--with-gcrypt=no, libgcrypt"
EXTRA_OECONF = "--with-gtk3=no --with-qt=no --enable-usr-local=no -enable-tshark"
LIC_FILES_CHKSUM = "file://README.linux;md5=631e077455b7972172eb149195e065b0"
-SRC_URI = "http://wiresharkdownloads.riverbed.com/wireshark/src/wireshark-1.12.0-rc2.tar.bz2 "
+SRC_URI = "http://wiresharkdownloads.riverbed.com/wireshark/src/wireshark-1.12.0-rc2.tar.bz2 \
+ file://va_list-can-t-be-NULL-on-ARM.patch \
+ "
SRC_URI[md5sum] = "dc1149073066a29f91116c168558262e"
SRC_URI[sha256sum]= "31009bb450126e9b12808267419f31016d14e6fde7b5e39c85ad37459908cffb"
--
1.9.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [meta-networking][PATCH] v3] Wireshark: build fix and recipe update
2014-07-21 22:20 [meta-networking][PATCH] v3] Wireshark: build fix and recipe update Armin Kuster
@ 2014-07-23 13:56 ` Martin Jansa
2014-07-23 17:49 ` akuster
2014-07-24 10:38 ` Martin Jansa
0 siblings, 2 replies; 5+ messages in thread
From: Martin Jansa @ 2014-07-23 13:56 UTC (permalink / raw)
To: openembedded-devel
[-- Attachment #1: Type: text/plain, Size: 6911 bytes --]
On Mon, Jul 21, 2014 at 03:20:28PM -0700, Armin Kuster wrote:
> Backported Arm build fix
> Added gtk3 support
> Cleaned up PACKAGECONFIG options
You cannot use IMAGE_FEATURES to define PACKAGECONFIG, see what I did
with it in master-next (and confirm that you're ok with my changes).
Also please notice wrong number of ']' in subject, no big deal, but I
had to strip 'v3]" from subject lines manually when cherry-picking from
patchwork.
> Signed-off-by: Armin Kuster <akuster@mvista.com>
> ---
> .../files/va_list-can-t-be-NULL-on-ARM.patch | 98 ++++++++++++++++++++++
> .../wireshark/wireshark_1.12.0-rc2.bb | 5 +-
> 2 files changed, 102 insertions(+), 1 deletion(-)
> create mode 100644 meta-networking/recipes-support/wireshark/files/va_list-can-t-be-NULL-on-ARM.patch
>
> diff --git a/meta-networking/recipes-support/wireshark/files/va_list-can-t-be-NULL-on-ARM.patch b/meta-networking/recipes-support/wireshark/files/va_list-can-t-be-NULL-on-ARM.patch
> new file mode 100644
> index 0000000..301c381
> --- /dev/null
> +++ b/meta-networking/recipes-support/wireshark/files/va_list-can-t-be-NULL-on-ARM.patch
> @@ -0,0 +1,98 @@
> +From 320c4f0d705a3ed94f710fb4b7b3eef897ff7bc4 Mon Sep 17 00:00:00 2001
> +From: Steev Klimaszewski <threeway@gmail.com>
> +Date: Thu, 19 Jun 2014 16:54:57 -0500
> +Subject: [PATCH] va_list can't be NULL on ARM.
> +
> +Bug: 10209
> +Change-Id: Ibd63a530450b7d2d4ec244e91c77caa731ba63aa
> +Signed-off-by: Steev Klimaszewski <threeway@gmail.com>
> +Signed-off-by: Balint Reczey <balint@balintreczey.hu>
> +Reviewed-on: https://code.wireshark.org/review/2464
> +Reviewed-by: Evan Huus <eapache@gmail.com>
> +Reviewed-by: Michael Mann <mmann78@netscape.net>
> +
> +The patch was imported from the wireshark git server
> + (https://code.wireshark.org/review/p/wireshark.git) as of commit id
> + 320c4f0d705a3ed94f710fb4b7b3eef897ff7bc4.
> +
> +Signed-off-by: Armin Kuster <akuster@mvista.com>
> +
> +---
> + epan/expert.c | 36 +++++++++++++++++++++++++++++-------
> + 1 file changed, 29 insertions(+), 7 deletions(-)
> +
> +diff --git a/epan/expert.c b/epan/expert.c
> +index 46be838..a69566d 100644
> +--- a/epan/expert.c
> ++++ b/epan/expert.c
> +@@ -381,15 +381,26 @@ expert_set_info_vformat(packet_info *pinfo, proto_item *pi, int group, int sever
> + tap_queue_packet(expert_tap, pinfo, ei);
> + }
> +
> +-void
> +-expert_add_info(packet_info *pinfo, proto_item *pi, expert_field *expindex)
> ++/* Helper function for expert_add_info() to work around compiler's special needs on ARM*/
> ++static inline void
> ++expert_add_info_internal(packet_info *pinfo, proto_item *pi, expert_field *expindex, ...)
> + {
> ++ /* the va_list is ignored */
> ++ va_list unused;
> + expert_field_info* eiinfo;
> +
> + /* Look up the item */
> + EXPERT_REGISTRAR_GET_NTH(expindex->ei, eiinfo);
> +
> +- expert_set_info_vformat(pinfo, pi, eiinfo->group, eiinfo->severity, *eiinfo->hf_info.p_id, FALSE, eiinfo->summary, NULL);
> ++ va_start(unused, expindex);
> ++ expert_set_info_vformat(pinfo, pi, eiinfo->group, eiinfo->severity, *eiinfo->hf_info.p_id, FALSE, eiinfo->summary, unused);
> ++ va_end(unused);
> ++}
> ++
> ++void
> ++expert_add_info(packet_info *pinfo, proto_item *pi, expert_field *expindex)
> ++{
> ++ expert_add_info_internal(pinfo, pi, expindex);
> + }
> +
> + void
> +@@ -406,22 +417,33 @@ expert_add_info_format(packet_info *pinfo, proto_item *pi, expert_field *expinde
> + va_end(ap);
> + }
> +
> +-proto_item *
> +-proto_tree_add_expert(proto_tree *tree, packet_info *pinfo, expert_field* expindex,
> +- tvbuff_t *tvb, gint start, gint length)
> ++/* Helper function for expert_add_expert() to work around compiler's special needs on ARM*/
> ++static inline proto_item *
> ++proto_tree_add_expert_internal(proto_tree *tree, packet_info *pinfo, expert_field* expindex,
> ++ tvbuff_t *tvb, gint start, gint length, ...)
> + {
> + expert_field_info* eiinfo;
> + proto_item *ti;
> ++ va_list unused;
> +
> + /* Look up the item */
> + EXPERT_REGISTRAR_GET_NTH(expindex->ei, eiinfo);
> +
> + ti = proto_tree_add_text(tree, tvb, start, length, "%s", eiinfo->summary);
> +- expert_set_info_vformat(pinfo, ti, eiinfo->group, eiinfo->severity, *eiinfo->hf_info.p_id, FALSE, eiinfo->summary, NULL);
> ++ va_start(unused, length);
> ++ expert_set_info_vformat(pinfo, ti, eiinfo->group, eiinfo->severity, *eiinfo->hf_info.p_id, FALSE, eiinfo->summary, unused);
> ++ va_end(unused);
> + return ti;
> + }
> +
> + proto_item *
> ++proto_tree_add_expert(proto_tree *tree, packet_info *pinfo, expert_field* expindex,
> ++ tvbuff_t *tvb, gint start, gint length)
> ++{
> ++ return proto_tree_add_expert_internal(tree, pinfo, expindex, tvb, start, length);
> ++}
> ++
> ++proto_item *
> + proto_tree_add_expert_format(proto_tree *tree, packet_info *pinfo, expert_field* expindex,
> + tvbuff_t *tvb, gint start, gint length, const char *format, ...)
> + {
> +--
> +1.9.1
> +
> diff --git a/meta-networking/recipes-support/wireshark/wireshark_1.12.0-rc2.bb b/meta-networking/recipes-support/wireshark/wireshark_1.12.0-rc2.bb
> index 66716c3..47e72f4 100644
> --- a/meta-networking/recipes-support/wireshark/wireshark_1.12.0-rc2.bb
> +++ b/meta-networking/recipes-support/wireshark/wireshark_1.12.0-rc2.bb
> @@ -14,6 +14,7 @@ PACKAGECONFIG += " ${@bb.utils.contains("IMAGE_FEATURES", "x11-base", "gtk2 grap
> PACKAGECONFIG += " ${@bb.utils.contains("DISTRO_FEATURES", "ipv6", "ipv6", "", d)}"
>
> PACKAGECONFIG[gtk2] = "--with-gtk2=yes,--with-gtk2=no, gtk+"
> +PACKAGECONFIG[gtk3] = "--with-gtk3=yes, --with-gtk3=no, gtk+3"
> PACKAGECONFIG[graphics] = "--enable-wireshark, --disable-wireshark,"
> PACKAGECONFIG[ipv6] = "--enable-ipv6,--disable-ipv6,"
> PACKAGECONFIG[gnutls] = "--with-gnutls=yes,--with-gnutls=no, gnutls"
> @@ -22,7 +23,9 @@ PACKAGECONFIG[gcrypt] = "--with-gcrypt=yes,--with-gcrypt=no, libgcrypt"
> EXTRA_OECONF = "--with-gtk3=no --with-qt=no --enable-usr-local=no -enable-tshark"
>
> LIC_FILES_CHKSUM = "file://README.linux;md5=631e077455b7972172eb149195e065b0"
> -SRC_URI = "http://wiresharkdownloads.riverbed.com/wireshark/src/wireshark-1.12.0-rc2.tar.bz2 "
> +SRC_URI = "http://wiresharkdownloads.riverbed.com/wireshark/src/wireshark-1.12.0-rc2.tar.bz2 \
> + file://va_list-can-t-be-NULL-on-ARM.patch \
> + "
>
> SRC_URI[md5sum] = "dc1149073066a29f91116c168558262e"
> SRC_URI[sha256sum]= "31009bb450126e9b12808267419f31016d14e6fde7b5e39c85ad37459908cffb"
> --
> 1.9.1
>
> --
> _______________________________________________
> 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] 5+ messages in thread
* Re: [meta-networking][PATCH] v3] Wireshark: build fix and recipe update
2014-07-23 13:56 ` Martin Jansa
@ 2014-07-23 17:49 ` akuster
2014-07-24 10:38 ` Martin Jansa
1 sibling, 0 replies; 5+ messages in thread
From: akuster @ 2014-07-23 17:49 UTC (permalink / raw)
To: Martin Jansa; +Cc: openembedded-devel
On 07/23/2014 06:56 AM, Martin Jansa wrote:
> On Mon, Jul 21, 2014 at 03:20:28PM -0700, Armin Kuster wrote:
>> Backported Arm build fix
>> Added gtk3 support
>> Cleaned up PACKAGECONFIG options
> You cannot use IMAGE_FEATURES to define PACKAGECONFIG, see what I did
> with it in master-next (and confirm that you're ok with my changes).
ok. thanks.
Changes look fine.
Signed-off-by: Armin Kuster <akuster@mvista.com>
>
> Also please notice wrong number of ']' in subject, no big deal, but I
> had to strip 'v3]" from subject lines manually when cherry-picking from
> patchwork.
thanks
regards,
Armin
>> Signed-off-by: Armin Kuster <akuster@mvista.com>
>> ---
>> .../files/va_list-can-t-be-NULL-on-ARM.patch | 98 ++++++++++++++++++++++
>> .../wireshark/wireshark_1.12.0-rc2.bb | 5 +-
>> 2 files changed, 102 insertions(+), 1 deletion(-)
>> create mode 100644 meta-networking/recipes-support/wireshark/files/va_list-can-t-be-NULL-on-ARM.patch
>>
>> diff --git a/meta-networking/recipes-support/wireshark/files/va_list-can-t-be-NULL-on-ARM.patch b/meta-networking/recipes-support/wireshark/files/va_list-can-t-be-NULL-on-ARM.patch
>> new file mode 100644
>> index 0000000..301c381
>> --- /dev/null
>> +++ b/meta-networking/recipes-support/wireshark/files/va_list-can-t-be-NULL-on-ARM.patch
>> @@ -0,0 +1,98 @@
>> +From 320c4f0d705a3ed94f710fb4b7b3eef897ff7bc4 Mon Sep 17 00:00:00 2001
>> +From: Steev Klimaszewski <threeway@gmail.com>
>> +Date: Thu, 19 Jun 2014 16:54:57 -0500
>> +Subject: [PATCH] va_list can't be NULL on ARM.
>> +
>> +Bug: 10209
>> +Change-Id: Ibd63a530450b7d2d4ec244e91c77caa731ba63aa
>> +Signed-off-by: Steev Klimaszewski <threeway@gmail.com>
>> +Signed-off-by: Balint Reczey <balint@balintreczey.hu>
>> +Reviewed-on: https://code.wireshark.org/review/2464
>> +Reviewed-by: Evan Huus <eapache@gmail.com>
>> +Reviewed-by: Michael Mann <mmann78@netscape.net>
>> +
>> +The patch was imported from the wireshark git server
>> + (https://code.wireshark.org/review/p/wireshark.git) as of commit id
>> + 320c4f0d705a3ed94f710fb4b7b3eef897ff7bc4.
>> +
>> +Signed-off-by: Armin Kuster <akuster@mvista.com>
>> +
>> +---
>> + epan/expert.c | 36 +++++++++++++++++++++++++++++-------
>> + 1 file changed, 29 insertions(+), 7 deletions(-)
>> +
>> +diff --git a/epan/expert.c b/epan/expert.c
>> +index 46be838..a69566d 100644
>> +--- a/epan/expert.c
>> ++++ b/epan/expert.c
>> +@@ -381,15 +381,26 @@ expert_set_info_vformat(packet_info *pinfo, proto_item *pi, int group, int sever
>> + tap_queue_packet(expert_tap, pinfo, ei);
>> + }
>> +
>> +-void
>> +-expert_add_info(packet_info *pinfo, proto_item *pi, expert_field *expindex)
>> ++/* Helper function for expert_add_info() to work around compiler's special needs on ARM*/
>> ++static inline void
>> ++expert_add_info_internal(packet_info *pinfo, proto_item *pi, expert_field *expindex, ...)
>> + {
>> ++ /* the va_list is ignored */
>> ++ va_list unused;
>> + expert_field_info* eiinfo;
>> +
>> + /* Look up the item */
>> + EXPERT_REGISTRAR_GET_NTH(expindex->ei, eiinfo);
>> +
>> +- expert_set_info_vformat(pinfo, pi, eiinfo->group, eiinfo->severity, *eiinfo->hf_info.p_id, FALSE, eiinfo->summary, NULL);
>> ++ va_start(unused, expindex);
>> ++ expert_set_info_vformat(pinfo, pi, eiinfo->group, eiinfo->severity, *eiinfo->hf_info.p_id, FALSE, eiinfo->summary, unused);
>> ++ va_end(unused);
>> ++}
>> ++
>> ++void
>> ++expert_add_info(packet_info *pinfo, proto_item *pi, expert_field *expindex)
>> ++{
>> ++ expert_add_info_internal(pinfo, pi, expindex);
>> + }
>> +
>> + void
>> +@@ -406,22 +417,33 @@ expert_add_info_format(packet_info *pinfo, proto_item *pi, expert_field *expinde
>> + va_end(ap);
>> + }
>> +
>> +-proto_item *
>> +-proto_tree_add_expert(proto_tree *tree, packet_info *pinfo, expert_field* expindex,
>> +- tvbuff_t *tvb, gint start, gint length)
>> ++/* Helper function for expert_add_expert() to work around compiler's special needs on ARM*/
>> ++static inline proto_item *
>> ++proto_tree_add_expert_internal(proto_tree *tree, packet_info *pinfo, expert_field* expindex,
>> ++ tvbuff_t *tvb, gint start, gint length, ...)
>> + {
>> + expert_field_info* eiinfo;
>> + proto_item *ti;
>> ++ va_list unused;
>> +
>> + /* Look up the item */
>> + EXPERT_REGISTRAR_GET_NTH(expindex->ei, eiinfo);
>> +
>> + ti = proto_tree_add_text(tree, tvb, start, length, "%s", eiinfo->summary);
>> +- expert_set_info_vformat(pinfo, ti, eiinfo->group, eiinfo->severity, *eiinfo->hf_info.p_id, FALSE, eiinfo->summary, NULL);
>> ++ va_start(unused, length);
>> ++ expert_set_info_vformat(pinfo, ti, eiinfo->group, eiinfo->severity, *eiinfo->hf_info.p_id, FALSE, eiinfo->summary, unused);
>> ++ va_end(unused);
>> + return ti;
>> + }
>> +
>> + proto_item *
>> ++proto_tree_add_expert(proto_tree *tree, packet_info *pinfo, expert_field* expindex,
>> ++ tvbuff_t *tvb, gint start, gint length)
>> ++{
>> ++ return proto_tree_add_expert_internal(tree, pinfo, expindex, tvb, start, length);
>> ++}
>> ++
>> ++proto_item *
>> + proto_tree_add_expert_format(proto_tree *tree, packet_info *pinfo, expert_field* expindex,
>> + tvbuff_t *tvb, gint start, gint length, const char *format, ...)
>> + {
>> +--
>> +1.9.1
>> +
>> diff --git a/meta-networking/recipes-support/wireshark/wireshark_1.12.0-rc2.bb b/meta-networking/recipes-support/wireshark/wireshark_1.12.0-rc2.bb
>> index 66716c3..47e72f4 100644
>> --- a/meta-networking/recipes-support/wireshark/wireshark_1.12.0-rc2.bb
>> +++ b/meta-networking/recipes-support/wireshark/wireshark_1.12.0-rc2.bb
>> @@ -14,6 +14,7 @@ PACKAGECONFIG += " ${@bb.utils.contains("IMAGE_FEATURES", "x11-base", "gtk2 grap
>> PACKAGECONFIG += " ${@bb.utils.contains("DISTRO_FEATURES", "ipv6", "ipv6", "", d)}"
>>
>> PACKAGECONFIG[gtk2] = "--with-gtk2=yes,--with-gtk2=no, gtk+"
>> +PACKAGECONFIG[gtk3] = "--with-gtk3=yes, --with-gtk3=no, gtk+3"
>> PACKAGECONFIG[graphics] = "--enable-wireshark, --disable-wireshark,"
>> PACKAGECONFIG[ipv6] = "--enable-ipv6,--disable-ipv6,"
>> PACKAGECONFIG[gnutls] = "--with-gnutls=yes,--with-gnutls=no, gnutls"
>> @@ -22,7 +23,9 @@ PACKAGECONFIG[gcrypt] = "--with-gcrypt=yes,--with-gcrypt=no, libgcrypt"
>> EXTRA_OECONF = "--with-gtk3=no --with-qt=no --enable-usr-local=no -enable-tshark"
>>
>> LIC_FILES_CHKSUM = "file://README.linux;md5=631e077455b7972172eb149195e065b0"
>> -SRC_URI = "http://wiresharkdownloads.riverbed.com/wireshark/src/wireshark-1.12.0-rc2.tar.bz2 "
>> +SRC_URI = "http://wiresharkdownloads.riverbed.com/wireshark/src/wireshark-1.12.0-rc2.tar.bz2 \
>> + file://va_list-can-t-be-NULL-on-ARM.patch \
>> + "
>>
>> SRC_URI[md5sum] = "dc1149073066a29f91116c168558262e"
>> SRC_URI[sha256sum]= "31009bb450126e9b12808267419f31016d14e6fde7b5e39c85ad37459908cffb"
>> --
>> 1.9.1
>>
>> --
>> _______________________________________________
>> Openembedded-devel mailing list
>> Openembedded-devel@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
>
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [meta-networking][PATCH] v3] Wireshark: build fix and recipe update
2014-07-23 13:56 ` Martin Jansa
2014-07-23 17:49 ` akuster
@ 2014-07-24 10:38 ` Martin Jansa
2014-07-24 20:21 ` akuster
1 sibling, 1 reply; 5+ messages in thread
From: Martin Jansa @ 2014-07-24 10:38 UTC (permalink / raw)
To: openembedded-devel
[-- Attachment #1: Type: text/plain, Size: 7584 bytes --]
On Wed, Jul 23, 2014 at 03:56:16PM +0200, Martin Jansa wrote:
> On Mon, Jul 21, 2014 at 03:20:28PM -0700, Armin Kuster wrote:
> > Backported Arm build fix
> > Added gtk3 support
> > Cleaned up PACKAGECONFIG options
>
> You cannot use IMAGE_FEATURES to define PACKAGECONFIG, see what I did
> with it in master-next (and confirm that you're ok with my changes).
>
> Also please notice wrong number of ']' in subject, no big deal, but I
> had to strip 'v3]" from subject lines manually when cherry-picking from
> patchwork.
Can you please fix these floating dependencies before we merge this?
WARN: packages/armv5te-oe-linux-gnueabi/wireshark/wireshark/latest lost
dependency on libcap libnl libnl-genl libnl-nf libnl-route
portaudio-v19 sbc
>
> > Signed-off-by: Armin Kuster <akuster@mvista.com>
> > ---
> > .../files/va_list-can-t-be-NULL-on-ARM.patch | 98 ++++++++++++++++++++++
> > .../wireshark/wireshark_1.12.0-rc2.bb | 5 +-
> > 2 files changed, 102 insertions(+), 1 deletion(-)
> > create mode 100644 meta-networking/recipes-support/wireshark/files/va_list-can-t-be-NULL-on-ARM.patch
> >
> > diff --git a/meta-networking/recipes-support/wireshark/files/va_list-can-t-be-NULL-on-ARM.patch b/meta-networking/recipes-support/wireshark/files/va_list-can-t-be-NULL-on-ARM.patch
> > new file mode 100644
> > index 0000000..301c381
> > --- /dev/null
> > +++ b/meta-networking/recipes-support/wireshark/files/va_list-can-t-be-NULL-on-ARM.patch
> > @@ -0,0 +1,98 @@
> > +From 320c4f0d705a3ed94f710fb4b7b3eef897ff7bc4 Mon Sep 17 00:00:00 2001
> > +From: Steev Klimaszewski <threeway@gmail.com>
> > +Date: Thu, 19 Jun 2014 16:54:57 -0500
> > +Subject: [PATCH] va_list can't be NULL on ARM.
> > +
> > +Bug: 10209
> > +Change-Id: Ibd63a530450b7d2d4ec244e91c77caa731ba63aa
> > +Signed-off-by: Steev Klimaszewski <threeway@gmail.com>
> > +Signed-off-by: Balint Reczey <balint@balintreczey.hu>
> > +Reviewed-on: https://code.wireshark.org/review/2464
> > +Reviewed-by: Evan Huus <eapache@gmail.com>
> > +Reviewed-by: Michael Mann <mmann78@netscape.net>
> > +
> > +The patch was imported from the wireshark git server
> > + (https://code.wireshark.org/review/p/wireshark.git) as of commit id
> > + 320c4f0d705a3ed94f710fb4b7b3eef897ff7bc4.
> > +
> > +Signed-off-by: Armin Kuster <akuster@mvista.com>
> > +
> > +---
> > + epan/expert.c | 36 +++++++++++++++++++++++++++++-------
> > + 1 file changed, 29 insertions(+), 7 deletions(-)
> > +
> > +diff --git a/epan/expert.c b/epan/expert.c
> > +index 46be838..a69566d 100644
> > +--- a/epan/expert.c
> > ++++ b/epan/expert.c
> > +@@ -381,15 +381,26 @@ expert_set_info_vformat(packet_info *pinfo, proto_item *pi, int group, int sever
> > + tap_queue_packet(expert_tap, pinfo, ei);
> > + }
> > +
> > +-void
> > +-expert_add_info(packet_info *pinfo, proto_item *pi, expert_field *expindex)
> > ++/* Helper function for expert_add_info() to work around compiler's special needs on ARM*/
> > ++static inline void
> > ++expert_add_info_internal(packet_info *pinfo, proto_item *pi, expert_field *expindex, ...)
> > + {
> > ++ /* the va_list is ignored */
> > ++ va_list unused;
> > + expert_field_info* eiinfo;
> > +
> > + /* Look up the item */
> > + EXPERT_REGISTRAR_GET_NTH(expindex->ei, eiinfo);
> > +
> > +- expert_set_info_vformat(pinfo, pi, eiinfo->group, eiinfo->severity, *eiinfo->hf_info.p_id, FALSE, eiinfo->summary, NULL);
> > ++ va_start(unused, expindex);
> > ++ expert_set_info_vformat(pinfo, pi, eiinfo->group, eiinfo->severity, *eiinfo->hf_info.p_id, FALSE, eiinfo->summary, unused);
> > ++ va_end(unused);
> > ++}
> > ++
> > ++void
> > ++expert_add_info(packet_info *pinfo, proto_item *pi, expert_field *expindex)
> > ++{
> > ++ expert_add_info_internal(pinfo, pi, expindex);
> > + }
> > +
> > + void
> > +@@ -406,22 +417,33 @@ expert_add_info_format(packet_info *pinfo, proto_item *pi, expert_field *expinde
> > + va_end(ap);
> > + }
> > +
> > +-proto_item *
> > +-proto_tree_add_expert(proto_tree *tree, packet_info *pinfo, expert_field* expindex,
> > +- tvbuff_t *tvb, gint start, gint length)
> > ++/* Helper function for expert_add_expert() to work around compiler's special needs on ARM*/
> > ++static inline proto_item *
> > ++proto_tree_add_expert_internal(proto_tree *tree, packet_info *pinfo, expert_field* expindex,
> > ++ tvbuff_t *tvb, gint start, gint length, ...)
> > + {
> > + expert_field_info* eiinfo;
> > + proto_item *ti;
> > ++ va_list unused;
> > +
> > + /* Look up the item */
> > + EXPERT_REGISTRAR_GET_NTH(expindex->ei, eiinfo);
> > +
> > + ti = proto_tree_add_text(tree, tvb, start, length, "%s", eiinfo->summary);
> > +- expert_set_info_vformat(pinfo, ti, eiinfo->group, eiinfo->severity, *eiinfo->hf_info.p_id, FALSE, eiinfo->summary, NULL);
> > ++ va_start(unused, length);
> > ++ expert_set_info_vformat(pinfo, ti, eiinfo->group, eiinfo->severity, *eiinfo->hf_info.p_id, FALSE, eiinfo->summary, unused);
> > ++ va_end(unused);
> > + return ti;
> > + }
> > +
> > + proto_item *
> > ++proto_tree_add_expert(proto_tree *tree, packet_info *pinfo, expert_field* expindex,
> > ++ tvbuff_t *tvb, gint start, gint length)
> > ++{
> > ++ return proto_tree_add_expert_internal(tree, pinfo, expindex, tvb, start, length);
> > ++}
> > ++
> > ++proto_item *
> > + proto_tree_add_expert_format(proto_tree *tree, packet_info *pinfo, expert_field* expindex,
> > + tvbuff_t *tvb, gint start, gint length, const char *format, ...)
> > + {
> > +--
> > +1.9.1
> > +
> > diff --git a/meta-networking/recipes-support/wireshark/wireshark_1.12.0-rc2.bb b/meta-networking/recipes-support/wireshark/wireshark_1.12.0-rc2.bb
> > index 66716c3..47e72f4 100644
> > --- a/meta-networking/recipes-support/wireshark/wireshark_1.12.0-rc2.bb
> > +++ b/meta-networking/recipes-support/wireshark/wireshark_1.12.0-rc2.bb
> > @@ -14,6 +14,7 @@ PACKAGECONFIG += " ${@bb.utils.contains("IMAGE_FEATURES", "x11-base", "gtk2 grap
> > PACKAGECONFIG += " ${@bb.utils.contains("DISTRO_FEATURES", "ipv6", "ipv6", "", d)}"
> >
> > PACKAGECONFIG[gtk2] = "--with-gtk2=yes,--with-gtk2=no, gtk+"
> > +PACKAGECONFIG[gtk3] = "--with-gtk3=yes, --with-gtk3=no, gtk+3"
> > PACKAGECONFIG[graphics] = "--enable-wireshark, --disable-wireshark,"
> > PACKAGECONFIG[ipv6] = "--enable-ipv6,--disable-ipv6,"
> > PACKAGECONFIG[gnutls] = "--with-gnutls=yes,--with-gnutls=no, gnutls"
> > @@ -22,7 +23,9 @@ PACKAGECONFIG[gcrypt] = "--with-gcrypt=yes,--with-gcrypt=no, libgcrypt"
> > EXTRA_OECONF = "--with-gtk3=no --with-qt=no --enable-usr-local=no -enable-tshark"
> >
> > LIC_FILES_CHKSUM = "file://README.linux;md5=631e077455b7972172eb149195e065b0"
> > -SRC_URI = "http://wiresharkdownloads.riverbed.com/wireshark/src/wireshark-1.12.0-rc2.tar.bz2 "
> > +SRC_URI = "http://wiresharkdownloads.riverbed.com/wireshark/src/wireshark-1.12.0-rc2.tar.bz2 \
> > + file://va_list-can-t-be-NULL-on-ARM.patch \
> > + "
> >
> > SRC_URI[md5sum] = "dc1149073066a29f91116c168558262e"
> > SRC_URI[sha256sum]= "31009bb450126e9b12808267419f31016d14e6fde7b5e39c85ad37459908cffb"
> > --
> > 1.9.1
> >
> > --
> > _______________________________________________
> > 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
--
Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 188 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [meta-networking][PATCH] v3] Wireshark: build fix and recipe update
2014-07-24 10:38 ` Martin Jansa
@ 2014-07-24 20:21 ` akuster
0 siblings, 0 replies; 5+ messages in thread
From: akuster @ 2014-07-24 20:21 UTC (permalink / raw)
To: Martin Jansa; +Cc: openembedded-devel
On 07/24/2014 03:38 AM, Martin Jansa wrote:
> On Wed, Jul 23, 2014 at 03:56:16PM +0200, Martin Jansa wrote:
>> On Mon, Jul 21, 2014 at 03:20:28PM -0700, Armin Kuster wrote:
>>> Backported Arm build fix
>>> Added gtk3 support
>>> Cleaned up PACKAGECONFIG options
>>
>> You cannot use IMAGE_FEATURES to define PACKAGECONFIG, see what I did
>> with it in master-next (and confirm that you're ok with my changes).
>>
>> Also please notice wrong number of ']' in subject, no big deal, but I
>> had to strip 'v3]" from subject lines manually when cherry-picking from
>> patchwork.
>
> Can you please fix these floating dependencies before we merge this?
>
> WARN: packages/armv5te-oe-linux-gnueabi/wireshark/wireshark/latest lost
> dependency on libcap libnl libnl-genl libnl-nf libnl-route
> portaudio-v19 sbc
>
That's no good. Thanks.
I need to get better at catching those prior to submitting.
will submit v3 shortly.
- Armin
>>
>>> Signed-off-by: Armin Kuster <akuster@mvista.com>
>>> ---
>>> .../files/va_list-can-t-be-NULL-on-ARM.patch | 98 ++++++++++++++++++++++
>>> .../wireshark/wireshark_1.12.0-rc2.bb | 5 +-
>>> 2 files changed, 102 insertions(+), 1 deletion(-)
>>> create mode 100644 meta-networking/recipes-support/wireshark/files/va_list-can-t-be-NULL-on-ARM.patch
>>>
>>> diff --git a/meta-networking/recipes-support/wireshark/files/va_list-can-t-be-NULL-on-ARM.patch b/meta-networking/recipes-support/wireshark/files/va_list-can-t-be-NULL-on-ARM.patch
>>> new file mode 100644
>>> index 0000000..301c381
>>> --- /dev/null
>>> +++ b/meta-networking/recipes-support/wireshark/files/va_list-can-t-be-NULL-on-ARM.patch
>>> @@ -0,0 +1,98 @@
>>> +From 320c4f0d705a3ed94f710fb4b7b3eef897ff7bc4 Mon Sep 17 00:00:00 2001
>>> +From: Steev Klimaszewski <threeway@gmail.com>
>>> +Date: Thu, 19 Jun 2014 16:54:57 -0500
>>> +Subject: [PATCH] va_list can't be NULL on ARM.
>>> +
>>> +Bug: 10209
>>> +Change-Id: Ibd63a530450b7d2d4ec244e91c77caa731ba63aa
>>> +Signed-off-by: Steev Klimaszewski <threeway@gmail.com>
>>> +Signed-off-by: Balint Reczey <balint@balintreczey.hu>
>>> +Reviewed-on: https://code.wireshark.org/review/2464
>>> +Reviewed-by: Evan Huus <eapache@gmail.com>
>>> +Reviewed-by: Michael Mann <mmann78@netscape.net>
>>> +
>>> +The patch was imported from the wireshark git server
>>> + (https://code.wireshark.org/review/p/wireshark.git) as of commit id
>>> + 320c4f0d705a3ed94f710fb4b7b3eef897ff7bc4.
>>> +
>>> +Signed-off-by: Armin Kuster <akuster@mvista.com>
>>> +
>>> +---
>>> + epan/expert.c | 36 +++++++++++++++++++++++++++++-------
>>> + 1 file changed, 29 insertions(+), 7 deletions(-)
>>> +
>>> +diff --git a/epan/expert.c b/epan/expert.c
>>> +index 46be838..a69566d 100644
>>> +--- a/epan/expert.c
>>> ++++ b/epan/expert.c
>>> +@@ -381,15 +381,26 @@ expert_set_info_vformat(packet_info *pinfo, proto_item *pi, int group, int sever
>>> + tap_queue_packet(expert_tap, pinfo, ei);
>>> + }
>>> +
>>> +-void
>>> +-expert_add_info(packet_info *pinfo, proto_item *pi, expert_field *expindex)
>>> ++/* Helper function for expert_add_info() to work around compiler's special needs on ARM*/
>>> ++static inline void
>>> ++expert_add_info_internal(packet_info *pinfo, proto_item *pi, expert_field *expindex, ...)
>>> + {
>>> ++ /* the va_list is ignored */
>>> ++ va_list unused;
>>> + expert_field_info* eiinfo;
>>> +
>>> + /* Look up the item */
>>> + EXPERT_REGISTRAR_GET_NTH(expindex->ei, eiinfo);
>>> +
>>> +- expert_set_info_vformat(pinfo, pi, eiinfo->group, eiinfo->severity, *eiinfo->hf_info.p_id, FALSE, eiinfo->summary, NULL);
>>> ++ va_start(unused, expindex);
>>> ++ expert_set_info_vformat(pinfo, pi, eiinfo->group, eiinfo->severity, *eiinfo->hf_info.p_id, FALSE, eiinfo->summary, unused);
>>> ++ va_end(unused);
>>> ++}
>>> ++
>>> ++void
>>> ++expert_add_info(packet_info *pinfo, proto_item *pi, expert_field *expindex)
>>> ++{
>>> ++ expert_add_info_internal(pinfo, pi, expindex);
>>> + }
>>> +
>>> + void
>>> +@@ -406,22 +417,33 @@ expert_add_info_format(packet_info *pinfo, proto_item *pi, expert_field *expinde
>>> + va_end(ap);
>>> + }
>>> +
>>> +-proto_item *
>>> +-proto_tree_add_expert(proto_tree *tree, packet_info *pinfo, expert_field* expindex,
>>> +- tvbuff_t *tvb, gint start, gint length)
>>> ++/* Helper function for expert_add_expert() to work around compiler's special needs on ARM*/
>>> ++static inline proto_item *
>>> ++proto_tree_add_expert_internal(proto_tree *tree, packet_info *pinfo, expert_field* expindex,
>>> ++ tvbuff_t *tvb, gint start, gint length, ...)
>>> + {
>>> + expert_field_info* eiinfo;
>>> + proto_item *ti;
>>> ++ va_list unused;
>>> +
>>> + /* Look up the item */
>>> + EXPERT_REGISTRAR_GET_NTH(expindex->ei, eiinfo);
>>> +
>>> + ti = proto_tree_add_text(tree, tvb, start, length, "%s", eiinfo->summary);
>>> +- expert_set_info_vformat(pinfo, ti, eiinfo->group, eiinfo->severity, *eiinfo->hf_info.p_id, FALSE, eiinfo->summary, NULL);
>>> ++ va_start(unused, length);
>>> ++ expert_set_info_vformat(pinfo, ti, eiinfo->group, eiinfo->severity, *eiinfo->hf_info.p_id, FALSE, eiinfo->summary, unused);
>>> ++ va_end(unused);
>>> + return ti;
>>> + }
>>> +
>>> + proto_item *
>>> ++proto_tree_add_expert(proto_tree *tree, packet_info *pinfo, expert_field* expindex,
>>> ++ tvbuff_t *tvb, gint start, gint length)
>>> ++{
>>> ++ return proto_tree_add_expert_internal(tree, pinfo, expindex, tvb, start, length);
>>> ++}
>>> ++
>>> ++proto_item *
>>> + proto_tree_add_expert_format(proto_tree *tree, packet_info *pinfo, expert_field* expindex,
>>> + tvbuff_t *tvb, gint start, gint length, const char *format, ...)
>>> + {
>>> +--
>>> +1.9.1
>>> +
>>> diff --git a/meta-networking/recipes-support/wireshark/wireshark_1.12.0-rc2.bb b/meta-networking/recipes-support/wireshark/wireshark_1.12.0-rc2.bb
>>> index 66716c3..47e72f4 100644
>>> --- a/meta-networking/recipes-support/wireshark/wireshark_1.12.0-rc2.bb
>>> +++ b/meta-networking/recipes-support/wireshark/wireshark_1.12.0-rc2.bb
>>> @@ -14,6 +14,7 @@ PACKAGECONFIG += " ${@bb.utils.contains("IMAGE_FEATURES", "x11-base", "gtk2 grap
>>> PACKAGECONFIG += " ${@bb.utils.contains("DISTRO_FEATURES", "ipv6", "ipv6", "", d)}"
>>>
>>> PACKAGECONFIG[gtk2] = "--with-gtk2=yes,--with-gtk2=no, gtk+"
>>> +PACKAGECONFIG[gtk3] = "--with-gtk3=yes, --with-gtk3=no, gtk+3"
>>> PACKAGECONFIG[graphics] = "--enable-wireshark, --disable-wireshark,"
>>> PACKAGECONFIG[ipv6] = "--enable-ipv6,--disable-ipv6,"
>>> PACKAGECONFIG[gnutls] = "--with-gnutls=yes,--with-gnutls=no, gnutls"
>>> @@ -22,7 +23,9 @@ PACKAGECONFIG[gcrypt] = "--with-gcrypt=yes,--with-gcrypt=no, libgcrypt"
>>> EXTRA_OECONF = "--with-gtk3=no --with-qt=no --enable-usr-local=no -enable-tshark"
>>>
>>> LIC_FILES_CHKSUM = "file://README.linux;md5=631e077455b7972172eb149195e065b0"
>>> -SRC_URI = "http://wiresharkdownloads.riverbed.com/wireshark/src/wireshark-1.12.0-rc2.tar.bz2 "
>>> +SRC_URI = "http://wiresharkdownloads.riverbed.com/wireshark/src/wireshark-1.12.0-rc2.tar.bz2 \
>>> + file://va_list-can-t-be-NULL-on-ARM.patch \
>>> + "
>>>
>>> SRC_URI[md5sum] = "dc1149073066a29f91116c168558262e"
>>> SRC_URI[sha256sum]= "31009bb450126e9b12808267419f31016d14e6fde7b5e39c85ad37459908cffb"
>>> --
>>> 1.9.1
>>>
>>> --
>>> _______________________________________________
>>> 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
>
>
>
>
>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2014-07-24 20:21 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-07-21 22:20 [meta-networking][PATCH] v3] Wireshark: build fix and recipe update Armin Kuster
2014-07-23 13:56 ` Martin Jansa
2014-07-23 17:49 ` akuster
2014-07-24 10:38 ` Martin Jansa
2014-07-24 20:21 ` akuster
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.