Uploaded image for project: 'Log4cxx'
  1. Log4cxx
  2. LOGCXX-21

Add check that libxml2 not libxml has been included

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 0.9.7
    • Fix Version/s: 0.10.0
    • Component/s: Build
    • Labels:
      None

      Description

      gnomexml.cpp is written expecting libxml2. If libxml2 is present but libxml is placed before it on the include path, the code will fail to compile but the reason is not clear. This occurred to me setting up an Ant build since when I was trying to resolve unsatisified includes in gnomexml.cpp I placed /usr/include/gnome-xml on the include path (seemed reasonable at the time) not /usr/include/libxml2. The fix was to check the value of LIBXML_VERSION in gnomexml.h and raise a compile error if the version number is less than 20000. The surround comments describe the issue and its resolution.

        Attachments

        1. gnomexml.patch
          0.9 kB
          Curt Arnold

          Activity

            People

            • Assignee:
              carnold@apache.org Curt Arnold
              Reporter:
              carnold@apache.org Curt Arnold
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: