Chromium Code Reviews
Help | Chromium Project | Sign in
(285)

Issue 552203: Use the newly added CheckLicense() canned check to verify license boiler plate on source files. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
4 years, 2 months ago by M-A Ruel
Modified:
2 years, 11 months ago
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

Use the newly added CheckLicense() canned check to verify license boiler plate on source files.

Don't check for the year since there was no clear concensus about systematically updating the year on every change.

TEST=none
BUG=28291
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=37361

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+21 lines, -18 lines) Lint Patch
M PRESUBMIT.py View 3 chunks +21 lines, -18 lines 0 comments ? errors Download
Trybot results:
Commit:

Messages

Total messages: 4
M-A Ruel
4 years, 2 months ago #1
Peter Kasting
Not every file has a license, e.g. .grd files don't. Is this careful about the ...
4 years, 2 months ago #2
M-A Ruel
Adding Nicolas for the correctness. The sources white list is DEFAULT_WHITE_LIST in http://src.chromium.org/viewvc/chrome/trunk/tools/depot_tools/presubmit_support.py?view=markup I just ...
4 years, 2 months ago #3
Nicolas Sylvain
4 years, 2 months ago #4
On Wed, Jan 27, 2010 at 3:24 PM, <maruel@chromium.org> wrote:

> Adding Nicolas for the correctness.
>
> The sources white list is DEFAULT_WHITE_LIST in
>
>
http://src.chromium.org/viewvc/chrome/trunk/tools/depot_tools/presubmit_suppo...
>
> I just realized it includes files without an extension, which is quite bad.
> This, *.json and *.txt should be removed from the white list before
> checking
> this change in. What do you think?

I really have no idea

>
>
> http://codereview.chromium.org/552203
>
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 1280:2d3e6564b7b6