1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

build: split up cpplint to avoid long cmd lines

Browse Source 
Refactors cpplint slightly to allow multiple runs of it. This allows
downstream projects to run cpplint on their dependencies.

PR-URL: https://github.com/nodejs/node/pull/14116
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Kunal Pathak <kunal.pathak@microsoft.com>
Reviewed-By: João Reis <reis@janeasystems.com>
This commit is contained in:
Kyle Farnung 2017-07-06 16:03:27 -07:00 committed by Refael Ackermann
parent 8206a01c99
commit d48472c20f
No known key found for this signature in database
GPG Key ID: CD704BD80FDDDB64
1 changed files with 9 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
vcbuild.bat
View File

@ -435,12 +435,16 @@ goto cpplint
:cpplint
if not defined cpplint goto jslint
echo running cpplint
call :run-cpplint src\*.c src\*.cc src\*.h test\addons\*.cc test\addons\*.h test\cctest\*.cc test\cctest\*.h test\gc\binding.cc tools\icu\*.cc tools\icu\*.h
call :run-python tools/check-imports.py
goto jslint
:run-cpplint
if "%*"=="" goto exit
echo running cpplint '%*'
set cppfilelist=
setlocal enabledelayedexpansion
for /f "tokens=*" %%G in ('dir /b /s /a src\*.c src\*.cc src\*.h ^
test\addons\*.cc test\addons\*.h test\cctest\*.cc test\cctest\*.h ^
test\gc\binding.cc tools\icu\*.cc tools\icu\*.h') do (
for /f "tokens=*" %%G in ('dir /b /s /a %*') do (
set relpath=%%G
set relpath=!relpath:*%~dp0=!
call :add-to-list !relpath!
@ -449,8 +453,7 @@ test\gc\binding.cc tools\icu\*.cc tools\icu\*.h') do (
set cppfilelist=%localcppfilelist%
)
call :run-python tools/cpplint.py %cppfilelist%
call :run-python tools/check-imports.py
goto jslint
goto exit
:add-to-list
echo %1 | findstr /c:"src\node_root_certs.h"