[Spice-devel] Problems compiling SPICE Windows client SDK 7.1

[John A. Sullivan III]

On Wed, 2011-07-06 at 13:23 +0300, Arnon Gilboa wrote:
> John A. Sullivan III wrote:
> > <snip>
> > We've made considerable progress.  We do not have VS2008 as we are not a
> > Windows shop and I assume in my Windows ignorance that that's a paid
> > licensed product so we'd have track which of us could work on it and
> > which couldn't so we've tried to make due with just the SDK.  That's
> > actually working well.
> >
> > The syntax problems seem to be because we had downloaded python 3.2.  We
> > removed that and installed Python 2.7 along with pyparsing.  It then
> > complained it could not find spice/types.h.  I will pass that along in
> > that it smells like a bug.  The instructions say to set
> > SPICE_PROTOCOL_DIR but redc.vcproj never references it.  Rather than
> >   
> SPICE_PROTOCOL_DIR is referenced in redc.vcproj in the lines:
> AdditionalIncludeDirectories=".;..;$(SPICE_PROTOCOL_DIR)...
> Sure you have set it to the full path of spice-protocol? close & open 
> your build env after that.
> > fiddle with the code or project file, I simply moved the protocol
> > directory to where it could find it in the include paths I did see in
> > redc.vcproj.
That's strange; it's not in mine:
jsullivan at jaseee:~/packaging/spice-sources/spice-0.8.1/client/windows$ ls redc.vcproj
redc.vcproj
jsullivan at jaseee:~/packaging/spice-sources/spice-0.8.1/client/windows$ grep SPICE_PROTOCOL redc.vcproj
jsullivan at jaseee:~/packaging/spice-sources/spice-0.8.1/client/windows$


> >
> > That worked a treat.  144 warnings but only one error:
> >
> >   c:\users\administrator\downloads\spice\spice-0.8.1\client\../common/marshaller.c(499): 
> > warning C4267: '=' : conversion from 'size_t' to 'uint32_t', 
> > possible loss of data [C:\Users\Administrator\Downloads\spice\spice-0.8.1\client\windows\redc.vcxproj]
> >
> >
> > "C:\Users\Administrator\Downloads\spice\spice-0.8.1\client\windows\redc.vcxproj" 
> > (default target) (1) ->(ClCompile target) ->  ..\application.cpp(2300): error C2065:
> >  'PACKAGE_VERSION' : undeclared identifier [C:\Users\Administrator\Downloads\spice\spice-0.8.1\client\windows\redc.vcxproj]
> >
> >   
> This one, which broke windows build, was patched after 0.8.1, see
> http://cgit.freedesktop.org/spice/spice/log/?h=0.8
> http://cgit.freedesktop.org/spice/spice/commit/?h=0.8&id=31358b640be13c3b0955fc9acc1ad9526cc280ec
<snip>
Yaniv pointed out that there is a freely available version of VS2008 so
we can go that route but would it be helpful to the community if we
persisted with SDK 7.1 and documented how or is that asking for trouble?
Thanks - John