Merge lp://qastaging/~gesha/linaro-license-protection/improve-eula into lp://qastaging/~linaro-automation/linaro-license-protection/trunk

Proposed by Georgy Redkozubov
Status: Merged
Approved by: James Tunnicliffe
Approved revision: 52
Merged at revision: 54
Proposed branch: lp://qastaging/~gesha/linaro-license-protection/improve-eula
Merge into: lp://qastaging/~linaro-automation/linaro-license-protection/trunk
Diff against target: 953 lines (+398/-350)
17 files modified
.htaccess (+70/-44)
android/~linaro-android/staging-imx53/test.txt (+1/-0)
android/~linaro-android/staging-origen/EULA.txt (+1/-0)
android/~linaro-android/staging-panda/EULA.txt (+1/-0)
android/~linaro-android/staging-panda/test.txt (+1/-1)
android/~linaro-android/staging-snowball/173/target/product/snowball/EULA.txt (+1/-0)
android/~linaro-android/staging-snowball/173/target/product/snowball/test.txt (+1/-0)
android/~linaro-android/staging-snowball/test.txt (+0/-1)
android/~linaro-android/staging-vexpress-a9/OPEN-EULA.txt (+1/-0)
android/~linaro-android/staging-vexpress-a9/test.txt (+1/-0)
licenses/license.php (+25/-0)
licenses/linaro.html (+120/-0)
licenses/samsung.html (+120/-0)
licenses/ste.html (+6/-298)
testing/apache2.conf.tmpl (+13/-0)
testing/mime.types (+6/-1)
testing/test_click_through_license.py (+30/-5)
To merge this branch: bzr merge lp://qastaging/~gesha/linaro-license-protection/improve-eula
Reviewer Review Type Date Requested Status
James Tunnicliffe (community) Approve
Review via email: mp+100379@code.qastaging.launchpad.net

Description of the change

This branch adds build-based EULA support. EULA/no-EULA handling is implemented by checking the presence of EULA.txt/OPEN-EULA.txt in build directory. If EULA.txt is present then the contents of EULA.txt is shown to the user prior letting him download protected artifacts. If OPEN-EULA.txt is found it means that user doesn't need to accept EULA and proceeds to download immediately. Otherwise "HTTP Error 403 Forbidden" is displayed.

To post a comment you must log in.
Revision history for this message
James Tunnicliffe (dooferlad) wrote :

This looks fine and should work with the scripts that we have to automatically accept licenses.

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