tools: fix C++ import checker argument expansion

[addaleax]

src: fix linter failures

Fix linter failures when running the linter on all source files.

tools: fix C++ import checker argument expansion

Makefile assumes that it can pass a list of files to the import
checker, whereas the import checker expects a single argument
that is interpreted as a blob.

Fix that mismatch by accepting multiple arguments in the import
checker.

Refs: #34565

Checklist

• make -j4 test (UNIX), or vcbuild test (Windows) passes
• tests and/or benchmarks are included
• commit message follows commit guidelines

[nodejs-github-bot]

Lite-CI: https://ci.nodejs.org/job/node-test-pull-request-lite-pipeline/4185

[richardlau]

There's still some sort of weirdness going on. The lint-cpp job from the linter workflow is failing for this PR:

Run make lint-cpp make lint-cpp shell: /bin/bash -e{0} env: PYTHON_VERSION: 3.8 NODE_VERSION: 10.x pythonLocation: /opt/hostedtoolcache/Python/3.8.5/x64 Running C++ linter... using statements aren't sorted in 'test/addons/repl-domain-abort/binding.cc'. Line 28: Actual: v8::Local, Expected: v8::Isolate Line 29: Actual: v8::Isolate, Expected: v8::Local Makefile:1337: recipe for target 'tools/.cpplintstamp' failed make: *** [tools/.cpplintstamp] Error 1 ##[error]Process completed with exit code 2. 

but I'm seeing more failures when I run against a fresh clone locally:

-bash-4.2$ ls -bash-4.2$ git clone https://github.com/addaleax/node -b fix-cpp-import-check-weirdness --depth=1 Cloning into 'node'... remote: Enumerating objects: 32499, done. remote: Counting objects: 100% (32499/32499), done. remote: Compressing objects: 100% (27110/27110), done. remote: Total 32499 (delta 7011), reused 14193 (delta 4235), pack-reused 0 Receiving objects: 100% (32499/32499), 74.99 MiB | 2.49 MiB/s, done. Resolving deltas: 100% (7011/7011), done. Checking out files: 100% (32854/32854), done. -bash-4.2$ cd node/ -bash-4.2$ git rev-parse HEAD 22c467d5ae4c505a2e61b6bcba644a72a8513ecc -bash-4.2$ make lint-cpp Running C++ linter... using statements aren't sorted in 'test/addons/repl-domain-abort/binding.cc'. Line 28: Actual: v8::Local, Expected: v8::Isolate Line 29: Actual: v8::Isolate, Expected: v8::Local File "test/addons/async-hooks-promise/binding.cc" does not use "NewStringType" File "test/addons/dlopen-ping-pong/binding.cc" does not use "NewStringType" using statements aren't sorted in 'test/addons/dlopen-ping-pong/binding.cc'. Line 19: Actual: v8::Object, Expected: v8::NewStringType Line 20: Actual: v8::NewStringType, Expected: v8::Object File "test/addons/non-node-context/binding.cc" does not use "Function" File "test/addons/non-node-context/binding.cc" does not use "FunctionTemplate" File "test/addons/non-node-context/binding.cc" does not use "NewStringType" File "test/addons/uv-handle-leak/binding.cc" does not use "Object" using statements aren't sorted in 'test/cctest/test_util.cc'. Line 61: Actual: node::StringEqualNoCase, Expected: node::Calloc Line 72: Actual: node::StringEqualNoCaseN, Expected: node::Malloc Line 87: Actual: node::ToLower, Expected: node::MaybeStackBuffer Line 101: Actual: node::Malloc, Expected: node::MaybeStackBuffer Line 109: Actual: node::Calloc, Expected: node::SPrintF Line 117: Actual: node::UncheckedMalloc, Expected: node::StringEqualNoCase Line 125: Actual: node::UncheckedCalloc, Expected: node::StringEqualNoCaseN Line 134: Actual: node::MaybeStackBuffer, Expected: node::ToLower Line 214: Actual: node::MaybeStackBuffer, Expected: node::UncheckedCalloc Line 257: Actual: node::SPrintF, Expected: node::UncheckedMalloc using statements aren't sorted in 'test/cctest/test_base64.cc'. Line 8: Actual: node::base64_encode, Expected: node::base64_decode Line 9: Actual: node::base64_decode, Expected: node::base64_encode using statements aren't sorted in 'test/embedding/embedtest.cc'. Line 19: Actual: v8::Value, Expected: v8::V8 Line 20: Actual: v8::V8, Expected: v8::Value make: *** [tools/.cpplintstamp] Error 1 -bash-4.2$ 

[richardlau]

Unfortunately it looks like Jenkins is currently broken (nodejs/build#2362 (comment) cc @nodejs/build-infra) -- would be interesting to see how the CI linter results tally.

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/32594/

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/32606/

[addaleax]

I can do a manual backport

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/32662/

[jasnell]

Landed in 93959b2...318c982