Bug 2119126
| Summary: | [RHEL8.6] Wireshark source rpm fails to build due to looking for incorrect libssh header files. | ||
|---|---|---|---|
| Product: | Red Hat Enterprise Linux 8 | Reporter: | Brandon Clark <brclark> |
| Component: | wireshark | Assignee: | Michal Ruprich <mruprich> |
| Status: | CLOSED ERRATA | QA Contact: | FrantiĊĦek Hrdina <fhrdina> |
| Severity: | low | Docs Contact: | |
| Priority: | unspecified | ||
| Version: | 8.6 | CC: | alanm, amike, ansasaki, areis, brclark, casantos, fhrdina, fkrska, jjelen, jorton, jwright, mkielian, mkolbas, mruprich |
| Target Milestone: | rc | Keywords: | Triaged |
| Target Release: | --- | Flags: | pm-rhel:
mirror+
|
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Whiteboard: | |||
| Fixed In Version: | wireshark-2.6.2-15.el8 | Doc Type: | If docs needed, set a value |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2022-11-08 10:03:34 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: | |||
|
Description
Brandon Clark
2022-08-17 15:38:46 UTC
Interesting, I thought these kinds of things would have been caught in the time when developing RHEL-8.6. Nevertheless, I am not aware of any conversation or announcement of changes of headers in libssh(please refer me to one if there was one that I missed). Changing headers in the middle of RHEL-X development does not sound like a good idea and if the owner of the library has very good reasons for it, he/she should notify everyone involved. Meaning everyone using the library, simple dnf query could help. This breakage was not caused by wireshark since this package has not been touched since RHEL-8.5.0 so I am reassigning this to libssh. Either this should have been communicated properly or a compat solution should have been created. Sounds like this is a side-effect of the changes in libssh header files structure and splitting the version information to separate file (to prevent bumping the version on several places for each release). These changed do not have any functional difference, but it looks like it confuses naive cmake modules in wireshark. The wireshark has this already fixed in upstream. We do not plan to introduce any backward compatibility in libssh side so I would suggest the customer to try the following patch for wireshark and probably suggest to backport it also in the wireshark, if you will be doing some update in the future. https://github.com/wireshark/wireshark/commit/fd7739de6bc679036c02c7aabbc3f71783751e3d I have already implemented my own patch that is functionally identical to the patch upstream you linked. Just please be aware that the latest version of the wireshark source RPM from Red Hat will not build due to this. I think you missed the point from comment #1 here Jakube. I know about the fix from upstream and the naivety of the wireshark check is not what is discussed here. The point is that you introduced a change in headers and did not bother to inform those who are involved. This could have been fixed in RHEL-8.6.0 together with the change in libssh. This will probably result in an unnecessary z-stream errata if the customer wants this fixed in 8.6.0. Make sure that if you make a change like this in the future, at least make a simple dnf query to check whether this could influence someone else. Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory (wireshark bug fix and enhancement update), and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://access.redhat.com/errata/RHBA-2022:7663 |