Re: [PATCH 0/2] two small patches for CEPH wireshark plugin

2013-01-28 Thread David Zafman
You could look at the wip-wireshark-zafman branch. I rebased it and force pushed it. It has changes to the wireshark.patch and a minor change I needed to get it to build. I'm surprised the recent checkin didn't include the change to packet-ceph.c which I needed to get it to build. David

[PATCH 0/2] two small patches for CEPH wireshark plugin

2013-01-24 Thread Danny Al-Gaaf
Danny Al-Gaaf (2): wireshark: fix guint64 print format handling wireshark: fix indention wireshark/ceph/packet-ceph.c | 46 ++-- 1 file changed, 23 insertions(+), 23 deletions(-) -- 1.8.1.1 -- To unsubscribe from this list: send the line unsubscribe

Re: [PATCH 0/2] two small patches for CEPH wireshark plugin

2013-01-24 Thread Sage Weil
Hi Danny! On Thu, 24 Jan 2013, Danny Al-Gaaf wrote: Danny Al-Gaaf (2): wireshark: fix guint64 print format handling wireshark: fix indention wireshark/ceph/packet-ceph.c | 46 ++-- 1 file changed, 23 insertions(+), 23 deletions(-) Since you

Re: [PATCH 0/2] two small patches for CEPH wireshark plugin

2013-01-24 Thread Sage Weil
On Thu, 24 Jan 2013, Danny Al-Gaaf wrote: Am 24.01.2013 19:31, schrieb Sage Weil: Hi Danny! [...] Since you brought up wireshark... We would LOVE LOVE LOVE it if this plugin could get upstream into wireshark. Yes, this would be great. IIRC, the problem (last time we checked,