Skip to content

Commit b4017a9

Browse files
dulmandakhfacebook-github-bot
authored andcommittedFeb 20, 2019
bump gradle to 5.0, android gradle plugin to 3.3.1 (#23324)
Summary: This PR bumps gradle to 5.0, which includes Kotlin DSL 1.0, and android gradle plugin to 3.3.1, which includes includes various bug fixes and performance improvements. Also Gradle 5.x requires Java 8. This is preparation for Kotlin DSL migration. [Android] [Changed] - Bump Gradle to 5.0 Pull Request resolved: #23324 Reviewed By: mdvacca Differential Revision: D14028563 Pulled By: hramos fbshipit-source-id: 61fe1a2d4ea5707d6f07945acbd950f852420e13
1 parent 19866ae commit b4017a9

File tree

6 files changed

+4
-6
lines changed

6 files changed

+4
-6
lines changed
 

‎build.gradle

+1-1
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ buildscript {
1010
jcenter()
1111
}
1212
dependencies {
13-
classpath("com.android.tools.build:gradle:3.3.1")
13+
classpath("com.android.tools.build:gradle:3.3.0")
1414
classpath("de.undercouch:gradle-download-task:3.4.3")
1515

1616
// NOTE: Do not place your application dependencies here; they belong

‎gradle/wrapper/gradle-wrapper.jar

0 Bytes
Binary file not shown.
+1-3
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,5 @@
1-
#Thu Jan 24 16:45:50 BRST 2019
21
distributionBase=GRADLE_USER_HOME
32
distributionPath=wrapper/dists
3+
distributionUrl=https\://services.gradle.org/distributions/gradle-5.0-all.zip
44
zipStoreBase=GRADLE_USER_HOME
55
zipStorePath=wrapper/dists
6-
distributionUrl=https\://services.gradle.org/distributions/gradle-4.10.2-all.zip
7-
android.debug.obsoleteApi=true

‎template/android/build.gradle

+1-1
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ buildscript {
1313
jcenter()
1414
}
1515
dependencies {
16-
classpath 'com.android.tools.build:gradle:3.3.1'
16+
classpath 'com.android.tools.build:gradle:3.3.0'
1717

1818
// NOTE: Do not place your application dependencies here; they belong
1919
// in the individual module build.gradle files
Binary file not shown.
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
distributionBase=GRADLE_USER_HOME
22
distributionPath=wrapper/dists
3+
distributionUrl=https\://services.gradle.org/distributions/gradle-5.0-all.zip
34
zipStoreBase=GRADLE_USER_HOME
45
zipStorePath=wrapper/dists
5-
distributionUrl=https\://services.gradle.org/distributions/gradle-4.10.2-all.zip

3 commit comments

Comments
 (3)

radeno commented on Feb 20, 2019

@radeno
Contributor

@dulmandakh @hramos guys, how facebook-github-bot works?
Committed code is different than referenced PR. It doesn't take latest changes but earlier.

hramos commented on Feb 20, 2019

@hramos
Contributor

I modified the diff after importing it. There’s several changes we need to make before we can land any Gradle related change. The PR initially bumped the Gradle wrapper, and it looks like the bot is referencing the old PR summary in the commit message, which is incorrect.

radeno commented on Feb 21, 2019

@radeno
Contributor

@hramos so we need re-apply this PR ? #23473 it uses android-gradle 3.3.1 because bugfixes and performance increase.

Please sign in to comment.