diff options
| author | Miklos Vajna <vmiklos@collabora.co.uk> | 2017-11-28 22:49:30 +0100 |
|---|---|---|
| committer | Miklos Vajna <vmiklos@collabora.co.uk> | 2017-11-29 09:21:16 +0100 |
| commit | 4e0dd43f9a90917676a46e451abaaaa2e52d7f2e (patch) | |
| tree | b0b58c81342f867dda4d6f6bdbc3bd9e9728d87e /solenv | |
| parent | 793bbac379c5800dc09ff76f093d45047e662ff0 (diff) | |
clang-format: don't pick up non-existing binary from PATH
When /opt/lo/bin/clang-format was missing, we "found" clang-format in
the last item of PATH, which doesn't make sense.
(Fix a spelling error in a formatted file to trigger the hook as part of
CI.)
Change-Id: I3b56a8ad02b4a55dba58e07286e31436a2842cf6
Diffstat (limited to 'solenv')
| -rw-r--r-- | solenv/clang-format/ClangFormat.pm | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/solenv/clang-format/ClangFormat.pm b/solenv/clang-format/ClangFormat.pm index 8cd891229843..26de82de06e5 100644 --- a/solenv/clang-format/ClangFormat.pm +++ b/solenv/clang-format/ClangFormat.pm @@ -73,9 +73,9 @@ sub find() { foreach my $dir (split /:/, $ENV{PATH}) { - $clang_format = "$dir/clang-format"; - if (is_matching_clang_format_version($clang_format, $version)) + if (is_matching_clang_format_version("$dir/clang-format", $version)) { + $clang_format = "$dir/clang-format"; last; } } |