Avro
  1. Avro
  2. AVRO-952

C++/CMake build does not detect flex & bison correctly

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Won't Fix
    • Affects Version/s: 1.6.0
    • Fix Version/s: None
    • Component/s: c++
    • Labels:

      Description

      Configure should test for flex and bison. CMake has built in support for those, we should use that. I've verified that it existed at 2.6, which is the minimum CMake release that we currently require.

      1. AVRO-952.patch
        2 kB
        Nebojsa Sabovic

        Activity

        Nebojsa Sabovic created issue -
        Hide
        Nebojsa Sabovic added a comment -

        Fix

        Show
        Nebojsa Sabovic added a comment - Fix
        Nebojsa Sabovic made changes -
        Field Original Value New Value
        Attachment AVRO-952.patch [ 12502502 ]
        Nebojsa Sabovic made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Hide
        Thiruvalluvan M. G. added a comment -

        This is a valid patch. However, I'm almost done with porting Avro C++ to Miscrosoft Windows. CMake + Windows + Flex/Bison appear to be a problem combination. Moreover, the Schema parser based on Flex/Bison is not complete and produces no good error messages. I've implemented a new parser on top of Json parser we already have in Avro C++. We can then implement full compliance to Avro standard including namespaces, docs etc.

        I'll file 2 Jiras one for getting rid of dependence on flex/bison and BoostRegex and another for porting to Microsoft Windows. With those patches we don't have to fix the Flex/Bison find problems.

        Show
        Thiruvalluvan M. G. added a comment - This is a valid patch. However, I'm almost done with porting Avro C++ to Miscrosoft Windows. CMake + Windows + Flex/Bison appear to be a problem combination. Moreover, the Schema parser based on Flex/Bison is not complete and produces no good error messages. I've implemented a new parser on top of Json parser we already have in Avro C++. We can then implement full compliance to Avro standard including namespaces, docs etc. I'll file 2 Jiras one for getting rid of dependence on flex/bison and BoostRegex and another for porting to Microsoft Windows. With those patches we don't have to fix the Flex/Bison find problems.
        Hide
        Thiruvalluvan M. G. added a comment -

        The C++ port no longer uses Flex or Bison and hence this problem no longer exists.

        Thank you Nebojsa Sabovic for looking into this issue.

        Show
        Thiruvalluvan M. G. added a comment - The C++ port no longer uses Flex or Bison and hence this problem no longer exists. Thank you Nebojsa Sabovic for looking into this issue.
        Thiruvalluvan M. G. made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Resolution Won't Fix [ 2 ]
        Doug Cutting made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Patch Available Patch Available
        1m 28s 1 Nebojsa Sabovic 04/Nov/11 19:00
        Patch Available Patch Available Resolved Resolved
        170d 9h 15m 1 Thiruvalluvan M. G. 23/Apr/12 05:16
        Resolved Resolved Closed Closed
        155d 17h 6m 1 Doug Cutting 25/Sep/12 22:22

          People

          • Assignee:
            Unassigned
            Reporter:
            Nebojsa Sabovic
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development