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.

Sign up for GitHub

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

Jump to bottom

make-standalone-toolchain.sh doesn't work with Cygwin #41

Closed
azharudd opened this issue Mar 23, 2016 · 2 comments
Closed

make-standalone-toolchain.sh doesn't work with Cygwin #41

azharudd opened this issue Mar 23, 2016 · 2 comments
Assignees
@DanAlbert
Milestone
r12

Comments

@azharudd
Copy link

The variable SYSTEM gets computed from "$HOST_OS-$HOST_ARCH" as "cygwin-x86_64" or cygwin-x86" and no such dir exists in toolchains/arm-linux-androideabi-4.9/prebuilt/ (obviously, windows or windows-x86_64 will be present)

Earlier, this script supported the option "--system=", which could be set to select the host and this wasn't an issue. This option is now removed causing it to error out.

Any pointers on this ?
@DanAlbert
Copy link
Member

I'll add a check for cygwin and rewrite to windows, that way you won't need --system anyway.

I don't have the ability to test this easily, so when I have a patch I'll post it here. It would be really helpful if you could test it for me.

@DanAlbert DanAlbert self-assigned this Mar 23, 2016
@DanAlbert DanAlbert added this to the r12 milestone Mar 23, 2016
@DanAlbert
Copy link
Member

Fix looks trivial. Give it a shot and let me know: https://android-review.googlesource.com/209970

@gohome gohome mentioned this issue May 5, 2016
Closed
miodragdinic pushed a commit to MIPS/ndk that referenced this issue Jan 12, 2017
@DanAlbert
Add support for cygwin64.
b8cf1e7 
We have the catch for cygwin-x86, but were missing one for
cygwin-x86_64.

Bug: android/ndk#41
Change-Id: I9af457b4854ee7aebc33618a99f6294441da0f14
@dan-ryan dan-ryan mentioned this issue Sep 11, 2018
Closed
@qwandor-google qwandor-google mentioned this issue Apr 3, 2020
Closed
@AlexanderSerov AlexanderSerov mentioned this issue Jun 7, 2021
Closed
@raju535482 raju535482 mentioned this issue Nov 28, 2022
Closed
@seoil0728 seoil0728 mentioned this issue Nov 29, 2023
Closed
@yujincheng08 yujincheng08 mentioned this issue Apr 11, 2024
Closed
@sruPL sruPL mentioned this issue Apr 30, 2024
Closed
@notyalcs notyalcs mentioned this issue May 24, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@DanAlbert DanAlbert

Labels
None yet
Projects
None yet
Milestone
r12
Development

No branches or pull requests
2 participants
@DanAlbert @azharudd