Merge lp:~asac/linaro-android-build-tools/add-glmark2-to-default-test-plan into lp:linaro-android-build-tools

Proposed by Alexander Sack
Status: Merged
Merged at revision: 417
Proposed branch: lp:~asac/linaro-android-build-tools/add-glmark2-to-default-test-plan
Merge into: lp:linaro-android-build-tools
Diff against target: 12 lines (+1/-1)
1 file modified
build-scripts/post-build-lava.py (+1/-1)
To merge this branch: bzr merge lp:~asac/linaro-android-build-tools/add-glmark2-to-default-test-plan
Reviewer Review Type Date Requested Status
Paul Sokolovsky Approve
Review via email: mp+89920@code.launchpad.net

Description of the change

Let's add glmark2 to the default test plan.

To post a comment you must log in.
Revision history for this message
Paul Sokolovsky (pfalcon) wrote :

I'd say there's something wrong with Launchpad's active-reviews page. I watch my personal review page: https://code.launchpad.net/~pfalcon/+activereviews and it really should include reviews for any teams I'm on! End result: I missed this previously. Looks good.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'build-scripts/post-build-lava.py'
2--- build-scripts/post-build-lava.py 2011-12-16 13:57:21 +0000
3+++ build-scripts/post-build-lava.py 2012-01-24 15:36:04 +0000
4@@ -88,7 +88,7 @@
5
6 test_plan = os.environ.get("LAVA_TEST_PLAN")
7 if test_plan == None:
8- test_plan = '0xbench, monkey'
9+ test_plan = '0xbench, glmark2, monkey'
10 test_plans = test_plan.split(',')
11 for index in range(len(test_plans)):
12 test_plans[index] = test_plans[index].strip()

Subscribers

People subscribed via source and target branches