Wireshark-dev: Re: [Wireshark-dev] Trunk Windows CMake builds are broken
From: Roland Knall <[email protected]>
Date: Thu, 21 Aug 2014 12:19:24 +0200

The #include "config.h" statement is a mistake, it should refer to #include <config.h>

I've committed a change to https://code.wireshark.org/review/#/c/3763/

It builds find on my Linux and Mac (using GLib 2.36).

Also looking at the Mac buildbot (GLib < 2.34 is the issue) right now, and maybe have a solution for that in the afternoon. 

regards,
Roland


On Thu, Aug 21, 2014 at 11:31 AM, Graham Bloice <[email protected]> wrote:
On 21 August 2014 10:04, Guy Harris <[email protected]> wrote:

On Aug 21, 2014, at 1:57 AM, Graham Bloice <[email protected]> wrote:

> This obviously worked before the merge of ExtCap, so that has disturbed things.  The CMake build should be using the generated one in the CMake build directory, i.e. C:\buildbot\wireshark\wireshark-master-64\win7x64\build\cmbuild\config.h, but the nmake build copy is getting in the mix.

I don't think that building using build system A and then, without first cleaning up all the generated files from build system A, building using build system B is guaranteed to work.  (I've had problems with, for example, an in-tree autotools build followed by an out-of-tree CMake build - *before* the extcap stuff was checked in, so it wasn't extcap's fault.)

We should either do a separate checkout for the Windows CMake build or should do an "nmake -f Makefile.nmake maintainer-clean" before the CMake build.


Unfortunately the Windows CMake build currently requires the zlib dll built by nmake, so until that gets fixed the Windows CMake can't be built from a totally clean system (or could be but without zlib support).

In general though, for out-of-tree builds, it would seem that using the quoted form of #include is a little dangerous, I've always had the idea that angle bracket form was for system or platform includes and quoted form for project includes, but maybe I was just confused.

--
Graham Bloice

___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <[email protected]>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
             mailto:[email protected]?subject=unsubscribe