Re: [Wireshark-commits] rev 45191: /trunk/ /trunk/epan/dissectors/: packet-btsap.c /trunk/: CMakeLists.txt

[Joerg Mayer <jmayer () loplof de>]

On Fri, Sep 28, 2012 at 04:29:14PM +0200, Jeff Morriss wrote:
> >  Get rid of HAVE_CONFIG_H for cmake. Fix one case where it was still used.
> [...]
> >   +0 -1      CMakeLists.txt    Modified
>
> I had been assuming that we needed to continue to define HAVE_CONFIG_H
> for backwards compatibility (e.g., for custom dissectors).  I guess
> any old dissector could/should just take it out though.  Anyway I had
> (intentionally) not removed the define (so it's still there in nmake
> and autofoo too).

Well, that way we found a missing cleanup in the main source ;-)
I'm all for removing it from the other build mothods as well, but
I don't care either way.

Ciao
     Jörg
-- 
Joerg Mayer                                           <jmayer () loplof de>
We are stuck with technology when what we really want is just stuff that
works. Some say that should read Microsoft instead of technology.
___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev () wireshark org>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-request () wireshark org?subject=unsubscribe