Wireshark mailing list archives

Re: Swap 'v'|'V' options for editcap and mergecap


From: chuck c <bubbasnmp () gmail com>
Date: Mon, 18 Oct 2021 23:17:52 -0500

#15483 - Longest journey, first steps...
https://gitlab.com/wireshark/wireshark/-/merge_requests/4611

I was hoping to work towards (for -h and -v) something
like LONGOPT_CAPTURE_COMMON in
https://gitlab.com/wireshark/wireshark/-/blob/master/capture_opts.h
and ws_log_print_usage() in
https://gitlab.com/wireshark/wireshark/-/blob/master/wsutil/wslog.c.

On Mon, Oct 18, 2021 at 9:12 PM Maynard, Christopher <
Christopher.Maynard () igt com> wrote:

It would be nice if they were all documented too.  See also:
https://gitlab.com/wireshark/wireshark/-/issues/15483.
- Chris

-----Original Message-----
From: Wireshark-dev <wireshark-dev-bounces () wireshark org> On Behalf
Of Gerald Combs
Sent: Monday, October 18, 2021 9:48 PM
To: Developer support list for Wireshark <wireshark-dev () wireshark org>;
chuck c <bubbasnmp () gmail com>
Subject: Re: [Wireshark-dev] Swap 'v'|'V' options for editcap and
mergecap

CAUTION: This email originated outside of IGT. Do not click links or open
attachments unless you recognize the sender and know the content is safe.



I'm not sure if we've ever swapped flags, but Wireshark's "-m" flag was
marked as deprecated prior to the 2.2 release (bcae998048) and removed
prior to 2.4 (37252634c4). I don't have any objections to doing something
similar for the "-v" and "-V" flags.

On 10/16/21 11:35 AM, chuck c wrote:
Is there any precedent for changing command line options after a
program
has been in production for some time?

Swapping "v" and "V" for editcap and mergecap would bring them in line
with the other binaries for calling show_version().
And also align with the verbose option ('V') for tshark and tfshark.
(little disruption depending on how many scripts exist that use these
commands)

Another approach would be to drop "-v" for version and only support "--
version" as tcpdump does:
https://www.tcpdump.org/manpages/tcpdump.1.html
<https://www.tcpdump.org/manpages/tcpdump.1.html>
(big disruption since it's muscle memory to have Q&A people run
"wireshark -v" and paste the output)

https://code.wireshark.org/review/#/c/2489/
<https://code.wireshark.org/review/#/c/2489/>
Add command-line argument to request the version to a number of the
utilities.

https://gitlab.com/wireshark/wireshark/-/commit/3773a756
<https://gitlab.com/wireshark/wireshark/-/commit/3773a756>












CONFIDENTIALITY NOTICE: This message is the property of International Game
Technology PLC and/or its subsidiaries and may contain proprietary,
confidential or trade secret information. This message is intended solely
for the use of the addressee. If you are not the intended recipient and
have received this message in error, please delete this message from your
system. Any unauthorized reading, distribution, copying, or other use of
this message or its attachments is strictly prohibited.

___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev () wireshark org>
Archives:    https://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-request () wireshark org?subject=unsubscribe

Current thread: