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

libuv has it's target_arch specified incorrectly #8826

Closed
alexcrichton opened this issue Aug 28, 2013 · 9 comments
Closed

libuv has it's target_arch specified incorrectly #8826

alexcrichton opened this issue Aug 28, 2013 · 9 comments
Assignees

Comments

@alexcrichton
Copy link
Member

Right now when you're building a 64-bit build it may error because libuv is built as a 32-bit library. It looks like gyp doesn't understand x86_64, but rather x64

cc @pnkfelix who was having problems with this.

@ghost ghost assigned alexcrichton Aug 28, 2013
@alexcrichton
Copy link
Member Author

@pnkfelix, I believe that https://gist.github.com/alexcrichton/6369271 will solve the problem, but can you confirm before I open a pull request?

@pnkfelix
Copy link
Member

@alexcrichton Unfortunately the patch did not fix the problem for me.

But you have now given me some places to look for a fix, so I'll see if I can find a fix myself.

@pnkfelix
Copy link
Member

on a subsequent run (using -j1), it seems like the patch does work. I'm not sure why my earlier run did not work. Maybe there is interference when building with -jN for N > 1?)

bors added a commit that referenced this issue Aug 29, 2013
It turns out that gyp (libuv's new build system) wants x64 for a 64-bit x86
architecture and ia32 for a 32-bit architecture, so this performs the relevant
mapping and then invokes libuv's configure script with the appropriate target
architecture.

This can be verified by running make with VERBOSE=1 and seeing that beforehand
on a 64-bit build libuv was passed "-arch i386" and now it's passed
"-arch x86_64"

Closes #8826
@pnkfelix pnkfelix reopened this Aug 31, 2013
@pnkfelix
Copy link
Member

The fix for this got reverted (accidentally?) by SHA 3c5a43e causing the bug to be re-injected.

@alexcrichton
Copy link
Member Author

This is only a bug with our usage of gyp though, so is this still an issue?

The libuv upgrade was reverted because it was the most likely suspect of a log of segfaults on the bots

@pnkfelix
Copy link
Member

Well, it seemed like I was encountering the same problems in my builds tonight that had motivated this fix in the first place.

But I also did not know what the reason was for SHA 3c5a43e ; if we are indeed not using gyp anymore (as implied by @alexcrichton 's note, then I'm not sure what is going on. I'll investigate more, and close this if I cannot confirm that it is a real problem more concretely.

@pnkfelix
Copy link
Member

okay, never mind, the libuv is being built in x86_64 mode. Closing.

@pnkfelix
Copy link
Member

(the aforementioned reversion was PR #8866)

@pnkfelix
Copy link
Member

(the problem I was seeing was an artifact of #8865 ; sorry for the noise.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants