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

Issue 5928005: Fixes to get ctest up and running again. (Closed)

Created:
10 years ago by sosa
Modified:
9 years, 6 months ago
Reviewers:
dgarrett
CC:
chromium-os-reviews_chromium.org, Mandeep Singh Baines, anush, sosa
Visibility:
Public.

Description

Fixes to get ctest up and running again. We had a major issue because the latest dev channel release pre-dates the virtio change. Because of this, ctest was connecting to update that channel using virtio (which is unsupported for that). This change drops a file telling the vm lib to use e1000 as before. Also, fixed stateful change logic. Change-Id: I394a8cece71bdd0d55efa21ba5b6d24804432c6b BUG=chromium-os:10434 TEST=Ran it with ctest and cros_au_test_harness on images with/without the file. Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=40fa0de

Patch Set 1 #

Patch Set 2 : Nits #

Patch Set 3 : No image to live #

Patch Set 4 : no verbose #

Unified diffs Side-by-side diffs Delta from patch set Stats (+43 lines, -19 lines) Patch
M bin/cros_au_test_harness.py View 1 10 chunks +28 lines, -18 lines 0 comments Download
M bin/ctest.py View 1 2 3 2 chunks +8 lines, -0 lines 0 comments Download
M lib/cros_vm_lib.sh View 1 chunk +7 lines, -1 line 0 comments Download

Messages

Total messages: 2 (0 generated)
dgarrett
LGTM. Is there some way to make eclipse more aggressive about cleaning up white space ...
10 years ago (2010-12-17 21:08:28 UTC) #1
sosa
10 years ago (2010-12-17 21:12:29 UTC) #2
If you go into window->preferences->pydev->editor->code style ...
there are a lot of options here (especially in code formatter).  You
could even select to auto format when saving ... though I usually just
explicitly use it (ctrl+shift+f)

-Sosa

On Fri, Dec 17, 2010 at 1:08 PM,  <dgarrett@chromium.org> wrote:
> LGTM.
>
> Is there some way to make eclipse more aggressive about cleaning up white
> space
> like this?
>
> http://codereview.chromium.org/5928005/
>

Powered by Google App Engine
This is Rietveld 408576698