Skip to content

Commit

Permalink
Use Python 3 in gyp bindings
Browse files Browse the repository at this point in the history
Python 3 is already used by gyp, and `python` binary is removed from
e.g. macOS 13.2.

Fixes RuntimeTools#653
  • Loading branch information
koterpillar committed Mar 23, 2022
1 parent 4862e68 commit 7975c4a
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
1 change: 1 addition & 0 deletions AUTHORS.md
Original file line number Diff line number Diff line change
Expand Up @@ -23,3 +23,4 @@ Authors ordered by first contribution:
- Richard Waller (https://github.com/rwalle61)
- Russell Howe (https://github.com/rhowe-gds)
- Gaby Baghdadi (https://github.com/gabylb)
- Alexey Kotlyarov (https://github.com/koterpillar)
6 changes: 3 additions & 3 deletions binding.gyp
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,8 @@
"srcdir%": "./src",
"agentcoredir%": "./omr-agentcore",
"nandir%": "<!(node -e \"try {require('nan')}catch (e){console.log(e)}\")",
'build_id%': '.<!(["python", "./generate_build_id.py"])',
'appmetricsversion%': '<!(["python", "./get_from_json.py", "./package.json", "version"])',
'build_id%': '.<!(["python3", "./generate_build_id.py"])',
'appmetricsversion%': '<!(["python3", "./get_from_json.py", "./package.json", "version"])',
"conditions": [
['OS=="aix"', {
"SHARED_LIB_SUFFIX": ".a",
Expand Down Expand Up @@ -119,7 +119,7 @@
'inputs': [ "<(srcdir)/appmetrics.cpp" ],
'outputs': [ "<(INTERMEDIATE_DIR)/appmetrics.cpp" ],
'action': [
'python',
'python3',
'./replace_in_file.py',
'<(srcdir)/appmetrics.cpp',
'<(INTERMEDIATE_DIR)/appmetrics.cpp',
Expand Down

0 comments on commit 7975c4a

Please sign in to comment.