Merge lp://qastaging/~pundiramit/linaro-android-build-tools/user-script into lp://qastaging/linaro-android-build-tools

Proposed by Amit Pundir
Status: Merged
Approved by: Данило Шеган
Approved revision: 544
Merged at revision: 544
Proposed branch: lp://qastaging/~pundiramit/linaro-android-build-tools/user-script
Merge into: lp://qastaging/linaro-android-build-tools
Diff against target: 68 lines (+15/-15)
1 file modified
build-scripts/create-user-build-script (+15/-15)
To merge this branch: bzr merge lp://qastaging/~pundiramit/linaro-android-build-tools/user-script
Reviewer Review Type Date Requested Status
Данило Шеган (community) Approve
Paul Sokolovsky Pending
Review via email: mp+140720@code.qastaging.launchpad.net

Description of the change

Last patches did not take care of escaping shell variables, which resulted in broken shell script e.g. https://snapshots.linaro.org/android/~linaro-android-restricted/vexpress-tc2-isw-jb-gcc47-armlt-stable-open-linaro-android-12.12-iks-release/4/linaro_android_build_cmds.sh. Fixing that.

To post a comment you must log in.
Revision history for this message
Данило Шеган (danilo) wrote :

Looks good, thanks.

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