Skip to content

Commit

Permalink
meson: Require Node.js >= 18 when GumJS is enabled
Browse files Browse the repository at this point in the history
Our JavaScript runtime autogen doesn't need it just yet, but will once
we upgrade frida-compile. Another reason is that somebody is likely to
build frida-core after frida-gum. They might install some version < 18
when encountering the check in this project, only to be disappointed
when frida-core tells them they need a newer version.
  • Loading branch information
oleavr committed Apr 19, 2024
1 parent b9d5ec4 commit 8f55fff
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion meson.build
Original file line number Diff line number Diff line change
Expand Up @@ -644,9 +644,13 @@ if have_gumjs
'sysprof=disabled',
])

node = find_program('node', version: '>=18.0.0', native: true, required: false)
if not node.found()
error('Need Node.js >= 18.0.0 to process JavaScript code at build-time')
endif
npm = find_program('npm', native: true, required: false)
if not npm.found()
error('Need Node.js and npm to process JavaScript code at build-time')
error('Need npm to process JavaScript code at build-time')
endif
else
v8_dep = disabler()
Expand Down

0 comments on commit 8f55fff

Please sign in to comment.