[ARROW-4462] [C++] Upgrade LZ4 v1.7.5 to v1.8.3 to compile with VS2017

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 0.13.0
• Component/s: C++, Developer Tools
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/21019
• Assignee: Areg Melik-Adamyan
• Reporter: Areg Melik-Adamyan
• Created: 03/Feb/19 08:10
• Updated: 11/Jan/23 07:34
• Resolved: 09/Feb/19 19:34

Description

By upgrading to LZ4 v1.8.3 the patch and patching step can be removed as it is incorporated into a newer version of VS2010 solution and also VS2017 solution is provided which ease the usage with newer versions. Is there a reason or fixed dependency on v1.7.5?

There is still an issue with newer than v8.1 MS Build Tools, and requires manual retargeting. Which can be fixed in CMake by introducing complex logic of reading registry tree HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\MSBuild\ToolsVersions\4.0\1*.0 and analyzing which version of tools are installed and then patching the solution and projects. But as this is an external dependency better to submit patch there.

Issue Links

• links to: GitHub Pull Request #3585