On 08/07/2016 06:17 PM, Pascal Quantin wrote:
2016-08-07 18:58 GMT+02:00 João Valverde <joao.valve...@tecnico.ulisboa.pt <mailto:joao.valve...@tecnico.ulisboa.pt>>: On 08/07/2016 05:23 PM, Hadriel Kaplan wrote: ---------- Forwarded message ---------- From: João Valverde <joao.valve...@tecnico.ulisboa.pt <mailto:joao.valve...@tecnico.ulisboa.pt>> Date: Sun, Aug 7, 2016 at 2:39 PM Subject: Re: [Wireshark-dev] Current Lua test failures on the buildbot To: Developer support list for Wireshark <wireshark-dev@wireshark.org <mailto:wireshark-dev@wireshark.org>> Seriously, what's the point of this craziness when this works fine: $ luarocks install lrexlib-pcre-2.8.0-1.src.rock $ cat wslua-pcre.lua rex = require("rex_pcre") print(rex.version()) $ tshark -r test/captures/empty.pcap -X lua_script:wslua-pcre.lua 8.39 2016-06-14 Digging around a bit I see that Hadriel didn't want to complicate the build process with a libpcre dependency (!) so he ended up writing a whole new lrexlib binding (!!!). I propose we write our own regular expression engine instead so we don't have to worry about pesky complications such as having a build dependency on GRegex. :-D At the time, wireshark didn't directly include PCRE as a third party library, and for some reason I assumed doing so would be a big deal. It seemed like folks didn't want to add more libraries to the dependencies list. So... I figured using the one exposed in Glib, which was already being used for display filters, would be less hassle overall - more hassle for me, less for others. The lrexlib portion was just direct source so didn't add build dependencies, and I did offer the Glib-regex binding code back upstream to the lrexlib folks. They didn't want it since they were only really handling bug fixes by then, not growing it. (and a Glib-regex binding would be odd anyway, since most people wouldn't care about just using PCRE directly) But if folks prefer to add a dependency for PCRE and lrexlib to Wireshark, then sure I could get rid of the Glib one and expose the lrexlib one into Wireshark's lua engine instead. Does this even require adding a PCRE build dependency to wireshark? I mean isn't lrexlib a runtime dependency for wslua? For example for Debian I think it's as simple as "apt-get install lua-rex-pcre". Maybe I'm misunderstanding, thanks for your help. :-) And what about OSX / Windows? Is it as easy to install a new dependency (I guess the answer is no)?
Indeed, for Windows at least, probably not so easy. We would have to build the PCRE DLL.
Pascal. ___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev@wireshark.org> Archives: https://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-requ...@wireshark.org?subject=unsubscribe
___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev@wireshark.org> Archives: https://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-requ...@wireshark.org?subject=unsubscribe
