Fix namespace pollution on Windows

As described in #116, the `io.h` file in this project conflicts with `io.h` of windows.

This fixes that (in the somewhat narrow compilation path that i'm exercising) by removing `include/substrait/common` from the include path, and instead relying on the (already existing) full include-path namespacing instead of relative includes of `io.h`.

CMakeLists.txt

@@ -29,6 +29,8 @@ option(
  "Enable substrait-cpp tests. This will enable all other build options automatically."
  ON)
 
+set(SUBSTRAIT_CPP_INCLUDE_DIR ${CMAKE_CURRENT_SOURCE_DIR}/include)
+
 # Local files come first.
 include_directories(include)
 include_directories(src)

src/substrait/common/CMakeLists.txt

@@ -21,8 +21,8 @@ add_dependencies(
 target_include_directories(
  substrait_io
  INTERFACE
- $<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/../../../include/substrait/common>
- $<INSTALL_INTERFACE:include/substrait/common>)
+ $<BUILD_INTERFACE:${SUBSTRAIT_CPP_INCLUDE_DIR}>
+ $<INSTALL_INTERFACE:include>)
 target_link_libraries(substrait_io substrait_proto substrait_textplan_converter
  substrait_textplan_loader absl::status absl::statusor)