qpid-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Chug Rolke" <cro...@redhat.com>
Subject Re: Review Request 14443: Build Amqp1.0 support into QpidMessaging for all platforms
Date Wed, 02 Oct 2013 19:45:32 GMT

This is an automatically generated e-mail. To reply, visit:

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

Review request for qpid and Andrew Stitcher.


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

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

Bugs: QPID-5115

Repository: qpid


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/


Windows and Linux clients may create Amqp 0.10 and 1.0 connections without setting any environment
make test passes all tests.


Chug Rolke

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message