uhop / node-re2

node.js bindings for RE2: fast, safe alternative to backtracking regular expression engines.
Other
495 stars 53 forks source link

Support build for Electron 11 on MacOS #97

Closed sergei-dyshel closed 3 years ago

sergei-dyshel commented 3 years ago

Build for Electron 11 on MacOS fails:

In file included from ../lib/replace.cc:1:
In file included from ../lib/./wrapped_re2.h:4:
In file included from ../node_modules/nan/nan.h:56:
In file included from /Users/sergei/Library/Caches/node-gyp/11.3.0/include/node/node.h:67:
In file included from /Users/sergei/Library/Caches/node-gyp/11.3.0/include/node/v8.h:30:
/Users/sergei/Library/Caches/node-gyp/11.3.0/include/node/v8-internal.h:418:38: error: no template named 'remove_cv_t' in namespace 'std'; did you mean 'remove_cv'?
            !std::is_same<Data, std::remove_cv_t<T>>::value>::Perform(data);
                                ~~~~~^~~~~~~~~~~
                                     remove_cv
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../include/c++/v1/type_traits:665:50: note: 'remove_cv' declared here
template <class _Tp> struct _LIBCPP_TEMPLATE_VIS remove_cv
                                                 ^

More context in https://github.com/electron/electron/issues/26364.

radicand commented 3 years ago

Could a new version be published with this change?

uhop commented 3 years ago

I am planning to do a new release for Node 16. This change will be included.