Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(116)

Issue 7888058: Add check_licenses step to the buildbot. (Closed)

Created:
9 years, 3 months ago by Paweł Hajdan Jr.
Modified:
9 years, 3 months ago
CC:
chromium-reviews
Visibility:
Public.

Description

Add check_licenses step to the buildbot. BUG=28291 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=101313

Patch Set 1 #

Total comments: 6

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+12 lines, -1 line) Patch
M masters/master.chromium/master_linux_cfg.py View 1 2 chunks +2 lines, -1 line 0 comments Download
M masters/master.tryserver.chromium/master.cfg View 1 1 chunk +1 line, -0 lines 0 comments Download
M scripts/master/factory/chromium_commands.py View 1 2 chunks +8 lines, -0 lines 0 comments Download
M scripts/master/factory/chromium_factory.py View 1 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Paweł Hajdan Jr.
9 years, 3 months ago (2011-09-14 21:54:31 UTC) #1
nsylvain
LGTM with nits http://codereview.chromium.org/7888058/diff/1/masters/master.chromium/master_linux_cfg.py File masters/master.chromium/master_linux_cfg.py (right): http://codereview.chromium.org/7888058/diff/1/masters/master.chromium/master_linux_cfg.py#newcode113 masters/master.chromium/master_linux_cfg.py:113: 'check_licenses', I'd personally remove both check_deps ...
9 years, 3 months ago (2011-09-15 16:54:49 UTC) #2
Paweł Hajdan Jr.
9 years, 3 months ago (2011-09-15 17:21:06 UTC) #3
http://codereview.chromium.org/7888058/diff/1/masters/master.chromium/master_...
File masters/master.chromium/master_linux_cfg.py (right):

http://codereview.chromium.org/7888058/diff/1/masters/master.chromium/master_...
masters/master.chromium/master_linux_cfg.py:113: 'check_licenses',
On 2011/09/15 16:54:49, nsylvain wrote:
> I'd personally remove both check_deps and check_licenses here. up to you
though.

Removed check_licenses then.

http://codereview.chromium.org/7888058/diff/1/masters/master.chromium/master_...
masters/master.chromium/master_linux_cfg.py:218: 'check_licenses',
On 2011/09/15 16:54:49, nsylvain wrote:
> I dont think we need it here.

Done.

http://codereview.chromium.org/7888058/diff/1/scripts/master/factory/chromium...
File scripts/master/factory/chromium_commands.py (right):

http://codereview.chromium.org/7888058/diff/1/scripts/master/factory/chromium...
scripts/master/factory/chromium_commands.py:78: self._check_licenses_tool =
J('src', 'tools', 'checklicenses', 'checklicenses.py')
On 2011/09/15 16:54:49, nsylvain wrote:
> 80 chars

Done.

Powered by Google App Engine
This is Rietveld 408576698