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

Issue 521073: Use local apt cache instead of local_repo when mastering an image. (Closed)

Created:
10 years, 11 months ago by tedbo
Modified:
9 years, 7 months ago
Reviewers:
Bill Richardson
CC:
chromium-os-reviews_googlegroups.com
Visibility:
Public.

Description

Use local apt cache instead of local_repo when mastering an image. This change switches to mastering an image without using debootstrap. We turn on the previously experimental bits that install a small set of packages manually before handing things over to apt. In both cases we use apt to download the packages so that it can populate it's local package cache. With this change we will no longer depend on the local_repo when mastering an image. Developers will not have to rebuild their local repo when repo_list_image.txt changes. Instead we will use and lazy-fill the local apt-cache. The first time you build_image.sh it will be slow since it needs to download the packages. Subsequent runs should be as before since it will use the local cache. If packages are added they should be lazily fetched in the next image build. Until we have a fully populated external mirror, developers will still have to add packages to repo_list_image.txt. Also, until make_chroot is switched over to use the external repo then developers will have to redo their local repo when the repo_list_dev.txt changes.

Patch Set 1 #

Patch Set 2 : Add comment, remove unused var. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+96 lines, -112 lines) Patch
M src/scripts/build_image.sh View 1 chunk +1 line, -1 line 0 comments Download
M src/scripts/common.sh View 4 chunks +5 lines, -5 lines 0 comments Download
M src/scripts/install_packages.sh View 1 2 chunks +90 lines, -106 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
tedbo
10 years, 11 months ago (2010-01-08 05:25:41 UTC) #1
Bill Richardson
10 years, 11 months ago (2010-01-08 16:05:09 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld 408576698