Skip to content
This repository has been archived by the owner on Feb 22, 2023. It is now read-only.

Commit

Permalink
[multiple] Java 8 target for all plugins with -Werror compiler arg (#…
Browse files Browse the repository at this point in the history
  • Loading branch information
brunobowden authored Sep 15, 2021
1 parent edc899e commit bb3ebb2
Show file tree
Hide file tree
Showing 13 changed files with 34 additions and 28 deletions.
8 changes: 4 additions & 4 deletions packages/android_alarm_manager/android/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -28,10 +28,6 @@ apply plugin: 'com.android.library'

android {
compileSdkVersion 29
compileOptions {
sourceCompatibility JavaVersion.VERSION_1_8
targetCompatibility JavaVersion.VERSION_1_8
}
defaultConfig {
minSdkVersion 16
testInstrumentationRunner "androidx.test.runner.AndroidJUnitRunner"
Expand All @@ -54,6 +50,10 @@ android {
}
}
}
compileOptions {
sourceCompatibility JavaVersion.VERSION_1_8
targetCompatibility JavaVersion.VERSION_1_8
}
}

dependencies {
Expand Down
1 change: 1 addition & 0 deletions packages/android_intent/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@

* Remove references to the V1 Android embedding.
* Updated Android lint settings.
* Specify Java 8 for Android build.

## 2.0.2

Expand Down
6 changes: 4 additions & 2 deletions packages/android_intent/android/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -37,8 +37,10 @@ android {
disable 'InvalidPackage'
disable 'GradleDependency'
}


compileOptions {
sourceCompatibility JavaVersion.VERSION_1_8
targetCompatibility JavaVersion.VERSION_1_8
}
testOptions {
unitTests.includeAndroidResources = true
unitTests.returnDefaultValues = true
Expand Down
4 changes: 2 additions & 2 deletions packages/camera/camera/android/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -39,8 +39,8 @@ android {
baseline file("lint-baseline.xml")
}
compileOptions {
sourceCompatibility = '1.8'
targetCompatibility = '1.8'
sourceCompatibility JavaVersion.VERSION_1_8
targetCompatibility JavaVersion.VERSION_1_8
}


Expand Down
1 change: 1 addition & 0 deletions packages/connectivity/connectivity/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@

* Remove references to the Android V1 embedding.
* Updated Android lint settings.
* Specify Java 8 for Android build.

## 3.0.6

Expand Down
4 changes: 4 additions & 0 deletions packages/connectivity/connectivity/android/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -50,4 +50,8 @@ android {
}
}
}
compileOptions {
sourceCompatibility JavaVersion.VERSION_1_8
targetCompatibility JavaVersion.VERSION_1_8
}
}
3 changes: 2 additions & 1 deletion packages/path_provider/path_provider/CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
## NEXT
## 2.0.4

* Updated Android lint settings.
* Specify Java 8 for Android build.

## 2.0.3

Expand Down
8 changes: 3 additions & 5 deletions packages/path_provider/path_provider/android/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -32,11 +32,9 @@ android {
disable 'InvalidPackage'
disable 'GradleDependency'
}
android {
compileOptions {
sourceCompatibility 1.8
targetCompatibility 1.8
}
compileOptions {
sourceCompatibility JavaVersion.VERSION_1_8
targetCompatibility JavaVersion.VERSION_1_8
}


Expand Down
2 changes: 1 addition & 1 deletion packages/path_provider/path_provider/pubspec.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ name: path_provider
description: Flutter plugin for getting commonly used locations on host platform file systems, such as the temp and app data directories.
repository: https://github.com/flutter/plugins/tree/master/packages/path_provider/path_provider
issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+label%3A%22p%3A+path_provider%22
version: 2.0.3
version: 2.0.4

environment:
sdk: ">=2.12.0 <3.0.0"
Expand Down
4 changes: 4 additions & 0 deletions packages/video_player/video_player/CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,7 @@
## 2.2.1

* Specify Java 8 for Android build.

## 2.2.0

* Add `contentUri` based VideoPlayerController.
Expand Down
8 changes: 3 additions & 5 deletions packages/video_player/video_player/android/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -37,11 +37,9 @@ android {
disable 'InvalidPackage'
disable 'GradleDependency'
}
android {
compileOptions {
sourceCompatibility 1.8
targetCompatibility 1.8
}
compileOptions {
sourceCompatibility JavaVersion.VERSION_1_8
targetCompatibility JavaVersion.VERSION_1_8
}

dependencies {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,10 @@ android {
lintOptions {
disable 'InvalidPackage'
}
compileOptions {
sourceCompatibility JavaVersion.VERSION_1_8
targetCompatibility JavaVersion.VERSION_1_8
}

defaultConfig {
applicationId "io.flutter.plugins.videoplayerexample"
Expand All @@ -40,13 +44,6 @@ android {
testInstrumentationRunner "androidx.test.runner.AndroidJUnitRunner"
}

android {
compileOptions {
sourceCompatibility 1.8
targetCompatibility 1.8
}
}

buildTypes {
release {
// TODO: Add your own signing config for the release build.
Expand Down
2 changes: 1 addition & 1 deletion packages/video_player/video_player/pubspec.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ description: Flutter plugin for displaying inline video with other Flutter
widgets on Android, iOS, and web.
repository: https://github.com/flutter/plugins/tree/master/packages/video_player/video_player
issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+label%3A%22p%3A+video_player%22
version: 2.2.0
version: 2.2.1

environment:
sdk: ">=2.12.0 <3.0.0"
Expand Down

0 comments on commit bb3ebb2

Please sign in to comment.