Bug 1560284

Summary:	wireshark-cli adds apparently unneeded dependency on python2
Product:	[Fedora] Fedora	Reporter:	Matías Zúñiga <matias.nicolas.zc>
Component:	wireshark	Assignee:	Michal Ruprich <mruprich>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	28	CC:	denis, huzaifas, lemenkov, mruprich, msehnout, phatina, rvokal
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2018-05-02 12:22:41 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	661442

Description Matías Zúñiga 2018-03-25 15:57:20 UTC

Installing wireshark pulls python2 as a dependency via wireshark-cli.

Searching for why this dependency is necessary i found 2 scripts, wireshark_be.py and wireshark_gen.py installed on site-packages. The only use i could find of these files is within the idl2wrs bash script [1], which is NOT included in the fedora package.

If this is true, and there are not other uses for this python2 code, i think these scripts should not be included, as this adds the python2 dependency.


[1] https://www.wireshark.org/docs/man-pages/idl2wrs.html

Comment 1 Matías Zúñiga 2018-03-25 18:18:00 UTC

Grepping the source code i can confirm that these file are only used in the removed idl2wrs script (and there are other references in documentation and Makefile). 

Looking at other distros, the Arch linux packages don't depend on python2, while the only Debian sub-package with python2 dependency is wireshark-dev, which contains the mentioned files, idl2wrs, and 2 other scripts [1] (also not included by fedora)

[1] https://packages.debian.org/sid/amd64/wireshark-dev/filelist

Comment 2 Michal Ruprich 2018-05-02 12:22:41 UTC

Fixed in current rawhide version.