gemini-testing / png-img

Lite libpng wrapper for node.js
MIT License
29 stars 13 forks source link

Support Node 14 #55

Closed sigorilla closed 3 years ago

sigorilla commented 3 years ago

Fixed: #54

xrsd commented 3 years ago

Может, есть смысл тогда и prebuild поддержать для 14-й ноды? См. .travis.yml

sigorilla commented 3 years ago

мержим?

xrsd commented 3 years ago

Да, я уже делаю фикс.

kaineer commented 3 years ago

Не компилится :(

$ npm i -DE png-img
...
npm ERR! ../src/PngImg.cc: In member function ‘void PngImg::InitStorage_()’:
npm ERR! ../src/PngImg.cc:71:20: error: ‘runtime_error’ is not a member of ‘std’
npm ERR!    71 |         throw std::runtime_error("Image is too large to allocate single buffer");
npm ERR!       |                    ^~~~~~~~~~~~~
npm ERR! make: *** [png_img.target.mk:118: Release/obj.target/png_img/src/PngImg.o] Error 1

Из того, что пока что накопал:

P.S. Попробовал добавить #include <stdexcept>, заработало. P.P.S. PR здесь: https://github.com/gemini-testing/png-img/pull/60