Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: libadblockplus-android/build.gradle

Issue 29557565: Issue 5800 - Add option to specify a list of V8 libraries (Closed)
Patch Set: moved 'packagingOptions' section to dynamic linking 'if' branch only Created Nov. 2, 2017, 11:30 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « README.md ('k') | libadblockplus-android/jni/Android.mk » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: libadblockplus-android/build.gradle
diff --git a/libadblockplus-android/build.gradle b/libadblockplus-android/build.gradle
index 069c44cf739f2aac8e3843c5d92bd7b4f7ee2e61..f8159101066630b283980377f5cec97a7ae7356e 100644
--- a/libadblockplus-android/build.gradle
+++ b/libadblockplus-android/build.gradle
@@ -29,4 +29,21 @@ android {
jni.srcDirs = ['jni']
}
}
+
+ def sharedV8LibFiles = System.getenv('SHARED_V8_LIB_FILENAMES')
+ if (sharedV8LibFiles != null) {
+ def sharedV8LibFilesSet = []
+ def sharedV8LibFilesArray = sharedV8LibFiles.split(' ')
+ sharedV8LibFilesArray.each { eachFileName ->
+ sharedV8LibFilesSet.add("**/" + eachFileName)
+ println "[Configuration] Excluding shared v8 library " + eachFileName + " from AAR"
+ }
+
+ packagingOptions {
+ excludes = sharedV8LibFilesSet
diegocarloslima 2017/11/08 13:25:55 I think we weren't in the same page in my last com
anton 2017/11/08 13:28:25 It's equal to what we had before last patch set. I
+ }
+ } else {
+ println "[Configuration] No shared v8 libraries excluded from AAR"
+ }
+
}
« no previous file with comments | « README.md ('k') | libadblockplus-android/jni/Android.mk » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld