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

[build_base] replace gcc to clang #7

Closed
dna2github opened this issue Aug 2, 2017 · 4 comments
Closed

[build_base] replace gcc to clang #7

dna2github opened this issue Aug 2, 2017 · 4 comments
Assignees
@dna2github
Labels
enhancement

Comments

@dna2github
Copy link
Owner

since ndk will drop gcc and use clang, need to spend some time to switch build script to use clang.
@dna2github dna2github self-assigned this Aug 2, 2017
@aschrijver
Copy link
Contributor

Is this what I am referring to here for Node compile: nodejs/node#14771 ?

@dna2github
Copy link
Owner Author

No, it is for https://developer.android.com/ndk/downloads/revision_history.html
GCC is no longer supported. thus need to rebuild nodeJS with clang and make sure it pass.

@aschrijver
Copy link
Contributor

Ah okay.

BTW J2V8 @drywolf has a beautiful build system in the works, that I have been test-driving yesterday: eclipsesource/J2V8#327

Here are my compilation results: nodejs/node#14783
And you probably had seen this already: https://stackoverflow.com/a/45649995/8295283

@dna2github
Copy link
Owner Author

built node-v10.10.0 with clang using modified version of android-configure; the configure shell script is provided by node source code. tested on Android 6; works! will get script checked-in soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dna2github dna2github

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@dna2github @aschrijver