LAVA Android Test

Merge lp://qastaging/~liuyq0307/lava-android-test/fix-1012573 into lp://qastaging/lava-android-test

  1. fix-1012573
  2. Merge into trunk
Proposed by Yongqin Liu
Status: Merged
Approved by: Michael Hudson-Doyle
Approved revision: 165
Merged at revision: 165
Proposed branch: lp://qastaging/~liuyq0307/lava-android-test/fix-1012573
Merge into: lp://qastaging/lava-android-test
Diff against target: 12 lines (+1/-1)
1 file modified
lava_android_test/swprofile.py (+1/-1)
To merge this branch: bzr merge lp://qastaging/~liuyq0307/lava-android-test/fix-1012573
Reviewer Review Type Date Requested Status
Michael Hudson-Doyle (community) Approve
Review via email: mp+110015@code.qastaging.launchpad.net

Description of the change

return empty string when can not get the ro.build.display.id property

To post a comment you must log in.
Revision history for this message
Andy Doan (doanac) wrote :

you can do this easier with just:

  return props.get('ro.build.display.id', '')

lp://qastaging/~liuyq0307/lava-android-test/fix-1012573 updated
165. By Yongqin Liu

modify according to the review comment

Revision history for this message
Yongqin Liu (liuyq0307) wrote :

> you can do this easier with just:
>
> return props.get('ro.build.display.id', '')

Ah, yeah, thanks.
updated.

Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :

Looks sane to me.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches