Bug 1255553 - [Feature Request] Build Wireshark with lua support
Summary: [Feature Request] Build Wireshark with lua support
Keywords:
Status: CLOSED DUPLICATE of bug 1259623
Alias: None
Product: Fedora
Classification: Fedora
Component: wireshark
Version: 22
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Peter Hatina
QA Contact: Fedora Extras Quality Assurance
URL: https://ask.wireshark.org/questions/4...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-08-20 23:18 UTC by Andrew Cook
Modified: 2016-12-07 15:26 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-08-21 13:11:40 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
Disable bitop module (5.16 KB, patch)
2015-08-21 13:20 UTC, Andrew Cook
no flags Details | Diff

Description Andrew Cook 2015-08-20 23:18:32 UTC
Description of problem:
No lua support in wireshark

This is useful for quickly writing dissectors

spec is already set up to build with lua, though currently disabled

Comment 1 Andrew Cook 2015-08-20 23:27:10 UTC
Apparently fails to build:

  CC       lua_bitop.lo
./lua_bitop.c: In function 'barg':
./lua_bitop.c:85:2: error: #error "Unknown number type, check LUA_NUMBER_* in luaconf.h"
 #error "Unknown number type, check LUA_NUMBER_* in luaconf.h"
  ^
./lua_bitop.c:61:10: warning: variable 'bn' set but not used [-Wunused-but-set-variable]
   BitNum bn;
          ^
Makefile:690: recipe for target 'lua_bitop.lo' failed

Comment 2 Andrew Cook 2015-08-21 01:10:23 UTC
Might outright require lua 5.2

Comment 3 Peter Lemenkov 2015-08-21 13:11:40 UTC
(In reply to Andrew Cook from comment #2)
> Might outright require lua 5.2

Andrew we can't do it for now. If you have a working patch which allows building against Lua 5.3.x that would be really helpful. You may want to check patch attached in Bug #1230286.

Feel free to reopen if find something.

*** This bug has been marked as a duplicate of bug 1230286 ***

Comment 4 Andrew Cook 2015-08-21 13:20:59 UTC
Created attachment 1065607 [details]
Disable bitop module

Got it to build by removing the bitop module, while fine for my use a proper solution would replace bitop.
5.3 adds bitwise operators which i assume are equivalent in all but syntax

Comment 5 Peter Lemenkov 2016-12-07 15:26:09 UTC

*** This bug has been marked as a duplicate of bug 1259623 ***


Note You need to log in before you can comment on or make changes to this bug.