-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Disable x86 (32-bit) Linux builds or issue a warning #4091
Comments
@fatcerberus no short/mid term x86 support on *nix. We might add a message warning the user for |
Oh, it's not supported? That's fine then, I thought it was because the
Implying it's supported. I agree there should be a warning for it, though. |
@fatcerberus would you like to send a PR? |
@obastemur I would, but what I’m confused about is that build.sh is only for Unix platforms; is x86 relevant to any of them? It seems to only be relevant for Windows nowadays, where build.sh doesn’t apply. In light of all that, it seems like the --arch=x86 option shouldn’t even exist? |
At the moment, yes, there is nothing can keep end-user from using it or there is no even warning. could be nice to have a warning message ? and we could close this issue with that? :) |
Yeah, what I was trying to figure out was whether to make a PR that adds the warning, or if I should just completely remove x86 from the list of architectures on the |
Simply adding a warning is enough IMHO. |
The errors have nothing to do with x86 support per se, |
As an extra note, x86 has never been supported on linux - so if it does actually build after the flags are fixed there will likely be some issues to iron out - I think there are few places in the source tree where there's architecture specific stuff that has x86 gated behind #ifdef windows or #ifdef win32 or the like. This is purely anecdotal I haven't gone through the source tree looking for these (I also don't have a linux machine so cannot easily experiment with this - whilst the linux and macOS builds are similar apple have removed support for running 32bit code from macOS, so I cannot run 32bit binaries on my machine). |
Tried to make a 32-bit ChakraCore build for Linux (Ubuntu), but the build failed. I was able to build for x64 without a hitch, but x86 proved futile.
Command line:
Gave me the following output:
The text was updated successfully, but these errors were encountered: