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

Issue 3362015: Add more special cases to licenses.py for direct source code checkouts.... (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
3 years, 7 months ago by Paweł Hajdan Jr.
Modified:
2 years, 11 months ago
CC:
chromium-reviews_chromium.org, pam+watch_chromium.org
Visibility:
Public.

Description

Add more special cases to licenses.py for direct source code checkouts.

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

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+12 lines, -0 lines) Lint Patch
M tools/licenses.py View 1 chunk +12 lines, -0 lines 2 comments 0 errors Download
Trybot results:
Commit:

Messages

Total messages: 6
Paweł Hajdan Jr.
3 years, 7 months ago #1
Paweł Hajdan Jr.
ping
3 years, 7 months ago #2
Lei Zhang
LGTM stamp.
3 years, 7 months ago #3
Mark Mentovai
LGTM
3 years, 7 months ago #4
scr
Hi, I'm trying my first use of the review tool. My first comment is more ...
3 years, 7 months ago #5
Paweł Hajdan Jr.
3 years, 7 months ago #6
http://codereview.chromium.org/3362015/diff/1/2
File tools/licenses.py (right):

http://codereview.chromium.org/3362015/diff/1/2#newcode73
tools/licenses.py:73: "Name": "protobuf",
On 2010/09/10 21:45:29, scr wrote:
> Should this indicate that this is protobuf2 with a string like "protobuf
> (version 2)"?

I don't think so. This licenses.py tool is just yet another bureaucracy, but it
seems we need it.
Sign in to reply to this message.

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