New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 670518 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jul 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug



Sign in to add a comment

Update all of the "get the code" and build documents

Project Member Reported by dpranke@chromium.org, Dec 2 2016

Issue description

This is a tracking bug to make sure all of the public and internal "get the code" pages and "build instructions" have been cleaned up and are up-to-date.

Currently, the main public instructions have been updated, but the Cast, ChromeOS, and Chromoting instructions still need to be updated.

I need to do another pass to sync up the internal instructions, and then figure out how to update the internal clank and bling instructions, since they have their own sites.
 
Cc: sdy@chromium.org
Project Member

Comment 2 by bugdroid1@chromium.org, Dec 2 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/27ea3ce48ed67b758957d09c822af573796049c3

commit 27ea3ce48ed67b758957d09c822af573796049c3
Author: dpranke <dpranke@chromium.org>
Date: Fri Dec 02 01:46:10 2016

Remove links to old build instructions.

It looks like the new instructions are reasonably good, so I think
we can remove the old ones now.

R=thakis@chromium.org
BUG= 670518 

Review-Url: https://codereview.chromium.org/2544963002
Cr-Commit-Position: refs/heads/master@{#435817}

[modify] https://crrev.com/27ea3ce48ed67b758957d09c822af573796049c3/docs/android_build_instructions.md
[modify] https://crrev.com/27ea3ce48ed67b758957d09c822af573796049c3/docs/ios_build_instructions.md
[modify] https://crrev.com/27ea3ce48ed67b758957d09c822af573796049c3/docs/linux_build_instructions.md
[modify] https://crrev.com/27ea3ce48ed67b758957d09c822af573796049c3/docs/mac_build_instructions.md
[delete] https://crrev.com/d0fd8325756269c1312e9ccb93ad6047d39d80f3/docs/old_android_build_instructions.md
[delete] https://crrev.com/d0fd8325756269c1312e9ccb93ad6047d39d80f3/docs/old_ios_build_instructions.md
[delete] https://crrev.com/d0fd8325756269c1312e9ccb93ad6047d39d80f3/docs/old_linux_build_instructions.md
[delete] https://crrev.com/d0fd8325756269c1312e9ccb93ad6047d39d80f3/docs/old_mac_build_instructions.md
[delete] https://crrev.com/d0fd8325756269c1312e9ccb93ad6047d39d80f3/docs/old_windows_build_instructions.md
[modify] https://crrev.com/27ea3ce48ed67b758957d09c822af573796049c3/docs/windows_build_instructions.md

Project Member

Comment 3 by bugdroid1@chromium.org, Dec 2 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/2989a785779298a49bc6024ec421c92884e181d7

commit 2989a785779298a49bc6024ec421c92884e181d7
Author: dpranke <dpranke@chromium.org>
Date: Fri Dec 02 02:57:12 2016

More edits to the linux build instructions.

This change folds the notes on building on the different Linux distros
into the main Linux build instructions. It also moves the "faster
builds" information into the same page, adds one note for
Debian Android builds.

R=thomasanderson@chromium.org
BUG= 670518 

Review-Url: https://codereview.chromium.org/2544793004
Cr-Commit-Position: refs/heads/master@{#435835}

[modify] https://crrev.com/2989a785779298a49bc6024ec421c92884e181d7/docs/android_build_instructions.md
[delete] https://crrev.com/23a4b06b83fe4b11f6d5a48078eb543ea88c747b/docs/linux_arch_build_instructions.md
[modify] https://crrev.com/2989a785779298a49bc6024ec421c92884e181d7/docs/linux_build_instructions.md
[delete] https://crrev.com/23a4b06b83fe4b11f6d5a48078eb543ea88c747b/docs/linux_debian_build_instructions.md
[delete] https://crrev.com/23a4b06b83fe4b11f6d5a48078eb543ea88c747b/docs/linux_faster_builds.md
[delete] https://crrev.com/23a4b06b83fe4b11f6d5a48078eb543ea88c747b/docs/linux_fedora_build_instructions.md
[delete] https://crrev.com/23a4b06b83fe4b11f6d5a48078eb543ea88c747b/docs/linux_mandriva_build_instructions.md
[delete] https://crrev.com/23a4b06b83fe4b11f6d5a48078eb543ea88c747b/docs/linux_open_suse_build_instructions.md

Project Member

Comment 4 by bugdroid1@chromium.org, Dec 5 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/821cd0366b7c93a883bd3b0a91c3ca387c46decd

commit 821cd0366b7c93a883bd3b0a91c3ca387c46decd
Author: dpranke <dpranke@chromium.org>
Date: Mon Dec 05 02:55:53 2016

Remove the docs for incremental linking on Windows.

Most if not all of this information is out of date, as incremental
linking is now handled more-or-less automatically by the build
and you probably shouldn't mess with the settings.

R=brucedawson@chromium.org
BUG= 670518 

Review-Url: https://codereview.chromium.org/2549443004
Cr-Commit-Position: refs/heads/master@{#436220}

[delete] https://crrev.com/82b4044552250f720fffe38be2685546ba5a6617/docs/windows_incremental_linking.md

Project Member

Comment 5 by bugdroid1@chromium.org, Dec 8 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/98888af6ae9dbce030da5e37d694f513533fe1a6

commit 98888af6ae9dbce030da5e37d694f513533fe1a6
Author: thomasanderson <thomasanderson@google.com>
Date: Thu Dec 08 06:58:50 2016

Add lib32z1 to install-build-deps-android.sh

This CL removes the instructions to install lib32z1 from
android_build-instructions.md, and instead adds lib32z1 to
install-build-deps-android.sh.  This package is available in
the default repos in all versions of Ubuntu and Debian.

BUG= 670518 
R=dpranke@chromium.org

Review-Url: https://codereview.chromium.org/2547253002
Cr-Commit-Position: refs/heads/master@{#437201}

[modify] https://crrev.com/98888af6ae9dbce030da5e37d694f513533fe1a6/build/install-build-deps-android.sh
[modify] https://crrev.com/98888af6ae9dbce030da5e37d694f513533fe1a6/docs/android_build_instructions.md

Status: Started (was: Assigned)
Labels: -Pri-1 Pri-2
Most of the docs have been updated. I'm downgrading this to P2 for the rest of the work.
Status: Fixed (was: Started)
All of the public docs are up-to-date, actually. I'll file a separate bug for the internal docs.

Sign in to add a comment