Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Invalid toolchain path for Windows x86 #45

Closed
seongsoosim opened this issue Mar 24, 2016 · 2 comments

Comments

Projects
None yet
2 participants
@seongsoosim
Copy link

commented Mar 24, 2016

The function of 'host-toolchain-path' defined in 'init.mk' uses $(HOST_TAG64) variable.
And this value represents as "windows-x86" on Windows 32 bit OS.
The problem is that all the toolchain doesn't have 'windows-x86' directory in android-ndk-r11b-windows-x86.zip package. It has only 'windows' directory.
So, it make a compile failure.

When I changed the directory name from "\toolchains\aarch64-linux-android-4.9\prebuilt\windows" to " "\toolchains\aarch64-linux-android-4.9\prebuilt\windows-x86", then the building was successful.

@DanAlbert DanAlbert self-assigned this Mar 24, 2016

@DanAlbert DanAlbert modified the milestones: r12, r11c Mar 24, 2016

@DanAlbert

This comment has been minimized.

@DanAlbert DanAlbert closed this Mar 25, 2016

@DanAlbert

This comment has been minimized.

Copy link
Member

commented Mar 29, 2016

My patch is no good. Will have another once my machine isn't bogged down running the test suite.

@DanAlbert DanAlbert reopened this Mar 29, 2016

@DanAlbert DanAlbert closed this Mar 31, 2016

DanAlbert added a commit that referenced this issue Apr 16, 2016

Fix the 32-bit Windows host tag.
Bug: #45
Change-Id: I4d7f40d6f5609b0067cd89d2b2b75f1195c360bf
(cherry picked from commit 3f3b154)

DanAlbert added a commit that referenced this issue Apr 16, 2016

Fix the check for the 32-bit Windows NDK.
Need to check HOST_ARCH64, not HOST_TAG64.

Bug: #45
Change-Id: Ic54cd186b7931957c02f21af4314a7068dfbbf3c
(cherry picked from commit 65a3f40)

DanAlbert added a commit that referenced this issue Apr 16, 2016

Fix the 32-bit Windows host tag.
Bug: #45
Change-Id: I4d7f40d6f5609b0067cd89d2b2b75f1195c360bf

DanAlbert added a commit that referenced this issue Apr 16, 2016

Fix the check for the 32-bit Windows NDK.
Need to check HOST_ARCH64, not HOST_TAG64.

Bug: #45
Change-Id: Ic54cd186b7931957c02f21af4314a7068dfbbf3c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.