----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14443/#review26646 -----------------------------------------------------------
There are some test scripts that will likely need to be altered also as they test for and try to load the amqpc module (swig python tests, interlink tests). - Gordon Sim On Oct. 2, 2013, 7:59 p.m., Chug Rolke wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/14443/ > ----------------------------------------------------------- > > (Updated Oct. 2, 2013, 7:59 p.m.) > > > Review request for qpid and Andrew Stitcher. > > > Bugs: QPID-5115 > https://issues.apache.org/jira/browse/QPID-5115 > > > Repository: qpid > > > Description > ------- > > This patch removes the amqpc library and builds Amqp1.0 support into > qpidmessaging. Amqp1.0 support is immediately available to clients without > loading extra modules. > > Amqp1.0 support for clients is controlled by setting the {protocol:amqp1.0} > connection option. > Amqp1.0 support for the broker is still isolated in the ampq library. The > broker may run with or without Ampq1.0 support by loading or not loading the > amqp library. > > > Diffs > ----- > > trunk/qpid/cpp/bld-winsdk.ps1 1528551 > trunk/qpid/cpp/src/CMakeLists.txt 1528551 > trunk/qpid/cpp/src/amqp.cmake 1528551 > > Diff: https://reviews.apache.org/r/14443/diff/ > > > Testing > ------- > > Windows and Linux clients may create Amqp 0.10 and 1.0 connections without > setting any environment variables. > make test passes all tests. > > > Thanks, > > Chug Rolke > >
