Loading a message with AMQP types

2012-11-01 Thread Raul Jimenez
Hello, I am trying to load a message with amqp types but am unsure on how to pack and unpack. If some one could point me towards the right direction I'd appreciate it. I have tried the following (messenger and message setup omitted): Note: I can send text without issue. server: int i=0; pn_data_t

RE: review request: patch to cmake for windows builds

2012-11-01 Thread Mary Hinton
I'm looking forward to the python cmake support for Visual Studio, especially for creating the _cproton library for python. I compared my qpid-proton dll project file for the one generated by cmake using the "Visual Studio 10" makefile selection. Ken's fix for the LIB64 on line 45 of CmakeLists.

Re: review request: patch to cmake for windows builds

2012-11-01 Thread Cliff Jansen
Search for "LEAN_AND_MEAN" and "ws2_32". I would drop all the warning flags and reintroduce if necessary. Presumably some are related to templates or other C++-isms that the proton code should be completely free of. On Thu, Nov 1, 2012 at 7:49 AM, Ken Giusti wrote: > Hi Cliff, > > I agree, but w

Re: review request: patch to cmake for windows builds

2012-11-01 Thread Rafael Schloming
FWIW, a lot of the patches that have been made to the build system so far were taken from the cpp broker's build and to be honest when you push for the reason they were done that way there is sometimes a good reason and sometimes no good reason, and sometimes a reason that doesn't apply to proton.

Re: review request: patch to cmake for windows builds

2012-11-01 Thread Ken Giusti
Hi Cliff, I agree, but where are those flags in the qpid source tree? Aside from the warning exclusion flags, that is (/wd). I've grepped about until blue in the fingers - about all I have found was: ./src/CMakeLists.txt:721:set (CMAKE_CXX_FLAGS_RELWITHDEBINFO "/MD /O2 /Ob2 /D NDEBUG")

Re: review request: patch to cmake for windows builds

2012-11-01 Thread Ken Giusti
I've created a project branch "kgiusti-msvc2010" on the apache site - no need to use github. Here is the current diff in all it's glory: https://reviews.apache.org/r/7826/ -K - Original Message - > On Wed, 2012-10-31 at 13:46 -0400, Ken Giusti wrote: > > Hi Cliff/Mary, > > > > I'm try