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

Issue 99072: Add an optional flag to gcl.ReadFile() so that presubmit can support CRLF PRE... (Closed)

Created:
11 years, 8 months ago by M-A Ruel
Modified:
9 years, 7 months ago
Reviewers:
jam
CC:
chromium-reviews_googlegroups.com, Jeff Bailey
Base URL:
svn://chrome-svn.corp.google.com/chrome/trunk/tools/
Visibility:
Public.

Description

Add an optional flag to gcl.ReadFile() so that presubmit can support CRLF PRESUBMIT.py scripts. Also add a version number to gcl.py. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=14644

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -5 lines) Patch
M depot_tools/gcl.py View 3 chunks +7 lines, -2 lines 0 comments Download
M depot_tools/presubmit.py View 2 chunks +3 lines, -2 lines 0 comments Download
M depot_tools/tests/presubmit_unittest.py View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 2 (0 generated)
M-A Ruel
11 years, 8 months ago (2009-04-27 19:52:31 UTC) #1
jam
11 years, 8 months ago (2009-04-27 20:28:03 UTC) #2
ok (although I don't really understand why this is needed.  I thought we wanted
to avoid checking in CRLF files, is there a reason why presubmit files would be
an exception?

Powered by Google App Engine
This is Rietveld 408576698