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] Windows build broken

From: Guy Harris <guy@xxxxxxxxxxxx>
Date: Sat, 9 Feb 2013 15:43:36 -0800
On Feb 9, 2013, at 3:16 PM, mmann78@xxxxxxxxxxxx wrote:

> I can build, but I can't use the VS2010 step-through debugger.  The usual cause of this is VS thinking its source is out of sync (like when I forget to recompile a dissector change, even if its just a comment change).  Based on recent developments, I suspect the problem is this:
>  
>         ..\tools\lemon\lemon t=..\tools\lemon\lempar.c dtd_grammar.lemon
>         cl  /DPCAP_VERSION=4_1_2 /Zi /W3 /MD /DWIN32_LEAN_AND_MEAN /DMSC_VER_REQ
> UIRED=1600  /D_CRT_SECURE_NO_DEPRECATE /D_CRT_NONSTDC_NO_DEPRECATE /D_BIND_TO_CU
> RRENT_CRT_VERSION=1 /MP /GS /w34295  /I. /I.. /I../wiretap /IC:\Wireshark-win32-
> libs\gtk2\include\glib-2.0  /IC:\Wireshark-win32-libs\gtk2\lib\glib-2.0\include
>  -DG_DISABLE_DEPRECATED  -DG_DISABLE_SINGLE_INCLUDES  /IC:\Wireshark-win32-libs\
> zlib125\include /IC:\Wireshark-win32-libs\c-ares-1.7.1-win32ws/include   /IC:\Wi
> reshark-win32-libs\lua5.1.4\include /IC:\Wireshark-win32-libs\gnutls-2.12.18-1.2
> -win32ws\include /DNOCRYPT /DIMPORT_LIGNUTLSDLL /IC:\Wireshark-win32-libs\WPdpac
> k\include   /IC:\Wireshark-win32-libs\libsmi-svn-40773-win32ws\include /IC:\Wire
> shark-win32-libs\GeoIP-1.4.8-2-win32ws/include -Fd.\ -c dtd_grammar.h dtd_parse.c
> Microsoft (R) 32-bit C/C++ Optimizing Compiler Version 16.00.40219.01 for 80x86
> Copyright (C) Microsoft Corporation.  All rights reserved.
> cl : Command line warning D9024 : unrecognized source file type 'dtd_grammar.h', object file assumed
> cl : Command line warning D9027 : source file 'dtd_grammar.h' ignored dtd_parse.c
>  
> It seems in line with the other changes originally necessary to build the plugins that used lex.

Actually, that's an issue with Lemon-generated parsers using Flex-generated lexical analyzers.  I've checked in a change that matches what was done for the MATE plugin; see the comment added as part of that.