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

Issue 8055016: Fix EOL issues. (Closed)

Created:
9 years, 2 months ago by M-A Ruel
Modified:
9 years, 2 months ago
Reviewers:
cmp
CC:
chromium-reviews, Nicolas Sylvain
Visibility:
Public.

Description

Fix EOL issues. Set svn:eol-style=CRLF for all .bat files and everything in bootstrap/win/ Set svn:eol-style=LF for everything else. Also delete depot_tools/git_cl/ since it's not used anymore. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=103051

Patch Set 1 #

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+582 lines, -582 lines) Patch
M depot_tools/.gitignore View 0 chunks +-1 lines, --1 lines 0 comments Download
M depot_tools/LICENSE View 0 chunks +-1 lines, --1 lines 0 comments Download
M depot_tools/OWNERS View 0 chunks +-1 lines, --1 lines 0 comments Download
M depot_tools/README View 0 chunks +-1 lines, --1 lines 0 comments Download
M depot_tools/README.codereview View 0 chunks +-1 lines, --1 lines 0 comments Download
MM depot_tools/README.gclient View 1 chunk +82 lines, -82 lines 0 comments Download
M depot_tools/README.git-cl View 0 chunks +-1 lines, --1 lines 0 comments Download
M depot_tools/README.testing View 0 chunks +-1 lines, --1 lines 0 comments Download
M depot_tools/WATCHLISTS View 0 chunks +-1 lines, --1 lines 0 comments Download
MM depot_tools/bootstrap/gclient.bat View 1 chunk +10 lines, -10 lines 0 comments Download
MM depot_tools/bootstrap/update_depot_tools.bat View 1 chunk +15 lines, -15 lines 0 comments Download
M depot_tools/bootstrap/win/README.google View 0 chunks +-1 lines, --1 lines 0 comments Download
MM depot_tools/bootstrap/win/get_file.js View 1 chunk +118 lines, -118 lines 0 comments Download
MM depot_tools/bootstrap/win/python.new.bat View 1 chunk +4 lines, -4 lines 0 comments Download
MM depot_tools/bootstrap/win/svn.new.bat View 1 chunk +4 lines, -4 lines 0 comments Download
MM depot_tools/bootstrap/win/unzip.js View 1 chunk +91 lines, -91 lines 0 comments Download
MM depot_tools/bootstrap/win/win_tools.bat View 1 chunk +144 lines, -144 lines 0 comments Download
MM depot_tools/chrome-update.bat View 1 chunk +5 lines, -5 lines 0 comments Download
MM depot_tools/chrome-update-create-task.bat View 1 chunk +52 lines, -52 lines 0 comments Download
M depot_tools/chromite_wrapper View 0 chunks +-1 lines, --1 lines 0 comments Download
MM depot_tools/cpplint.bat View 1 chunk +5 lines, -5 lines 0 comments Download
M depot_tools/create-chromium-git-src View 0 chunks +-1 lines, --1 lines 0 comments Download
M depot_tools/drover.bat View 0 chunks +-1 lines, --1 lines 0 comments Download
M depot_tools/drover.py View 0 chunks +-1 lines, --1 lines 0 comments Download
MM depot_tools/gcl.bat View 1 chunk +5 lines, -5 lines 0 comments Download
MM depot_tools/gclient.bat View 1 chunk +43 lines, -43 lines 0 comments Download
M depot_tools/git-cl-upload-hook View 0 chunks +-1 lines, --1 lines 0 comments Download
MM depot_tools/hammer.bat View 1 chunk +23 lines, -23 lines 0 comments Download
M depot_tools/repo View 0 chunks +-1 lines, --1 lines 0 comments Download
M depot_tools/update_depot_tools View 0 chunks +-1 lines, --1 lines 0 comments Download
M depot_tools/watchlists.py View 0 chunks +-1 lines, --1 lines 0 comments Download
M depot_tools/weekly View 0 chunks +-1 lines, --1 lines 0 comments Download
M depot_tools/wtf View 0 chunks +-1 lines, --1 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
M-A Ruel
9 years, 2 months ago (2011-09-27 15:25:38 UTC) #1
cmp
lgtm
9 years, 2 months ago (2011-09-27 19:31:56 UTC) #2
cmp
Should we invest in a PRESUBMIT check to ensure that new files have the right ...
9 years, 2 months ago (2011-09-27 19:32:40 UTC) #3
M-A Ruel
9 years, 2 months ago (2011-09-27 19:34:01 UTC) #4
On 2011/09/27 19:32:40, cmp wrote:
> Should we invest in a PRESUBMIT check to ensure that new files have the right
> status set?

No, it's mostly a past issue due to not using auto-props in the old days.

Powered by Google App Engine
This is Rietveld 408576698