Closed psaiz closed 2 years ago
vendor
is a changing reference to the git repository google/re2, which contains C++ and some other code. It is not part of node-re2
codebase, but a separate project we do not control. I suggest proposing the suggested change there.
node-re2
uses some C++ files from google/re2
project to compile a binary extension. No python files, or any other files, are used for that.
Hi Eugene, Thanks for looking into this. I've created the issue there, and they are looking into it. If it gets accepted, would it be possible to have a new version of node-re2 with those changes?
Thanks again
If it gets accepted, would it be possible to have a new version of node-re2 with those changes?
Of course. Please let me know, and I will issue a new technical release.
Hi Eugene. Here I come again :-) The patch to google/re has been accepted. Would it be possible to get a new version of your module?
node-re2
uses tagged releases of google/re2. IIRC, they usually release (tag) once a month on the 1st. The latest release was 2021-11-1, which is used by node-re2
at this moment.
So as soon as they cut a release, I'll update and republish this project.
Hi @uhop, there was indeed a new version of google/re2 on the 1st of the month. Would you mind creating a new one of yours? Thanks again,
Published today as 1.17.4.
Hi,
The module deploys two python files with a shebang pointing to python (instead of python3). This creates some issues to create rpms. Would it be possible to change the shebangs to use python3 ?
Best regards, pablo