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

Issue 2014753002: Update build instructions for Linux and Windows. (Closed)

Created:
4 years, 7 months ago by brettw
Modified:
4 years, 7 months ago
Reviewers:
scottmg
CC:
chromium-reviews
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Update build instructions for Linux and Windows. Removes redundant instructions from the platform-specific plages and the main "get the code" page on the sites page. The platform pages should have only non-redundant platform-specific information. When I did this the Linux page was basically empty with a link the the prerequistes page, so I consolidated these two pages, leaving a placeholder in the old prerequisites page for now to avoid breaking links. There was a lot of out-of-date information removed. Committed: https://crrev.com/c25693b3002a6a9a8c074cf9460ae3dca262bbaf Cr-Commit-Position: refs/heads/master@{#396071}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+163 lines, -325 lines) Patch
M docs/clang.md View 2 chunks +3 lines, -5 lines 0 comments Download
M docs/linux_build_instructions.md View 1 chunk +102 lines, -83 lines 0 comments Download
M docs/linux_build_instructions_prerequisites.md View 1 chunk +4 lines, -158 lines 0 comments Download
M docs/linux_faster_builds.md View 2 chunks +19 lines, -48 lines 0 comments Download
M docs/windows_build_instructions.md View 5 chunks +35 lines, -31 lines 0 comments Download

Messages

Total messages: 14 (7 generated)
brettw
4 years, 7 months ago (2016-05-26 00:02:28 UTC) #2
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/2014753002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/2014753002/1
4 years, 7 months ago (2016-05-26 00:03:49 UTC) #4
scottmg
lgtm
4 years, 7 months ago (2016-05-26 00:14:58 UTC) #5
commit-bot: I haz the power
Dry run: None
4 years, 7 months ago (2016-05-26 00:47:46 UTC) #9
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/2014753002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/2014753002/1
4 years, 7 months ago (2016-05-26 00:48:33 UTC) #10
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 7 months ago (2016-05-26 01:12:07 UTC) #12
commit-bot: I haz the power
4 years, 7 months ago (2016-05-26 01:14:50 UTC) #14
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/c25693b3002a6a9a8c074cf9460ae3dca262bbaf
Cr-Commit-Position: refs/heads/master@{#396071}

Powered by Google App Engine
This is Rietveld 408576698