Opened 10 years ago

Closed 9 years ago

#22746 closed enhancement (fixed)

wireshark 1.2.4 enhancement: libsmi

Reported by: cwt@… Owned by: ghosthound
Priority: Normal Milestone:
Component: ports Version: 1.8.1
Keywords: Cc: hsivank@…, miken32 (Michael Newton)
Port: wireshark

Description

For some number of releases going back at least to 1.0.6, wireshark has replaced net-snmp for mib decoding, instead using libsmi. At least as of wireshark 1.0.6, our port can trivially be modified to support a libsmi variant:

variant libsmi {

configure.args-append --with-libsmi depends_lib-append port:libsmi

}

I was able to build a functional, mib translating wireshark using libsmi for wireshark 1.0.6 with this change. I have not tried for 1.2.4 but it would seem likely to work here as well.

It may not even make sense to have a net-snmp variant any longer because I believe wireshark will not use it even if asked for (should be verified).

Attachments (1)

wireshark.diff (632 bytes) - added by miken32 (Michael Newton) 9 years ago.
So what are we waiting for, a patch? Here it is!

Download all attachments as: .zip

Change History (7)

comment:1 Changed 10 years ago by jmroot (Joshua Root)

Keywords: wireshark snmp removed
Owner: changed from macports-tickets@… to ricci@…

comment:2 Changed 10 years ago by hsivank@…

Cc: hsivank@… added

Cc Me!

comment:3 Changed 10 years ago by hsivank@…

Hello cwt,

Work great with wireshark-devel (1.3.2) ! I posted a patch to add libsmi variant #22822

Thanks

comment:4 Changed 10 years ago by miken32 (Michael Newton)

Cc: miken32@… added

Cc Me!

comment:5 Changed 10 years ago by miken32 (Michael Newton)

Patch works fine for 1.25. Wireshark has removed the net-snmp requirement from their build completely, due to licensing preferences. The --with-net-snmp argument does nothing.

Changed 9 years ago by miken32 (Michael Newton)

Attachment: wireshark.diff added

So what are we waiting for, a patch? Here it is!

comment:6 Changed 9 years ago by ghosthound

Resolution: fixed
Status: newclosed

resolved in r67255

Note: See TracTickets for help on using tickets.