diff options
| author | thestig@chromium.org <thestig@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-05-20 18:06:46 +0000 |
|---|---|---|
| committer | thestig@chromium.org <thestig@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-05-20 18:06:46 +0000 |
| commit | 554803b6b16ebce7aa94ba5d28088099ae6554e3 (patch) | |
| tree | 3746381656dee1b2b86048ec234b73d0d8d40c6e /tools/checklicenses | |
| parent | a457da86dcdf31ff9277ebcbb93aeb2266e56b05 (diff) | |
| download | chromium_src-554803b6b16ebce7aa94ba5d28088099ae6554e3.zip chromium_src-554803b6b16ebce7aa94ba5d28088099ae6554e3.tar.gz chromium_src-554803b6b16ebce7aa94ba5d28088099ae6554e3.tar.bz2 | |
checklicenses: Do not report unused suppressions when checking a sub-directory.
In this case, it's not very useful and full of false positives.
NOTRY=true
Review URL: https://codereview.chromium.org/293843002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271711 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'tools/checklicenses')
| -rwxr-xr-x | tools/checklicenses/checklicenses.py | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/tools/checklicenses/checklicenses.py b/tools/checklicenses/checklicenses.py index 5277787..73d0763 100755 --- a/tools/checklicenses/checklicenses.py +++ b/tools/checklicenses/checklicenses.py @@ -460,11 +460,12 @@ def check_licenses(options, args): if success: print "\nSUCCESS\n" - unused_suppressions = set( - PATH_SPECIFIC_WHITELISTED_LICENSES.keys()).difference(used_suppressions) - if unused_suppressions: - print "\nNOTE: unused suppressions detected:\n" - print '\n'.join(unused_suppressions) + if not len(args): + unused_suppressions = set( + PATH_SPECIFIC_WHITELISTED_LICENSES.keys()).difference(used_suppressions) + if unused_suppressions: + print "\nNOTE: unused suppressions detected:\n" + print '\n'.join(unused_suppressions) return 0 else: |