Skip to content

Commit

Permalink
Fix the check for the 32-bit Windows NDK.
Browse files Browse the repository at this point in the history
Need to check HOST_ARCH64, not HOST_TAG64.

Bug: #45
Change-Id: Ic54cd186b7931957c02f21af4314a7068dfbbf3c
  • Loading branch information
DanAlbert committed Mar 30, 2016
1 parent e205a50 commit 65a3f40
Showing 1 changed file with 5 additions and 7 deletions.
12 changes: 5 additions & 7 deletions build/core/init.mk
Original file line number Diff line number Diff line change
Expand Up @@ -284,11 +284,9 @@ ifeq ($(HOST_OS),windows)
HOST_EXEEXT := .exe
endif

# If we are on Windows, we need to check that we are not running
# Cygwin 1.5, which is deprecated and won't run our toolchain
# binaries properly.
#
ifeq ($(HOST_TAG),windows-x86)
# If we are on Windows, we need to check that we are not running Cygwin 1.5,
# which is deprecated and won't run our toolchain binaries properly.
ifeq ($(HOST_OS),cygwin)
# On cygwin, 'uname -r' returns something like 1.5.23(0.225/5/3)
# We recognize 1.5. as the prefix to look for then.
Expand All @@ -303,9 +301,9 @@ ifeq ($(HOST_TAG),windows-x86)
# special-case the host-tag
HOST_TAG := windows

# For 32-bit systems, HOST_TAG == HOST_TAG64. Make sure that remains true
# here.
ifeq ($(HOST_TAG64),x86)
# For 32-bit systems, HOST_TAG64 should be HOST_TAG, but we just updated
# HOST_TAG, so update HOST_TAG64 to match.
ifeq ($(HOST_ARCH64),x86)
HOST_TAG64 = $(HOST_TAG)
endif
endif
Expand Down

0 comments on commit 65a3f40

Please sign in to comment.