lidavidm commented on a change in pull request #11157:
URL: https://github.com/apache/arrow/pull/11157#discussion_r708758482



##########
File path: cpp/build-support/build-lz4-lib.sh
##########
@@ -17,9 +17,14 @@
 # specific language governing permissions and limitations
 # under the License.
 #
+MAKE="make"
+if [ -n "$MSYSTEM" ]; then
+  MAKE="mingw32-make"
+fi

Review comment:
       I found that the mingw-w64-x86_64-make package doesn't install `make` 
but rather `mingw32-make`, unfortunately. (And from what I understand, the 
msys2 make package shouldn't be mixed with the mingw toolchain we use here.)




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to