Merge lp://staging/~fagerstedt-axel/linaro-android-build-tools/build-android-testsuite-restricted into lp://staging/linaro-android-build-tools

Proposed by Axel Fagerstedt
Status: Merged
Approved by: Georgy Redkozubov
Approved revision: 626
Merged at revision: 632
Proposed branch: lp://staging/~fagerstedt-axel/linaro-android-build-tools/build-android-testsuite-restricted
Merge into: lp://staging/linaro-android-build-tools
Diff against target: 216 lines (+195/-1)
3 files modified
build-scripts/build-android-testsuite-restricted (+110/-0)
build-scripts/fetch_prebuilt.py (+83/-0)
node/prepare_build_config.py (+2/-1)
To merge this branch: bzr merge lp://staging/~fagerstedt-axel/linaro-android-build-tools/build-android-testsuite-restricted
Reviewer Review Type Date Requested Status
Georgy Redkozubov Approve
Review via email: mp+167126@code.staging.launchpad.net

Description of the change

Adds a new build type which builds stand alone testsuites and bundles them with prebuilt android images fetched from snapshots.

The build script assumes that the specified repo manifest will contain a directory "build" with a makefile providing the targets build and install. It further assumes that making the two targets will produce a directory "out" containing the directories "data" and "system" with any build output in a android style directory structure.

The changes have been tested with the job: https://android-build.linaro.org/builds/~linaro-android-restricted/test-private-test-suite

To post a comment you must log in.
Revision history for this message
Georgy Redkozubov (gesha) wrote :

In general looks good.

One thing I would like to be tested is that out/ dir exists and 'thw' I guess is a mistype:

74 + # move thw boot tarball
75 + mv boot.tar.bz2 out/.
76 +
77 + # we are assuming that the makefile has given us a out/ structure
78 + # containing data/ and system/ with the test files

626. By Axel Fagerstedt

build-android-testsuite: Verify that out has been created.

Revision history for this message
Axel Fagerstedt (fagerstedt-axel) wrote :

Updated.

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