On 03/05/2010 09:52 PM, [email protected] wrote:
Author: jonathan
Date: Fri Mar  5 21:52:42 2010
New Revision: 919632

URL: http://svn.apache.org/viewvc?rev=919632&view=rev
Log:
Tests for presence of xqilla/ast/XQEffectiveBooleanValue.hpp; if not
present, sets a constant to enable legacy API for effective boolean
value.

Modified:
     qpid/trunk/qpid/cpp/configure.ac

Modified: qpid/trunk/qpid/cpp/configure.ac
URL: 
http://svn.apache.org/viewvc/qpid/trunk/qpid/cpp/configure.ac?rev=919632&r1=919631&r2=919632&view=diff
==============================================================================
--- qpid/trunk/qpid/cpp/configure.ac (original)
+++ qpid/trunk/qpid/cpp/configure.ac Fri Mar  5 21:52:42 2010
@@ -346,6 +346,8 @@

  test $want_xml = no&&  use_xml=no

+
+
  # If the user doesn't say not to use XML, see if it's available.
  if test $use_xml != no; then
    # Then see if XQilla is available
@@ -364,6 +366,13 @@
    # Else XQilla is available - use it to build
    test $use_xml = yes&&
      AC_DEFINE([HAVE_XML], [1], [Compile-in XML Exchange support.])
+
+  # Check to see if we need to use legacy calls for effective boolean value
+  xqilla_has_ebv=yes
+  AC_CHECK_HEADER([xqilla/ast/XQEffectiveBooleanValue.hpp], , 
[xqilla_has_ebv=no])
+  test $xqilla_has_ebv = no&&
+    AC_DEFINE([XQILLA_2_1_3], [1], [Use the old XQilla 2.1.3 API to get 
effective boolean value.])
+
  fi

This seems like either a poorly named define or the wrong test. (E.g. the header isn't in xqilla 2.0.0 either, which is currently supported).

Did you perhaps mean to set XQILLA_BEFORE_2_2_3 here? That is now checked for in XmlExchange.cpp but doesn't seem to be defined anywhere.

These changes also break the build where the new header file is not present.

---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project:      http://qpid.apache.org
Use/Interact: mailto:[email protected]

Reply via email to