[android] Update to Gradle 7.0.0 alpha #198

rtsisyk · 2021-02-28T06:58:05Z

rtsisyk commented

2021-02-28 06:58:05 +00:00

Fixes compilation on Apple Silicon.

Signed-off-by: Roman Tsisyk roman@tsisyk.com

Fixes compilation on Apple Silicon. Signed-off-by: Roman Tsisyk <roman@tsisyk.com>

vng (Migrated from github.com) requested changes 2021-02-28 12:00:05 +00:00

vng (Migrated from github.com) left a comment

Doesn't work with current stable Android Studio. Do not merge ..

biodranik (Migrated from github.com) reviewed 2021-02-28 13:01:04 +00:00

biodranik (Migrated from github.com) left a comment

LGTM с комментариями

android/build.gradle

					
				@ -6,7 +6,7 @@ buildscript {

				  }

				  dependencies {

biodranik (Migrated from github.com) commented

2021-02-28 12:54:14 +00:00

Это не выпиливается?

android/flavors/debug/AndroidManifest.xml

					
				@ -3,3 +3,2 @@

				    xmlns:tools="http://schemas.android.com/tools"

				    tools:remove="sharedUserId, sharedUserLabel">

				    xmlns:tools="http://schemas.android.com/tools">

				  <uses-permission android:name="android.permission.READ_LOGS"/>

biodranik (Migrated from github.com) commented

2021-02-28 12:58:07 +00:00

Зачем убирать? Может потом наши другие аппы будут взаимодействовать.

android/gradle.properties

					
				@ -26,4 +26,3 @@ org.gradle.caching=true

				org.gradle.jvmargs=-Xmx8g

				android.useAndroidX=true

				android.enableJetifier=true

				android.enableR8=false

biodranik (Migrated from github.com) commented

2021-02-28 13:00:12 +00:00

Почему было выключено? В истории коммитов что написано?

biodranik commented

2021-02-28 13:03:58 +00:00

(Migrated from github.com)

С таким подходом удобнее накидывать коммиты сверху, а потом их сквошнуть в 1 при мерже

rtsisyk reviewed 2021-03-03 08:36:42 +00:00

android/build.gradle

					
				@ -6,7 +6,7 @@ buildscript {

				  }

				  dependencies {

rtsisyk commented

2021-03-03 08:36:41 +00:00

Пока нет. Давайте отдельно это упражнение проведем.

rtsisyk reviewed 2021-03-03 08:37:22 +00:00

android/flavors/debug/AndroidManifest.xml

					
				@ -3,3 +3,2 @@

				    xmlns:tools="http://schemas.android.com/tools"

				    tools:remove="sharedUserId, sharedUserLabel">

				    xmlns:tools="http://schemas.android.com/tools">

				  <uses-permission android:name="android.permission.READ_LOGS"/>

rtsisyk commented

2021-03-03 08:37:22 +00:00

Проблема в следующем. Мы используем разные сертификаты для подписи дебага, беты и релиза в сторе. С разными сертификатами эта фича толком не работает.

rtsisyk reviewed 2021-03-03 08:39:31 +00:00

android/gradle.properties

					
				@ -26,4 +26,3 @@ org.gradle.caching=true

				org.gradle.jvmargs=-Xmx8g

				android.useAndroidX=true

				android.enableJetifier=true

				android.enableR8=false

rtsisyk commented

2021-03-03 08:39:31 +00:00

Смотри 2c6fb9a185
Поразбираюсь что это такое.

Смотри 2c6fb9a1853726ccc2d7a81e4aacf49d8b3961dc Поразбираюсь что это такое.

biodranik commented

2021-03-03 09:08:01 +00:00

(Migrated from github.com)

Надо разобраться и вернуть.
Это нормально, что сертификаты разные для разных версий. Но шаред надо оставить. Какие именно у тебя ошибки?

On 3 Mar 2021, at 09:37, Roman Tsisyk notifications@github.com wrote:

@rtsisyk commented on this pull request.

In android/flavors/debug/AndroidManifest.xml:

@@ -1,7 +1,5 @@

<manifest xmlns:android="http://schemas.android.com/apk/res/android"

xmlns:tools="http://schemas.android.com/tools"

tools:remove="sharedUserId, sharedUserLabel">
Проблема в следующем. Мы используем разные сертификаты для подписи дебага, беты и релиза в сторе. С разными сертификатами эта фича толком не работает.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Надо разобраться и вернуть. Это нормально, что сертификаты разные для разных версий. Но шаред надо оставить. Какие именно у тебя ошибки? > On 3 Mar 2021, at 09:37, Roman Tsisyk <notifications@github.com> wrote: > > > @rtsisyk commented on this pull request. > > In android/flavors/debug/AndroidManifest.xml: > > > @@ -1,7 +1,5 @@ > <?xml version="1.0" encoding="utf-8"?> > <manifest xmlns:android="http://schemas.android.com/apk/res/android" > - xmlns:tools="http://schemas.android.com/tools" > - tools:remove="sharedUserId, sharedUserLabel"> > Проблема в следующем. Мы используем разные сертификаты для подписи дебага, беты и релиза в сторе. С разными сертификатами эта фича толком не работает. > > — > You are receiving this because you commented. > Reply to this email directly, view it on GitHub, or unsubscribe.

biodranik commented

2021-03-03 09:10:32 +00:00

(Migrated from github.com)

Видимо не могли декодить обфусцированные креши.

On 3 Mar 2021, at 09:39, Roman Tsisyk notifications@github.com wrote:

@rtsisyk commented on this pull request.

In android/gradle.properties:

@@ -26,4 +26,3 @@ org.gradle.caching=true
org.gradle.jvmargs=-Xmx8g
android.useAndroidX=true
android.enableJetifier=true
-android.enableR8=false
Смотри 2c6fb9a
Поразбираюсь что это такое.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Видимо не могли декодить обфусцированные креши. > On 3 Mar 2021, at 09:39, Roman Tsisyk <notifications@github.com> wrote: > > > @rtsisyk commented on this pull request. > > In android/gradle.properties: > > > @@ -26,4 +26,3 @@ org.gradle.caching=true > org.gradle.jvmargs=-Xmx8g > android.useAndroidX=true > android.enableJetifier=true > -android.enableR8=false > Смотри 2c6fb9a > Поразбираюсь что это такое. > > — > You are receiving this because you commented. > Reply to this email directly, view it on GitHub, or unsubscribe.

rtsisyk reviewed 2021-03-06 08:25:51 +00:00

android/gradle.properties

					
				@ -26,4 +26,3 @@ org.gradle.caching=true

				org.gradle.jvmargs=-Xmx8g

				android.useAndroidX=true

				android.enableJetifier=true

				android.enableR8=false

rtsisyk commented

2021-03-06 08:25:50 +00:00

A problem occurred evaluating root project 'android'.
> Failed to apply plugin 'com.android.internal.application'.
   > The option 'android.enableR8' is deprecated.
     It was removed in version 7.0 of the Android Gradle plugin.
     Please remove it from `gradle.properties`.

https://imstudio.medium.com/android-journey-proguard-d8-r8-what-are-they-e8f2bfe079a7

Ну, видимо, не отключить этот R8 уже теперь никак. Гугл один хрен рано или поздно релизнет новую Android студию с Gradle 7.0.0 и всеми теми версиями, что я тут проталкиваю в альфе....

``` A problem occurred evaluating root project 'android'. > Failed to apply plugin 'com.android.internal.application'. > The option 'android.enableR8' is deprecated. It was removed in version 7.0 of the Android Gradle plugin. Please remove it from `gradle.properties`. ``` https://imstudio.medium.com/android-journey-proguard-d8-r8-what-are-they-e8f2bfe079a7 Ну, видимо, не отключить этот R8 уже теперь никак. Гугл один хрен рано или поздно релизнет новую Android студию с Gradle 7.0.0 и всеми теми версиями, что я тут проталкиваю в альфе....

rtsisyk reviewed 2021-03-06 08:30:29 +00:00

android/flavors/debug/AndroidManifest.xml

					
				@ -3,3 +3,2 @@

				    xmlns:tools="http://schemas.android.com/tools"

				    tools:remove="sharedUserId, sharedUserLabel">

				    xmlns:tools="http://schemas.android.com/tools">

				  <uses-permission android:name="android.permission.READ_LOGS"/>

rtsisyk commented

2021-03-06 08:30:29 +00:00

Конкретно в этом манифесте как раз наоборот для дебаг версии шаринг отключался. По сути шаринг был только между бетой (собиратся на ci и выкладывается в Firebase AppDistribution) и релизом (AppStore).
Я открыл тикет, что мы хотим это почнить. Скорее всего, надо тогда одинаковые ключи использовать.
https://github.com/omapsapp/omapsapp/issues/201

Конкретно в этом манифесте как раз наоборот для дебаг версии шаринг отключался. По сути шаринг был только между бетой (собиратся на ci и выкладывается в Firebase AppDistribution) и релизом (AppStore). Я открыл тикет, что мы хотим это почнить. Скорее всего, надо тогда одинаковые ключи использовать. https://github.com/omapsapp/omapsapp/issues/201

biodranik (Migrated from github.com) reviewed 2021-03-08 22:27:29 +00:00

biodranik (Migrated from github.com) left a comment

Какая польза от перехода на альфу? Не наловим ли багов?

.github/workflows/android-beta.yaml

biodranik (Migrated from github.com) commented

2021-02-28 13:01:59 +00:00

Крэшлитика остаётся?

android/AndroidManifest.xml

biodranik (Migrated from github.com) commented

2021-02-28 13:02:28 +00:00

Мы же арабский поддерживаем, зачем выпилил?

android/build.gradle

biodranik (Migrated from github.com) commented

2021-03-08 22:24:38 +00:00

А почем убрал?

android/flavors/debug/AndroidManifest.xml

					
				@ -3,3 +3,2 @@

				    xmlns:tools="http://schemas.android.com/tools"

				    tools:remove="sharedUserId, sharedUserLabel">

				    xmlns:tools="http://schemas.android.com/tools">

				  <uses-permission android:name="android.permission.READ_LOGS"/>