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

Issue 2852001: Prepare vboot_reference ebuild to work with the new makefile and build flow. (Closed)

Created:
10 years, 6 months ago by vb
Modified:
9 years, 6 months ago
Reviewers:
Bill Richardson
CC:
chromium-os-reviews_chromium.org, Mandeep Singh Baines
Base URL:
ssh://git@chromiumos-git/chromiumos-overlay.git
Visibility:
Public.

Description

Prepare vboot_reference ebuild to work with the new makefile and build flow. It works for local build only, will address pulling source with the git after http://codereview.chromium.org/2845001 has been submitted.

Patch Set 1 : Prepare vboot_reference ebuild to work with the new makefile and build flow. #

Total comments: 2

Patch Set 2 : Modified based on the review comments #

Unified diffs Side-by-side diffs Delta from patch set Stats (+42 lines, -18 lines) Patch
M chromeos-base/vboot_reference/vboot_reference-1.0.ebuild View 1 2 chunks +8 lines, -18 lines 0 comments Download
A chromeos-base/vboot_reference/vboot_reference-9999.ebuild View 1 1 chunk +34 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
vb
10 years, 6 months ago (2010-06-14 22:15:46 UTC) #1
Bill Richardson
http://codereview.chromium.org/2852001/diff/6001/7001 File chromeos-base/vboot_reference/vboot_reference-1.0.ebuild (right): http://codereview.chromium.org/2852001/diff/6001/7001#newcode22 chromeos-base/vboot_reference/vboot_reference-1.0.ebuild:22: "Try running 'make clean' in the package root directory" ...
10 years, 6 months ago (2010-06-14 22:26:43 UTC) #2
vb
http://codereview.chromium.org/2852001/diff/6001/7001 File chromeos-base/vboot_reference/vboot_reference-1.0.ebuild (right): http://codereview.chromium.org/2852001/diff/6001/7001#newcode22 chromeos-base/vboot_reference/vboot_reference-1.0.ebuild:22: "Try running 'make clean' in the package root directory" ...
10 years, 6 months ago (2010-06-14 22:43:36 UTC) #3
Bill Richardson
10 years, 6 months ago (2010-06-14 22:47:19 UTC) #4
Okay. LGTM, then.

Thanks.

Powered by Google App Engine
This is Rietveld 408576698