Huge thanks to our Platinum Members Endace and LiveAction,
and our Silver Member Veeam, for supporting the Wireshark Foundation and project.

Wireshark-dev: Re: [Wireshark-dev] Building RPM proprietarry plugin including math.h fails

From: Anders Broman <anders.broman@xxxxxxxxxxxx>
Date: Wed, 29 Aug 2018 04:15:50 +0000

-----Original Message-----
From: Wireshark-dev <wireshark-dev-bounces@xxxxxxxxxxxxx> On Behalf Of João Valverde
Sent: den 29 augusti 2018 01:40
To: wireshark-dev@xxxxxxxxxxxxx
Subject: Re: [Wireshark-dev] Building RPM proprietarry plugin including math.h fails



On 29/08/18 00:30, João Valverde wrote:
> 
> 
> On 28/08/18 19:39, João Valverde wrote:
>>
>>
>> On 28/08/18 16:59, Anders Broman wrote:
>>>
>>>
>>> Den tis 28 aug. 2018 16:25Ed Beroset <beroset@xxxxxxxxxxxxxx 
>>> <mailto:beroset@xxxxxxxxxxxxxx>> skrev:
>>>
>>>     On 08/28/2018 09:35 AM, Anders Broman wrote:
>>>      > Hi,
>>>      >
>>>      > tfo/packet-tfo.c:3754: undefined reference to `pow'
>>>      >
>>>      > collect2: error: ld returned 1 exit status
>>>      >
>>>      > when running make-rpm-package
>>>
>>>     That's the symptom of missing the math library on the linker 
>>> command
>>>     line.  You'd need to add '-lm' to the linker line, if that's 
>>> what
>>>     you're
>>>     asking about.
>>>
>>>     Ed
>>>
>>>
>>> Sure but somthing changed recently in the cmake files and broke it, 
>>> as it has worked before.
>>> I'm trying a fix in the cmake file in epan.
>>> Anders
>>>
>>
>> You could try using PUBLIC with M_LIBRARIES.
>>
>> https://code.wireshark.org/review/#/c/29208/16/epan/CMakeLists.txt
>>
>> Is this your own custom build or not?
> 
> I see now that you said proprietary plugin in the subject. Just add 
> M_LIBRARIES to your build?

If indeed that is the problem (very likely) the decision to make these libraries PRIVATE was intentional because the only external linkage requirement that I saw was glib.

Yes this was the problem as I suspected yesterday, it just took a while to verify as I had to delete the build directory to have the cmake changes take effect. Another question
Is this bug https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=15084 also related to the cmake changes?
Regards
Anders
___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev@xxxxxxxxxxxxx>
Archives:    https://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-request@xxxxxxxxxxxxx?subject=unsubscribe