Re: [Wireshark-dev] Editcap link warnings

2013-03-08 Thread Bálint Réczey
Thanks! We could remove even this hack by linking editcap with libwireshark. Is there still a strong reason for not doing so? Cheers, Balint 2013/3/7 Gerald Combs ger...@wireshark.org: This didn't work because prior includes (wsgetopt.h in editcap.c and privileges.h in capinfos.c) lock in the

Re: [Wireshark-dev] Editcap link warnings

2013-03-07 Thread Bálint Réczey
I think the attached patches would make it be a bit less hack-ish. Cheers, Balint 2013/3/7 Gerald Combs ger...@wireshark.org: -DENABLE_STATIC instead of -DWS_BUILD_DLL worked on my machine. Checked in in r48158. On 3/6/13 8:52 AM, Christopher Maynard wrote: Pascal Quantin pascal.quantin@...

Re: [Wireshark-dev] Editcap link warnings

2013-03-07 Thread Bálint Réczey
If no one objects I commit those tomorrow. Cheers, Balint 2013/3/7 Bálint Réczey bal...@balintreczey.hu: I think the attached patches would make it be a bit less hack-ish. Cheers, Balint 2013/3/7 Gerald Combs ger...@wireshark.org: -DENABLE_STATIC instead of -DWS_BUILD_DLL worked on my

Re: [Wireshark-dev] Editcap link warnings

2013-03-07 Thread Pascal Quantin
2013/3/7 Gerald Combs ger...@wireshark.org This didn't work because prior includes (wsgetopt.h in editcap.c and privileges.h in capinfos.c) lock in the export behavior. I checked in your changes in r48170 along with a workaround to reset the export definitions. It's clunkier than I would like

Re: [Wireshark-dev] Editcap link warnings

2013-03-07 Thread Gerald Combs
This didn't work because prior includes (wsgetopt.h in editcap.c and privileges.h in capinfos.c) lock in the export behavior. I checked in your changes in r48170 along with a workaround to reset the export definitions. It's clunkier than I would like but Visual C++ is happier now. On 3/7/13 8:23

Re: [Wireshark-dev] Editcap link warnings

2013-03-06 Thread Christopher Maynard
Pascal Quantin pascal.quantin@... writes: Hi all,I added this workaround a few days ago in r48021 but Gerald reverted it in r48072 as it was breaking getopt(). So we have to figure out how to solve this properly. OK, I missed that. How about something like this instead? Index: Makefile.common

[Wireshark-dev] Editcap link warnings

2013-03-05 Thread Bill Meier
Recent Windows 7 and Windows XP Buildbot builds have been giving locally defined symbol ... imported in ... warnings while linking editcap: (I get the same warnings on my system). From the Window 7 Buildbot log: Linking editcap.exe link

Re: [Wireshark-dev] Editcap link warnings

2013-03-05 Thread Christopher Maynard
Bill Meier wmeier@... writes: Recent Windows 7 and Windows XP Buildbot builds have been giving locally defined symbol ... imported in ... warnings while linking editcap: (I get the same warnings on my system). Me too. Not sure if this is the right way to fix it, but this seems to work:

Re: [Wireshark-dev] Editcap link warnings

2013-03-05 Thread Pascal Quantin
2013/3/6 Christopher Maynard christopher.mayn...@gtech.com Bill Meier wmeier@... writes: Recent Windows 7 and Windows XP Buildbot builds have been giving locally defined symbol ... imported in ... warnings while linking editcap: (I get the same warnings on my system). Me too. Not