Wireshark-dev: Re: [Wireshark-dev] Build instability
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Kaul <mykaul@xxxxxxxxx>
Date: Tue, 10 Dec 2013 23:51:20 +0200
+ another failure that's fixed now - and I've managed to get Wireshark to compile!
Woohoo! and thanks for the fixes.

Y.


On Tue, Dec 10, 2013 at 11:44 PM, Kaul <mykaul@xxxxxxxxx> wrote:
Also fixed, now epan/dissectors/packet-smpp.c is completely broken.
This should fix it:
svn diff epan/dissectors/packet-smpp.c
Index: epan/dissectors/packet-smpp.c
===================================================================
--- epan/dissectors/packet-smpp.c       (revision 53918)
+++ epan/dissectors/packet-smpp.c       (working copy)
@@ -85,7 +85,7 @@

 /* Forward declarations         */
 void proto_register_smpp(void);
-void proto_reg_handoff_smpp(void)
+void proto_reg_handoff_smpp(void);
 static int dissect_smpp(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void* data);
 static guint get_smpp_pdu_len(packet_info *pinfo, tvbuff_t *tvb, int offset);
 static int dissect_smpp_pdu(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void* data _U_);



On Tue, Dec 10, 2013 at 10:59 PM, Kaul <mykaul@xxxxxxxxx> wrote:
And the above has been fixed.
Regretfully, replaced by another compilation failure: 

Y.


On Tue, Dec 10, 2013 at 8:19 AM, Kaul <mykaul@xxxxxxxxx> wrote:
Above compilation failure has been fixed.
Regretfully, only to be replaced by another compilation failure:


On Mon, Dec 9, 2013 at 2:49 PM, Kaul <mykaul@xxxxxxxxx> wrote:


On Sun, Dec 8, 2013 at 9:52 PM, Jakub Zawadzki <darkjames-ws@xxxxxxxxxxxx> wrote:
On Sun, Dec 08, 2013 at 09:23:23PM +0200, Kaul wrote:
> Thanks for the automated build links - I guess I'll watch them more closely
> and only sync when my platform passes.
>
> Interesting - I'm pulling from trunk and still fail on that. Perhaps it
> wasn't fixed entirely, or I have to do some cleanup?

I agree, it isn't fixed, from_hex() is defined inside:
  #if defined(HAVE_LIBGNUTLS) && defined(HAVE_LIBGCRYPT)

but it's used outside this #ifdef.


Kuba.

>
>
> On Sun, Dec 8, 2013 at 6:51 PM, Hauke Mehrtens <hauke@xxxxxxxxxx> wrote:
>
> > On 12/08/2013 04:30 PM, Kaul wrote:
> > > Hi all,
> > >
> > > I've been trying to enhance a specific dissector for two weeks now.
> > > Since I'm afraid of diverging the code (although I'm working on a
> > > specific dissector), I update my code base often (~ once a day).
> > > Regretfully, 5 times (in 2 weeks!) this has resulted in compilation
> > failure.
> > > I'm pretty sure (subjectively) this hasn't happened in the past. Any
> > > measures we can put (continuous build system?) in place to prevent this?
> > >
> > > Today's failure (2nd failure only today btw):
> > > make[5]: Entering directory `/home/yanivk/wireshark/epan/dissectors'
> > >   CC       libdissectors_la-packet-ssl-utils.lo
> > > packet-ssl-utils.c: In function 'ssl_keylog_parse_session_id':
> > > packet-ssl-utils.c:4201:9: error: implicit declaration of function
> > > 'from_hex_char' [-Werror=implicit-function-declaration]
> > >          if (from_hex_char(line[2*i]) !=
> > > (ssl_session->session_id.data[i] >> 4) ||
> > >          ^
> > > packet-ssl-utils.c:4218:5: error: implicit declaration of function
> > > 'from_hex' [-Werror=implicit-function-declaration]
> > >      if (!from_hex(&ssl_session->master_secret, line, len))
> > >      ^
> > > cc1: all warnings being treated as errors
> > > make[5]: *** [libdissectors_la-packet-ssl-utils.lo] Error 1
> > >
> > > Thanks,
> > > Y.
> >
> > Hi,
> >
> > There are automated build, they are described here:
> > http://www.wireshark.org/docs/wsdg_html_chunked/ChIntroAutomated.html
> >
> > The last build results are here:
> > http://buildbot.wireshark.org/trunk/one_line_per_build
> >
> > The problem you current have was already fixed in current trunk some
> > hours ago.
> >
> > Hauke
> >

> ___________________________________________________________________________
> Sent via:    Wireshark-dev mailing list <wireshark-dev@xxxxxxxxxxxxx>
> Archives:    http://www.wireshark.org/lists/wireshark-dev
> Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
>              mailto:wireshark-dev-request@xxxxxxxxxxxxx?subject=unsubscribe

___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev@xxxxxxxxxxxxx>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-request@xxxxxxxxxxxxx?subject=unsubscribe