build,win: delegate lint-cpp to make
* look for GNU Make in the Path or use wsl PR-URL: https://github.com/nodejs/node/pull/28102 Fixes: https://github.com/nodejs/node/issues/28086 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: João Reis <reis@janeasystems.com>
This commit is contained in:
parent
43a5170858
commit
4b1bcae681
52
vcbuild.bat
52
vcbuild.bat
@ -593,51 +593,17 @@ goto lint-cpp
|
|||||||
|
|
||||||
:lint-cpp
|
:lint-cpp
|
||||||
if not defined lint_cpp goto lint-js
|
if not defined lint_cpp goto lint-js
|
||||||
call :run-lint-cpp src\*.c src\*.cc src\*.h test\addons\*.cc test\addons\*.h test\js-native-api\*.cc test\js-native-api\*.cc test\js-native-api\*.h test\node-api\*.cc test\node-api\*.cc test\node-api\*.h test\cctest\*.cc test\cctest\*.h tools\icu\*.cc tools\icu\*.h
|
if defined NODEJS_MAKE goto run-make-lint
|
||||||
python tools/check-imports.py
|
where make > NUL 2>&1 && make -v | findstr /C:"GNU Make" 1> NUL
|
||||||
|
if "%ERRORLEVEL%"=="0" set "NODEJS_MAKE=make PYTHON=python" & goto run-make-lint
|
||||||
|
where wsl > NUL 2>1
|
||||||
|
if "%ERRORLEVEL%"=="0" set "NODEJS_MAKE=wsl make" & goto run-make-lint
|
||||||
|
echo Could not find GNU Make, needed for linting C/C++
|
||||||
goto lint-js
|
goto lint-js
|
||||||
|
|
||||||
:run-lint-cpp
|
:run-make-lint
|
||||||
if "%*"=="" goto exit
|
%NODEJS_MAKE% lint-cpp
|
||||||
echo running lint-cpp '%*'
|
goto lint-js
|
||||||
set cppfilelist=
|
|
||||||
setlocal enabledelayedexpansion
|
|
||||||
for /f "tokens=*" %%G in ('dir /b /s /a %*') do (
|
|
||||||
set relpath=%%G
|
|
||||||
set relpath=!relpath:*%~dp0=!
|
|
||||||
call :add-to-list !relpath! > nul
|
|
||||||
)
|
|
||||||
( endlocal
|
|
||||||
set cppfilelist=%localcppfilelist%
|
|
||||||
)
|
|
||||||
python tools/cpplint.py %cppfilelist% > nul
|
|
||||||
goto exit
|
|
||||||
|
|
||||||
:add-to-list
|
|
||||||
@rem Subroutine used to filter items from the cpplint file list
|
|
||||||
echo %1 | findstr /c:"src\node_root_certs.h" > nul 2>&1
|
|
||||||
if %errorlevel% equ 0 goto exit
|
|
||||||
|
|
||||||
echo %1 | findstr /c:"src\tracing\trace_event.h" > nul 2>&1
|
|
||||||
if %errorlevel% equ 0 goto exit
|
|
||||||
|
|
||||||
echo %1 | findstr /c:"src\tracing\trace_event_common.h" > nul 2>&1
|
|
||||||
if %errorlevel% equ 0 goto exit
|
|
||||||
|
|
||||||
echo %1 | findstr /r /c:"test\\addons\\[0-9].*_.*\.h" > nul 2>&1
|
|
||||||
if %errorlevel% equ 0 goto exit
|
|
||||||
|
|
||||||
echo %1 | findstr /r /c:"test\\addons\\[0-9].*_.*\.cc" > nul 2>&1
|
|
||||||
if %errorlevel% equ 0 goto exit
|
|
||||||
|
|
||||||
echo %1 | findstr /c:"test\js-native-api\common.h" > nul 2>&1
|
|
||||||
if %errorlevel% equ 0 goto exit
|
|
||||||
|
|
||||||
echo %1 | findstr /c:"test\node-api\common.h" > nul 2>&1
|
|
||||||
if %errorlevel% equ 0 goto exit
|
|
||||||
|
|
||||||
set "localcppfilelist=%localcppfilelist% %1"
|
|
||||||
goto exit
|
|
||||||
|
|
||||||
:lint-js
|
:lint-js
|
||||||
if defined lint_js_ci goto lint-js-ci
|
if defined lint_js_ci goto lint-js-ci
|
||||||
|
Loading…
x
Reference in New Issue
Block a user