diff options
author | Rod Vagg <rod@vagg.org> | 2018-01-23 21:49:25 +1100 |
---|---|---|
committer | Rod Vagg <rod@vagg.org> | 2018-01-25 08:18:24 +1100 |
commit | 57bd27eda8ae0f540c14c8480128bca42b387314 (patch) | |
tree | 5102c45e897172b28db20b9d3f2258af3878675d /tools/doc | |
parent | d5d024d6ec4241bfb6449c314ee620cf6953ab25 (diff) | |
download | android-node-v8-57bd27eda8ae0f540c14c8480128bca42b387314.tar.gz android-node-v8-57bd27eda8ae0f540c14c8480128bca42b387314.tar.bz2 android-node-v8-57bd27eda8ae0f540c14c8480128bca42b387314.zip |
Revert "build,test: make building addon tests less fragile"
This reverts commit d9b59def72c718aaad3eefb6bf43f409ccefe4d2.
Breaks downloadable source tarball builds as we remove some files prior
to creating a tarball but those files are included in the comprehensive
list of dependencies listed in .deps.
Ref: https://github.com/nodejs/node/pull/17407
PR-URL: https://github.com/nodejs/node/pull/18287
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Diffstat (limited to 'tools/doc')
-rw-r--r-- | tools/doc/addon-verify.js | 11 |
1 files changed, 1 insertions, 10 deletions
diff --git a/tools/doc/addon-verify.js b/tools/doc/addon-verify.js index 51ba75ca45..4da99d64d7 100644 --- a/tools/doc/addon-verify.js +++ b/tools/doc/addon-verify.js @@ -5,7 +5,6 @@ const fs = require('fs'); const path = require('path'); const marked = require('marked'); -const auto = 'Auto-generated by `node tools/doc/addon-verify.js`'; const rootDir = path.resolve(__dirname, '..', '..'); const doc = path.resolve(rootDir, 'doc', 'api', 'addons.md'); const verifyDir = path.resolve(rootDir, 'test', 'addons'); @@ -50,20 +49,12 @@ for (const header in addons) { files = Object.entries(files).map(([name, content]) => { if (name === 'test.js') content = boilerplate(name, content); - content = `// ${auto}\n${content}`; - if (name.endsWith('.h')) { - content = content.replace(/(#(ifndef|define) \w+_H)/g, - '$1 // NOLINT(build/header_guard)'); - content = content.replace(/(#endif)$/, - '$1 // NOLINT(build/header_guard)'); - } - if (!content.endsWith('\n')) content += '\n'; // Pacify linter. return { name, content, path: path.resolve(dir, name) }; }); files.push({ path: path.resolve(dir, 'binding.gyp'), - content: `# ${auto}\n` + JSON.stringify({ + content: JSON.stringify({ targets: [ { target_name: 'binding', |