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

[John A. Sullivan III]

On Wed, 2011-07-06 at 07:43 -0400, John A. Sullivan III wrote:
> 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
<snip>
Well . . . some success and some failure.  Perhaps I am just mangling
this but there seems to be some inconsistency in the release sources for
Windows.  I did finally get it to build in VS2008 on XP.  But, I had to:

1) accommodate the non-use of SPICE_PROTOCOL_DIR as described above.
2) apply the patch to fixthe PACKAGE_VERSION problem mentioned above
3) apply the patch to backlevel freetype2312MT_D.lib
(http://patchwork.freedesktop.org/patch/4440/)
4) edit redc.rc to replace afxres.h with windows.h

Once I got it to build from spice-0.8.1, I was a little disappointed
that it appears to be the same binary as available for download site
(http://www.spice-space.org/download/binaries/spice-client-win32-0.6.3.zip) but labeled as 0.6.3!

I then thought I would take these changes and put them onto my Windows7
test box to compile a 64 bit client - the one I've was trying to do with
just the SDK and msbuild.  All went fine until almost the very end where
it generated an error about not being able to
find ..\generated_marshallers1.cpp.  There is
a ..\generated_marshallers.cpp but not 1.cpp.  What is causing that
problem?

Thanks - John