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

Issue 2825021: build_image: pull out kernel partition creation (Closed)

Created:
10 years, 6 months ago by Will Drewry
Modified:
9 years, 7 months ago
Reviewers:
Bill Richardson, adlr
CC:
chromium-os-reviews_chromium.org
Base URL:
ssh://git@gitrw.chromium.org//crosutils.git
Visibility:
Public.

Description

build_image: pull out kernel partition creation Makes kernel partition creation standalone. This is motivated both by the ability to build test kernel partitions easily as well the need to create all kernel command line configuration after the rootfs has been completely created. Instead of a massive overhaul, I'll do this refactor in pieces. TEST=manually rebuilt the image BUG=chromium-os:327

Patch Set 1 #

Total comments: 2

Patch Set 2 : add keep_work and note when we are using non-"to" output #

Unified diffs Side-by-side diffs Delta from patch set Stats (+139 lines, -67 lines) Patch
M build_image View 1 2 chunks +13 lines, -67 lines 0 comments Download
A build_kernel_image.sh View 1 1 chunk +126 lines, -0 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
Will Drewry
10 years, 6 months ago (2010-06-24 17:56:44 UTC) #1
Bill Richardson
One nit, but LGTM. http://codereview.chromium.org/2825021/diff/1/2 File build_image (right): http://codereview.chromium.org/2825021/diff/1/2#newcode542 build_image:542: load_kernel_test "${OUTPUT_IMG}" "${OUTPUT_DIR}/key_alg8.vbpubk" This uses ...
10 years, 6 months ago (2010-06-24 18:00:05 UTC) #2
adlr
LGTM if you address Bill's comment
10 years, 6 months ago (2010-06-24 18:07:33 UTC) #3
Will Drewry
Thanks for the quick review. I'm running build_image now... http://codereview.chromium.org/2825021/diff/1/2 File build_image (right): http://codereview.chromium.org/2825021/diff/1/2#newcode542 build_image:542: ...
10 years, 6 months ago (2010-06-24 18:16:26 UTC) #4
Bill Richardson
Yeah, much better. Thanks. LGTM
10 years, 6 months ago (2010-06-24 18:35:41 UTC) #5
Will Drewry
10 years, 6 months ago (2010-06-24 19:27:51 UTC) #6
On 2010/06/24 18:35:41, Bill Richardson wrote:
> Yeah, much better. Thanks.
> 
> 
> LGTM

Built fine - testing a boot now before pushing

Powered by Google App Engine
This is Rietveld 408576698