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

[Arnon Gilboa]

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 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
>     144 Warning(s)
>     1 Error(s)
>
> Time Elapsed 00:01:31.26
>
> That comes from:
>         case SPICE_OPT_VERSION: {
>             std::ostringstream os;
>             os << argv[0] << " "<< PACKAGE_VERSION << std::endl;
>             Platform::term_printf(os.str().c_str());
>             return false;
>         }
>
> That seems to pull from :
> windows/Makefile.in:PACKAGE_VERSION = @PACKAGE_VERSION@
>
> Not being a developer either (as you can probably tell, I'm a pretty
> useless management type :)  ) I don't know why this is not being pulled
> in - perhaps because I am directly building the vcxproj file rather than
> using the sln file although I do not see where that would be in the .sln
> file.
>
> This looks like our last hurdle and it does look like there is a real
> bug in the redc.vcproj file (or in the documentation depending on how
> you look at it!).  Any suggestions on how to fix this last problem?
> Thanks - John
>
>