-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14443/
-----------------------------------------------------------

(Updated Oct. 2, 2013, 7:45 p.m.)


Review request for qpid and Andrew Stitcher.


Changes
-------

Good point - if proton was found then it would be linked in even if amqp1.0 
support was turned off.

This patch nullifies the proton variables if BUILD_AMQP is false and 
qpidmessaging will completely ignore proton.


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 (updated)
-----

  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

Reply via email to