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

Getting rid of the usage of android ndk in rustc::driver #5915

Closed
ILyoan opened this issue Apr 17, 2013 · 5 comments
Closed

Getting rid of the usage of android ndk in rustc::driver #5915

ILyoan opened this issue Apr 17, 2013 · 5 comments
Labels
C-cleanup Category: PRs that clean code up or issues documenting cleanup. E-easy Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue. O-android Operating system: Android

Comments

@ILyoan
Copy link
Contributor

ILyoan commented Apr 17, 2013

Currently, there is an android hack on rustc:driver to compile binaries through Android NDK due to lack of ARM unwind supporting of LLVM.
Now LLVM ARM unwind supporting is in progress here. Once it is landed, we can remove the hack.
@ILyoan
Copy link
Contributor Author

ILyoan commented Apr 18, 2013

The patch for LLVM ARM unwind supporting is landed by r179591

Now Android NDK can be removed from rustc::driver but this involves upgrade of LLVM.

@graydon
Copy link
Contributor

graydon commented May 2, 2013

declined for milestone; this is just a cleanup bug

@pnkfelix
Copy link
Member

see also #5903

@pnkfelix
Copy link
Member

also LLVM upgrade bug #7115 seems related, in light of ILyoan's comment

@ILyoan
Copy link
Contributor Author

ILyoan commented Aug 29, 2013

Dup of #8809, Closing this because #8809 has more recent information.

@ILyoan ILyoan closed this as completed Aug 29, 2013
flip1995 pushed a commit to flip1995/rust that referenced this issue Aug 28, 2020
@bors
Auto merge of rust-lang#5915 - matthiaskrgr:deps_, r=Manishearth
838c201 
deps: bump cargo_metadata and semver

cargo_metadata 0.9.1 -> 0.11.1
semver 0.9.0 -> 0.10.0

changelog: none
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C-cleanup Category: PRs that clean code up or issues documenting cleanup. E-easy Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue. O-android Operating system: Android
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@graydon @pnkfelix @ILyoan