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


_______________________________________________
Spice-devel mailing list
Spice-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/spice-devel

Reply via email to