-
Notifications
You must be signed in to change notification settings - Fork 6
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
Optionally use Clang compiler #9
Comments
Already using clang here on OS X. Works just fine. |
Are you using |
Mac OS X aliases |
Yes, it is, just wanted to be sure :-) Anyway it's a huge effort, specially the Travis part! :-D |
If you are in th mood, go for it ;-) Have you tried to compile Node.js with
|
Node.js throws clang specific arguments at CXX and CC when trying to compile for OS X so GCC isn't really an option unless I can override those args or just get GCC to ignore them. I can compile for Linux on OS X, which works for a while before I get an internal compiler error. |
The compiler error, seems to be resulting from an invalid file path though, but I'm not really sure where the path is coming from.
I believe the problem is with the last two commands. Take notice of the |
Maybe is it time to create a pull-request to allow to compile Node.js on
That's a shame, because that's just what we need for NodeOS, to compile it |
Can you post the full backtrace?
|
|
Not a lot of help, but the rest of the compile works fine. Not a memory issue either (Monitoring it the entire build). I can only assume it's the broken file path. |
It could be, but I've only get internal errors on cc1 due to out of memory |
8GB of RAM & around 4-5GB free upon fresh reboot.
|
Sh*t, then I don't know what happens here... Not-capitalized filesystem?
|
I'm compiling node.js on my case sensitive file system. It's probably just the broken file path. |
You also have to remember, I'm compiling Linux on OS X so weird errors are to be expected. |
Regarding to this point, are you sure you are using only the include files |
I'll double check |
Continuing discussion from NodeOS/NodeOS#267 (comment)
Starting @piranna and @luii
The text was updated successfully, but these errors were encountered: