search

robertklep / node-metrohash

Node.js bindings for MetroHash
MIT License
26 stars 8 forks source link

Illegal instruction when built with g++ 7 #15

Open stanleyk opened 5 years ago

stanleyk commented 5 years ago

Everything works fine when built with g++ 5, but with g++ 7 I get the

Illegal instruction (core dumped)

error.

robertklep commented 5 years ago

I don't currently have any machines with g++ 7, and my knowledge of C++ is too little to be able to point you towards a possible reason and fix :(

stanleyk commented 5 years ago

I can run some tests with g++ 7 for you if you tell me what I should do. ;)

robertklep commented 5 years ago

I think it needs to start with running Node.js from a debugger to see where exactly the crash happens. I'll check and see if Raspbian comes with g++ 7 so I can check it myself, but that won't be before friday (if not, I'll see if I can get it up and running in a Docker container).

robertklep commented 5 years ago

Finally got around to testing. With g++ 9 it seems to work just fine (Ubuntu 19.10): image

However, that's with Node v10 because there still isn't a v12 available for Ubuntu 19.10 (at least not as a pre-built package).