[C++] Do not transitively depend on libboost_system

[asfimport]

We picked up this dependency recently. I don't think this is a blocker for 0.7.0, but it impacts static linkers (e.g. linkers of parquet-cpp)

This was introduced in ARROW-1339 94b7cfa

cc @majetideepak

Reporter: Wes McKinney / @wesm
Assignee: Deepak Majeti / @majetideepak

Related issues:

• [CPP] Remove libboost_system dependency (is depended upon by)

PRs and other links:

• GitHub Pull Request #1105

_{Note: This issue was originally created as ARROW-1536. Please see the migration documentation for further details.}

[asfimport]

Wes McKinney / @wesm:
@majetideepak I took a quick look at this. boost::system symbols are getting pulled in as a transitive dependency to boost::filesystem. We need boost::filesystem for handling unicode file paths on Windows. Trying to conditionally use boost::filesystem only on Windows would be a lot of work without a great deal of benefit.

One solution to this problem would be to add an option to vendor the requisite Boost bits in parquet-cpp, like we've done in Arrow

https://github.com/apache/arrow/blob/master/cpp/CMakeLists.txt#L129

https://github.com/apache/arrow/blob/master/cpp/cmake_modules/ThirdpartyToolchain.cmake#L139

[asfimport]

Deepak Majeti / @majetideepak:
@wesm I made an attempt here #1105. Please let me know what you think. Thanks!

[asfimport]

Wes McKinney / @wesm:
Issue resolved by pull request 1105
#1105