Commit Graph

24 Commits

Author SHA1 Message Date
Scott Mansell e38c0a1bbd lint: replace egrep with grep -E
Starting with grep 3.8, egrep throws the following warning
   egrep: warning: egrep is obsolescent; using grep -E

Whch is annoying... so we will use grep -E
2023-02-02 17:23:40 +13:00
Pierre Bourdon c92de420ef
lint: update to clang-format-13 2023-02-02 04:56:08 +01:00
OatmealDome a25c835d00 lint: Don't check for WSL2 host path on non-Linux OSes 2022-08-15 16:13:18 -04:00
Shawn Hoffman a93babb1b3 lint: use git.exe if the source resides on v9fs
speedup for running script under wsl2 when sources are mounted from host
2022-06-08 21:10:31 -07:00
Léo Lam 815b590dfe
Update clang-format to version 12
12 is the latest release and various editors (VS, VSCode and CLion)
ship with clang-format 12 by default.
2021-08-30 00:20:18 +02:00
Shawn Hoffman 9399e8b2bc require clang-format 10.0 2021-01-28 09:56:31 -08:00
Léo Lam 4cc2d97294 Require clang-format 9 and reformat source code
This updates the lint script to require clang-format 9 and reformats
existing source code. Since VS2019 ships with clang-format 9 this
should make auto reformats less painful.

This also updates the clang-format configuration to set
BraceWrapping.AfterCaseLabel to true to ensure consistent brace
style; otherwise clang-format 9+ defaults to putting braces on
the same line as switch case labels.
2020-01-08 22:18:15 +01:00
Techjar a72f56f558 Update lint.sh to clang-format 7.0 2019-05-05 19:25:42 -04:00
spycrab f43c6d68f2 Tools/lint.sh: Look for specific versions of clang-format first 2019-03-09 15:08:22 +01:00
booto 712f44edd9 lint: remove superfluous logging
Make lint script be silent unless there's a problem. This line doesn't
work anyway, trying to redirect to fd 0.
2018-10-05 19:35:46 -04:00
riking c5ecb718ac Add Android code style enforcement to lint.sh
This requires buildbot changes: the path to the Android Studio
installation must be supplied in an environment variable.

Modified files are copied out to a temporary directory, Android Studio
is asked to format the files, and a git diff is performed.
2018-08-26 19:18:31 -07:00
spycrab d27e85e9d7 Tools: Bump lint.sh version 2018-04-12 10:49:55 +02:00
N.E.C f9b8365190 Tools: Check for git and clang-format version 3.8.* in lint.sh 2017-08-16 22:28:02 -07:00
Michael M 4c92d8c81e Tools/lint.sh: lint all of Source, not just Core 2017-08-13 19:04:56 -07:00
Michael Maltese 6939590598 Tools/lint.sh: turn on Bash 'strict mode'
Now fails if any commands fail. I noticed that on the buildbot, the
arguments to this script can be incorrect (somehow?) which can result in
no lint checks happening but the buildbot still passing.
2017-07-18 15:36:15 -07:00
Léo Lam 9c63544445 Tools: Allow diffing against arbitrary revisions
Instead of applying PRs as patches and leaving them as staged changes,
the lint builder now checks out the PR branch, so the lint script
must be able to call git diff with more than just --cached.
(The buildbot will call it with "master...".)
2017-06-05 02:19:08 +02:00
Sepalani 83489e2766 lint: Check for newline at EOF 2017-05-08 19:20:42 +01:00
Léo Lam 1fe31f6f19 Tools: Fix the lint script
It turns out that last fix was only partial, and it didn't fix the
issue completely as paths were fixed, but staged changes were not
in the list. This should fix it for real, this time. Sorry about that…

(git diff needs --cached to include staged changes, unlike git status.)
2016-08-11 21:11:17 +02:00
Léo Lam cbb77532c6 Tools: Fix lint by removing unneeded awk
git diff --name-only already took care of only returning the name, so
the awk is unneeded and makes it return only empty file names.

Facepalm, I know. Sorry for this oversight.

(Also fixes something that lint didn't catch because of this)
2016-08-02 22:00:41 +02:00
Léo Lam 671c0bc06f Tools: Don't run lint on deleted files
Generates a harmless "No such file or directory" otherwise when a file
is removed in a commit.
2016-07-26 00:51:10 +02:00
Pierre Bourdon 8a1bbaa563 lint: bash is hard, actually return errors 2016-06-24 14:27:35 +02:00
Pierre Bourdon 3af8f71472 linter: run clang-format and fail on diffs 2016-06-24 13:23:10 +02:00
Pierre Bourdon e1c65406dc linter: Stop checking for include ordering (inconsistent with clang-format) 2016-06-24 13:14:41 +02:00
Pierre Bourdon 15ad14a9dc Add linter script which will replace the ad-hoc commands ran by the linter Buildbot. 2015-10-31 18:43:37 +01:00