Harald Welte has posted comments on this change. ( 
https://gerrit.osmocom.org/13253 )

Change subject: Use dumpcap for ttcn3-tcpdump-*.sh if available
......................................................................


Patch Set 1:

I really like the idea and I'm surprised we didn't do this sooner.  However, 
the fact that dumpcap is installed doesn't mean that a) it's setuid-root, or b) 
the current user has the right group/permission

On Debian systems, for example, it's part of the package configuration that is 
performed during package installation.

I think for this to work properly we'd need some way to test if capturing 
actually worked or if it was rejected before taking a decision for dumpcap.


--
To view, visit https://gerrit.osmocom.org/13253
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I77df04d51a469c924cf727f0596cc33565909746
Gerrit-Change-Number: 13253
Gerrit-PatchSet: 1
Gerrit-Owner: Max <[email protected]>
Gerrit-Reviewer: Daniel Willmann <[email protected]>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-Reviewer: Max <[email protected]>
Gerrit-Reviewer: osmith <[email protected]>
Gerrit-CC: Harald Welte <[email protected]>
Gerrit-Comment-Date: Thu, 14 Mar 2019 20:44:55 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: No

Reply via email to