-
Notifications
You must be signed in to change notification settings - Fork 7.3k
Release binaries are missing symbols #6801
Comments
FWIW, checkout out the v0.11.10 tag and running |
all of those are built on ancient centos 5.7 to accommodate people seeking equally ancient glibc versions |
Let me guess, really old gcc and binutils? What happens when you use a back-port of newer versions? For comparison, the v0.11.10 release binary has ~18.5k symbols while a local build has about ~72k, almost 4x as many. |
@tjfontaine does that mean the 0.10.24 binaries weren't built on the same machines as 0.11.10? Could there have been a change in V8 that is confusing the older toolchain about what symbols need to be preserved while optimizing? |
@rmg no they were built on the same machine and toolchain, most of the symbol differences I see above in ben's original post are related to the normal v8 api differences between v0.10 and master, aside from the single symbol that's actually in question I will look and see what other toolchains I have available via back ports |
I don't know if this is of any help, but apparently chromium stopped supporting RHEL6 after chromium 28 - https://code.google.com/p/chromium/issues/detail?id=227320 around april 2013. |
We're using gcc/g++ 4.4 from their backports, and have binutils220 as well, but that doesn't include a newer ld, so we're still using the ld from binutils 2.17. Seems that could be the actual culprit here as we're doing whole-archive now there's no reason for it to be dropping any symbols. |
@bnoordhuis @rmg I've installed gcc 4.7 and binutils 2.35 and my nm seems right, please confirm this solves it for you guys? http://nodejs.org/dist/nightlies/master/v0.11.11-nightly-20140108/node-v0.11.11-nightly-20140109-linux-x64.tar.gz |
@tjfontaine sure thing. I'll report back when I've got results. |
In the immortal words of every developer ever... "works for me!" 👍 That seems to have done the trick. |
Yep, works. Thanks, TJ. |
Taking v8::Object::CreationContext() as an example:
v0.10.24:
Local (clean) build:
v0.11.10:
/cc @tjfontaine @rmg
The text was updated successfully, but these errors were encountered: