package wireshark with osmocom patches
- packages our wireshark as packages in OBS as part of the feeds
- possibly even re-bases our non-mainline patches before each build
This is particularly important for the AMR voice playback, as wireshark has refused to include this upstream. We can also include any other not-yet-mainline code that we have at osmocom, like osmux statistics, Jacob's EGPRS patch, Ericsson RSL support, ...
It would probably be best if both the package as well as the file names are modified to not conflict with the distribution-standard wireshark package that mostly everyone has installed. Let's hope this is not too hard to do. Alternatively we could install into something like an /opt/osmocom PREFIX. A bit ugly, but then it would avoid having to patch the wireshark source?
Please coordinate with Daniel on this.
#3 Updated by pespin over 2 years ago
- Assignee changed from daniel to laforge
Other patches from daniel/osmux should be already merged in upstream.
There seem to be other branches in our wireshark repo at osmocom.org:
laforge/gsup Dissector for the Osmocom GSUP Protocol Harald Welte 3 months laforge/pending Add dissector for OsmoTRX protocol Harald Welte 2 weeks laforge/qcdiag qcdiag_log: Implement RRC dissection Harald Welte 15 months laforge/trx Add dissector for OsmoTRX protocol Harald Welte 5 weeks
What's the status of these branches? Are they merged upstream? Should I copy all those patches into the same branch and use the branch as source to build our own wireshark?
Assigning to laforge as the branches are from him.
#4 Updated by laforge over 2 years ago
- Assignee changed from laforge to pespin
- gsup has just been merged upstream
- trx is about to get merged upstream
- qcdiag is nowhere near
- for "pending" you will have to rebase/review. I think at least the EGPRS patch as well as the "hand RLC payload to LLC" are not upstream.
#5 Updated by pespin over 2 years ago
- Status changed from Feedback to In Progress
I started adding commits to osmocom/master branch.
gsup: Already in master, we get it for free.
trx: Available in laforge/trx and in wireshark's gerrit, still not merged.
qcdiag: Several commits in laforge/qcdiag. I didn't look at those yet.
pendign: In laforge/pendign. Several L2TP, GPRS-RLC and EGPRS patches.
AMR: Original commit in daniel/osmux. During rebase I had to fix some small bits and add all the CMake build bits since autoconf was dropped in master. New branch can be found in pespin/amr and is already merged in osmocom/master.
I also started work on building wireshark in OBS: https://build.opensuse.org/package/show/home:pespin:branches:network:osmocom:nightly/wireshark#
I had to modify debian/control to depend on libgnutsl28-dev instead of libgnutls-dev. Debian package also has this patch. See 7580784d4bc98544684607c70c6d490c0f9316cc in our wireshark repo in osmocom/master branch.
I also had to modify the Project Config to tell OBS which package to use as preferred, otherwise it won't build: