Avro
  1. Avro
  2. AVRO-1140

Buffer.hh includes Config.hh without "../"

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.7.1
    • Fix Version/s: 1.7.1
    • Component/s: c++
    • Labels:
      None

      Description

      Buffer.hh includes the file Config.hh, which is located in the parent directory. Config.hh is included like via #include "Config.hh", but this should be #include "../Config.hh". Both work when compiling the AVRO distribution, but if you install AVRO using make install, g++ complains that it can't find Config.hh, which is included via DataFile.hh in my case.

      1. AVRO-1140.patch
        0.4 kB
        Jan van der Lugt

        Activity

        Hide
        Thiruvalluvan M. G. added a comment -

        Verified that the fix works on Windows (Visual Studio Express 2010) and Cygwin

        Show
        Thiruvalluvan M. G. added a comment - Verified that the fix works on Windows (Visual Studio Express 2010) and Cygwin
        Hide
        Thiruvalluvan M. G. added a comment -

        Verified that the patch works on Mac.

        Committed revision 1374938. Thank you Jan van der Lugt.

        Show
        Thiruvalluvan M. G. added a comment - Verified that the patch works on Mac. Committed revision 1374938. Thank you Jan van der Lugt.
        Hide
        Jan van der Lugt added a comment -

        Thank you for the commit!

        Show
        Jan van der Lugt added a comment - Thank you for the commit!

          People

          • Assignee:
            Unassigned
            Reporter:
            Jan van der Lugt
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - 5m
              5m
              Remaining:
              Remaining Estimate - 5m
              5m
              Logged:
              Time Spent - Not Specified
              Not Specified

                Development