I think you linked to the wrong issue?

Also is the addition of --without-snapshot necessary?

FYI: as author of the linked issue, I can confirm that has nothing to do with this PR.

Hum, sorry for the wrong issue mention... Dunno what happened; first post updated.
@mscdex --without-snapshot allowed me to complete the build. I get a weird syntax error on a ) when I ommit it. Will post a log output for this error next monday if you want.

@mscdex as @bsegault said, if use --with-snapshot, we can't build pass for Android.

Here is the build log.

/home/kiki/node.js/node-v5.4.1/out/Release/mksnapshot: 1: /home/kiki/node.js/node-v5.4.1/out/Release/mksnapshot: Syntax error: word unexpected (expecting ")")
make[1]: *** [/home/kiki/node.js/node-v5.4.1/out/Release/obj.target/v8_snapshot/geni/snapshot.cc] Error 2
make[1]: Leaving directory `/home/kiki/node.js/node-v5.4.1/out'
make: *** [node] Error 2

I'm seeking some feedback about how Node.js is used on Android if anyone watching this thread would like to tell us about it: nodejs/build#359

Closing, obsoleted by #5544 which landed in commit 271201f. We're building with -fPIE -fpie now so this should be working.