[libvirt PATCH] wireshark: Don't include config.h

[Michal Prívozník]

On 9/4/20 5:12 PM, Andrea Bolognani wrote:
> On Fri, 2020-09-04 at 15:30 +0200, Michal Prívozník wrote:
>> On 9/4/20 2:21 PM, Andrea Bolognani wrote:
>>> On Fri, 2020-09-04 at 11:50 +0200, Michal Prívozník wrote:
>>>> The patch looks good. ws_version.h was introduced with 2.9.0 release
>>>> which is 1.5 years old. Given that the dissector is aimed mostly on us,
>>>> developers to help us debug RPC issues, I think we can safely bump the
>>>> minimal wireshark version required (currently 2.4.0 which is 3 years old).
>>>
>>> That sounds reasonable in theory, but if you look at
>>>
>>>     https://gitlab.com/abologna/libvirt/-/pipelines/185421025
>>>
>>> you'll see that even platforms that ship pretty recent Wireshark[1]
>>> don't include ws_version.h among the headers.
>>>
>>> Not building the dissector on those non-obsolete platforms seems
>>> excessively harsh, so I think an approach similar to the one I
>>> described above is still necessary. And at that point, you might as
>>> well not bump the minimum required version and keep building the
>>> dissector on the current list of platforms...
>>
>> Any idea why they are not installing the file? Because while current
>> solution is hacky, intentionally removing a header file that a package
>> wants installed is way worse.
> 
> I don't think it's done on purpose: it's probably just a bug in the
> Debian packaging that got propagated to Ubuntu.
> 
> Even assuming that's the case, it will take some time for it to be
> addressed in sid, and the first Ubuntu LTS that will carry the
> resulting fix is almost two years out... So I think we have to just
> support both ws_version.h and config.h for a while.
> 

Ah, so on one hand we have progressive distro that doesn't install 
internal header files, on the other we have LTS distros where changing 
something may take years to take effect. In that case our only option is 
to implement both ways. Or motivate LTS distros to implement the change 
sooner ;-)

Since you wrote the original patch, do you want to write this one too? 
Or do you want me to do it?

Michal