Merge lp://staging/~deeptik/linaro-android-build-tools/source_overlay_check into lp://staging/linaro-android-build-tools

Proposed by Deepti B. Kalakeri
Status: Merged
Approved by: Georgy Redkozubov
Approved revision: 543
Merged at revision: 545
Proposed branch: lp://staging/~deeptik/linaro-android-build-tools/source_overlay_check
Merge into: lp://staging/linaro-android-build-tools
Diff against target: 16 lines (+6/-0)
1 file modified
build-scripts/build-android (+6/-0)
To merge this branch: bzr merge lp://staging/~deeptik/linaro-android-build-tools/source_overlay_check
Reviewer Review Type Date Requested Status
Georgy Redkozubov Approve
Milo Casagrande (community) Approve
Review via email: mp+140632@code.staging.launchpad.net

Description of the change

Adding check for presence of SOURCE_OVERLAY in the configuration for the android builds.
With this check introduced the builds should exit if the SOURCE_OVERLAY is not part of the configuration and should pass to completion otherwise. Here are the following builds which check the SOURCE_OVERLAY check for the behaviour respectively for failure on non-exisitence of SOURCE_OVERLAY and success when it exists.
https://ec2-23-20-244-116.compute-1.amazonaws.com/jenkins/job/deepti_without_source_overlay_galaxynexus-jb-gcc44-aosp-stable-blob/1/console
https://ec2-23-20-244-116.compute-1.amazonaws.com/jenkins/job/deepti_linaro-android_panda-ics-gcc47-tilt-tracking-blob/1/console

To post a comment you must log in.
Revision history for this message
Milo Casagrande (milo) wrote :

Looks good to me.
+1

review: Approve
Revision history for this message
Georgy Redkozubov (gesha) wrote :

Looks good

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