Skip to content

Commit

Permalink
tools,bootstrap: preprocess gypi files to json
Browse files Browse the repository at this point in the history
PR-URL: #19140
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
devsnek authored and MylesBorins committed Mar 20, 2018

Verified

This commit was signed with the committer’s verified signature. The key has expired.
addaleax Anna Henningsen
1 parent 5a1437c commit 9613e02
Showing 2 changed files with 6 additions and 8 deletions.
9 changes: 1 addition & 8 deletions lib/internal/process.js
Original file line number Diff line number Diff line change
@@ -118,16 +118,9 @@ function setupMemoryUsage() {
function setupConfig(_source) {
// NativeModule._source
// used for `process.config`, but not a real module
var config = _source.config;
const config = _source.config;
delete _source.config;

// strip the gyp comment line at the beginning
config = config.split('\n')
.slice(1)
.join('\n')
.replace(/"/g, '\\"')
.replace(/'/g, '"');

process.config = JSON.parse(config, function(key, value) {
if (value === 'true') return true;
if (value === 'false') return false;
5 changes: 5 additions & 0 deletions tools/js2c.py
Original file line number Diff line number Diff line change
@@ -261,6 +261,11 @@ def JS2C(source, target):
if '/' in name or '\\' in name:
name = '/'.join(re.split('/|\\\\', name)[1:])

# if its a gypi file we're going to want it as json
# later on anyway, so get it out of the way now
if name.endswith(".gypi"):
lines = re.sub(r'#.*?\n', '', lines)
lines = re.sub(r'\'', '"', lines)
name = name.split('.', 1)[0]
var = name.replace('-', '_').replace('/', '_')
key = '%s_key' % var

0 comments on commit 9613e02

Please sign in to comment.