Wireshark mailing list archives
Re: [Wireshark-commits] rev 47987: /trunk/tools/lemon/ /trunk/tools/lemon/: lemon.c
From: Anders Broman <a.broman () bredband net>
Date: Sat, 02 Mar 2013 00:21:58 +0100
Evan Huus skrev 2013-03-01 22:53:
It might just be lack of trying to synchronise, but I tried to push changes upstream a couple of years ago and the interest was minimal...On Fri, Mar 1, 2013 at 4:34 PM, Anders Broman <a.broman () bredband net> wrote:Evan Huus skrev 2013-03-01 22:03:On Fri, Mar 1, 2013 at 3:56 PM, Anders Broman <a.broman () bredband net> wrote:Evan Huus skrev 2013-03-01 21:38:Not entirely sure this sort of thing is necessary - lemon is not our tool, so even if Wireshark switches to C++ there's no immediately obvious reason to start compiling other people's code (which we happen to use) in C++ as well. EvanIt's nice to get rid of the warnings regardless, isn't it?For tools with an upstream maintainer independent of Wireshark (as is the case with lemon - it is maintained by sqlite for some reason) I think it is generally better to avoid carrying a delta with upstream unless absolutely necessary. It may be worth submitting your changes to sqlite, then pulling a new version of lemon once they merge the changes.This is actually copying changes from upstream to Wireshark unfortunately the two has diverged and strict back porting isn't possible(?)I wouldn't have expected them to diverge at all, although we may no longer have the latest upstream version. Does anybody know why we might have needed local changes to our copy of lemon?
/Anders
___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev () wireshark org> Archives: http://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-request () wireshark org?subject=unsubscribe
___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev () wireshark org> Archives: http://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-request () wireshark org?subject=unsubscribe
Current thread:
- Re: [Wireshark-commits] rev 47987: /trunk/tools/lemon/ /trunk/tools/lemon/: lemon.c Evan Huus (Mar 01)
- Re: [Wireshark-commits] rev 47987: /trunk/tools/lemon/ /trunk/tools/lemon/: lemon.c Anders Broman (Mar 01)
- Re: [Wireshark-commits] rev 47987: /trunk/tools/lemon/ /trunk/tools/lemon/: lemon.c Evan Huus (Mar 01)
- Re: [Wireshark-commits] rev 47987: /trunk/tools/lemon/ /trunk/tools/lemon/: lemon.c Anders Broman (Mar 01)
- Re: [Wireshark-commits] rev 47987: /trunk/tools/lemon/ /trunk/tools/lemon/: lemon.c Evan Huus (Mar 01)
- Re: [Wireshark-commits] rev 47987: /trunk/tools/lemon/ /trunk/tools/lemon/: lemon.c Anders Broman (Mar 01)
- Re: [Wireshark-commits] rev 47987: /trunk/tools/lemon/ /trunk/tools/lemon/: lemon.c Evan Huus (Mar 01)
- Re: [Wireshark-commits] rev 47987: /trunk/tools/lemon/ /trunk/tools/lemon/: lemon.c Anders Broman (Mar 01)