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] Status Cmake Win32 support

From: Joerg Mayer <jmayer@xxxxxxxxx>
Date: Sun, 13 Oct 2013 00:22:34 +0200
On Sat, Oct 12, 2013 at 07:58:45PM +0100, Graham Bloice wrote:
> >
> > - Are there other platforms that would benefit from something similar to
> >   what I did with the setenv.bat script?
> 
> 
> The setenv.bat (I hope that is the file you're referring to) doesn't work
> with a MSVC2010 solution, as the paths to the libraries directory is wrong,
> it has ...\builddir\lib, it should be ...\builddir\Debug\lib, and the bin
> directory is wrong, it has \msvc2010\bin, it should be
> ...\builddir\Debug\bin.  note that the Debug portion of these paths will
> vary depending on the type of build done, Debug is the default, there are
> also; MinSizeRel, Release and RelWithDebInfo.  I think there will be a
> CMake variable for that somewhere.

My whole setup has only been tested with '-G "NMake Makefiles"' in that
environment it's using \lib and \bin directly. I'd be interested to know
where that \Debug in the path comes from. cmake has something that I have
not used so far, which is called CONFIGURATIONS. But unless configurations
are used I don't know where that Debug is supposed to come from. Do you
experience this only with the VS generator or with the nmake generator as
well?
Once I know where this comes from I have no problem of adding \${whatever}
to the paths.
Which reminds me: Configurations are something we should also add to the
build at some time ;)

> I've also noted that sometimes the linker has a malformed option on it:
> '/LARGEADDRESSAWARE;setargv.obj'.

Fixed in revision 52574 by removing both options. None seems to make a
difference on my system.

Ciao
        Jörg

-- 
Joerg Mayer                                           <jmayer@xxxxxxxxx>
We are stuck with technology when what we really want is just stuff that
works. Some say that should read Microsoft instead of technology.