Feature/android api level31 #266

neon-izm · 2023-03-10T11:19:07Z

solve #265

議論ポイント

以下2点は悩みどころです。

ACCESS_FINE_LOCATION の権限は必要か
- SDK内容的にはACCESS_COARSE_LOCATIONでも動くような気がします
Unityバージョンを上げるかどうか
- Android (Google Play Store)は API Target Level 31を要求していますが、現在のリポジトリ内のUnityバージョンであるUnity2021.3.0f1はAndroid SDK 30がインストールされてしまうため、そのままのビルドではストア審査が通らない
- readmeにその旨記載しておく、あるいは素直にUnityバージョンを上げる、どちらが望ましいでしょうか？
- 例えばUnity2021.3.0f1の同梱Android SDK 30では`Android.Permission.RequestUserPermission("android.permission.BLUETOOTH_SCAN") が成功しないです。

Release v1.5.0

Fix documents

Adjust readme

Develop

release 1.5.1

neon-izm · 2023-03-10T11:21:14Z

toio-sdk-unity/Assets/ble-plugin-unity/Plugins/Android/Runtime/BlePermissionRequest.cs

+        const string androidBluetoothPermission = "android.permission.BLUETOOTH";
+        const string androidBluetoothAdminPermission = "android.permission.BLUETOOTH_ADMIN";
+        const string androidBluetoothAdvertisePermission = "android.permission.BLUETOOTH_ADVERTISE";
+        const string androidBluetoothScanPermission = "android.permission.BLUETOOTH_SCAN";


API Level 31 Bluetooth permissions
https://developer.android.com/about/versions/12/features/bluetooth-permissions

にあるように権限の細分化が必要。しかしUnity2021.3.0f1はこれらの新しい権限のリクエストが失敗する点に注意。

neon-izm · 2023-03-10T11:26:39Z

toio-sdk-unity/Assets/ble-plugin-unity/Plugins/Android/src/BleDeviceObj.java

@@ -169,7 +169,7 @@ public BleDeviceObj(BluetoothDevice device, Context cxt) {
        this.charastricsKeyHashMap = new HashMap<CharastricsKey,BluetoothGattCharacteristic>();
        this.pubCharastricsKeyHashMap = new HashMap<CharastricsKey,BluetoothGattCharacteristic>();
        this.charastricsKeys = new ArrayList<CharastricsKey>(32);
-        device.connectGatt(cxt,true,this);
+        device.connectGatt(cxt,true,this,BluetoothDevice.TRANSPORT_LE);


明示的にLEを指定しないと問題が起きそうです。

JuulLabs/kable#61

MrkKSen · 2023-03-10T12:29:56Z

@neon-izm さん
詳細な情報を書いていただきありがとうございます！

・ACCESS_FINE_LOCATION の権限は必要か

ブルートゥースだけのためなので、ACCESS_COARSE_LOCATIONでも動けるならそれで良いかと思います。

・Unityバージョンを上げるかどうか

とてもわかりやすい解説で助かりました。こちらに関しては運営の判断が必要なので、また来週検討させてください。

neon-izm · 2023-03-11T04:35:08Z

@MrkKSen sanありがとうございます。
一応選択肢としてはUnityのバージョンアップでAndroid API Level31の追加権限に対応する以外に権限取得部を
https://github.com/yasirkula/UnityAndroidRuntimePermissions に置き換える、もアリかもしれません。

良い点

Unityのバージョンアップが不要
他のバージョンのUnityで使うユーザに優しい

悪い点

依存ライブラリが増える

という感じです。

MrkKSen · 2023-03-14T13:18:50Z

@neon-izm さん
ご返信ありがとうございます。
いくつか報告と相談をさせてください。

Unityのバージョンアップはしない

toio SDK for Unity は原則 Unity のLTSバージョンに基づきたいと考えておりますので、そろそろ2022LTSが出る時期なので、バージョンアップはその時にしたいと思っております。

手元にある Android 11 (API 30) のスマホで試した結果

現在頂いたPRの内容で、Unity2021(API 30)でビルドするとエラーが発生。
AndroidManifest.xml:20:5-22:58: AAPT: error: attribute android:usesPermissionFlags not found
Unity2021(API 30)で、ACCESS_COARSE_LOCATION は動けなさそうです。
Unity2022(API 31)で、BLUETOOTH_ADVERTISE はいらないです。

なので、manifest は以下なのが良いかもしれません。

<!--（以上略）-->
    <uses-permission android:name="android.permission.BLUETOOTH"
                     android:maxSdkVersion="30" />
    <uses-permission android:name="android.permission.BLUETOOTH_ADMIN"
                     android:maxSdkVersion="30" />
    <uses-permission android:name="android.permission.BLUETOOTH_SCAN"/>
    <uses-permission android:name="android.permission.BLUETOOTH_CONNECT" />
    <uses-permission android:name="android.permission.ACCESS_FINE_LOCATION" />
<!--（以下略）-->

Android 12で動けるかはわからないですが、もし問題なければこれで対応して頂きたいと考えております。
ドキュメントについてはPRを受け入れてからこちらで考えてみます。いかがでしょうか？

その他

ちなみに紹介頂いたライブラリ https://github.com/yasirkula/UnityAndroidRuntimePermissions ですが、BlePermissionRequest.cs の権限取得部を置き換えるとして、なんの違いがあるかが理解できていません、ご教示いただけるとありがたいです！

neon-izm · 2023-03-23T16:44:02Z

toio-sdk-unity/Assets/Plugins/Android/AndroidManifest.xml

+    <uses-permission android:name="android.permission.BLUETOOTH_CONNECT"
+                     android:minSdkVersion="31"/>
+    <!-- application must have ble hardware  -->
+    <uses-feature android:name="android.hardware.bluetooth_le" android:required="true"/>


BLE非対応機器にインストールできないように指定

neon-izm · 2023-03-23T16:49:02Z

@MrkKSen san遅れましたが追加のコミットをしました。

FBを受けての変更点

AndroidManifestの権限を見直しました。おそらくこれでAndroid8~13までの権限が網羅できたと思います。

AndroidRuntimePermission導入は無くても良いか

なくても大丈夫そうです！

Unityバージョンアップについて

了解です。LTSが出たらまた考えていただければと思います。

残件について

現時点のPRでおおよそやりたいことは実現できそうですが、引き続き現在のGooglePlay Storeへの提出にはTarget APIが31以上という制約が遺っています。
このライブラリのUnityの現在のバージョンではTarget APIが30までしか対応していないので、ストアリリース時はAndroid SDKを別でインストールしてTarget API=31を指定する必要がある旨をドキュメントに追記お願いしたいです！

MrkKSen · 2023-03-24T09:47:41Z

@neon-izm さま
ご対応ありがとうございます！

ドキュメント追記の件承知しました。

一つの疑問がありますが、以下の部分で minSdkVersion="31" が設定されていますが、それでも API 30 でビルドする場合は、AndroidManifest.xml:20:5-22:58: AAPT: error: attribute android:usesPermissionFlags not found というエラーが出てきます（多分31の新仕様なので）。

    <uses-permission android:name="android.permission.BLUETOOTH_SCAN"
             android:minSdkVersion="31"
             android:usesPermissionFlags="neverForLocation" />

usesPermissionFlags に関して、一つの manifest ファイルで後方互換性を保つのは無理でしょうか？
そうであれば一旦今回は削除して、30と同様に FineLocation を取得したほうが手っ取り早いかなと考えておりますが、いかがでしょうか？
（Unity LTS版のアップグレード対応時に、また複数の Manifest を用意してユーザーに選んでもらう方式を検討できます。）

neon-izm · 2023-03-24T13:39:47Z

@MrkKSen san FBありがとうございます。

そもそもこのPRがAndroid のTarget API>=31での動作を行うためのものなのでTarget API=30でエラーが起きるのは仕方ないのかな、という気はします（存在しない権限なので）

おっしゃるとおり後方互換性をうまく保つ方法は残念ながら無いと思います。
（Android Manifest.xml内でコメントを書いておいてお茶を濁す、とかでしょうか）

現実的には以下2個の対応が有り得ると思っています。どうしましょうね…

このPRはマージせず、toio SDKはTarget API=30を維持する

メンテコスト的にはこれが楽です。
欠点はGoogle Playへのapkアップロード時にTarget API設定でリジェクトされて、開発者がビックリします。

このPRをマージして、Target API=31にする

開発者はGoogle Play Storeへの公開も行えるし、将来のアップデートコストが低いです。

欠点は 一時的に Unity2021のバージョンに同梱されているAndroid SDKではビルド出来なくなってドキュメントでのフォローが必要です。(Android SDKを追加でインストールしてUnity Editorで指定する、という旨の記載）

MrkKSen · 2023-03-27T06:48:22Z

@neon-izm さま
ご回答ありがとうございます。
Android Manifest.xmlの後方互換性を保つ方法はないこと承知しました。

とすると、このPRをマージすると、toio SDK for Unity の対応 Unityバージョンが 2021 なのに、デフォルトの状態では Unity 2021 で動けないという状況になるので、合理的ではないかと思いました。

なのでこのPRを main に取り入れるのは Unity 2022 LTS にアップグレードしてからにして、当面の対応としては、一旦専用のブランチにマージし、main のドキュメントで「google playに配布したい方、API 30 を使いたいかたはこのブランチを参考にしてください」みたいに追記する形を考えておりますが、いかがでしょうか？

neon-izm · 2023-03-27T11:02:13Z

良いと思います。ではこのPRはそのまま放置というか、2022LTSへのアップデートの際にマージよろしくお願いします&readmeへの追記お願いします

neon-izm · 2023-06-09T10:11:48Z

@MrkKSen sanそろそろUnity2022LTSも出たので、上記PRの件含めて検討おねがいします！

https://unity.com/ja/releases/lts

MrkKSen · 2023-06-13T07:29:51Z

@neon-izm さん
出ましたね！他の修正合わせて対応していきますので、一旦 develop にマージさせていただきます。
ご対応ありがとうございました！

MABUCHI-HIROKI and others added 8 commits April 22, 2022 16:14

Merge pull request morikatron#249 from morikatron/develop

1272736

Release v1.5.0

Merge pull request morikatron#251 from morikatron/develop

00e4c81

Fix documents

Merge pull request morikatron#253 from morikatron/develop

280133a

Adjust readme

Merge pull request morikatron#255 from morikatron/develop

f7e616d

Develop

Merge pull request morikatron#257 from morikatron/develop

f324598

Develop

Merge pull request morikatron#261 from morikatron/develop

b701ea4

release 1.5.1

android api level 31

623b517

permission request should devide with api level

079399e

neon-izm commented Mar 10, 2023

View reviewed changes

neon-izm mentioned this pull request Mar 10, 2023

Android Target API 31以降への対応 #265

Closed

neon-izm commented Mar 10, 2023

View reviewed changes

neon-izm added 2 commits March 24, 2023 01:40

feedback permissions

19ae430

remove advertise

b79ec14

neon-izm commented Mar 23, 2023

View reviewed changes

MrkKSen changed the base branch from main to develop April 3, 2023 05:07

MrkKSen approved these changes Apr 3, 2023

View reviewed changes

MrkKSen merged commit 385c372 into morikatron:develop Jun 13, 2023

neon-izm deleted the feature/android_api_level31 branch June 17, 2023 05:07

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feature/android api level31 #266

Feature/android api level31 #266

neon-izm commented Mar 10, 2023 •

edited

Loading

neon-izm Mar 10, 2023

neon-izm Mar 10, 2023

MrkKSen commented Mar 10, 2023 •

edited

Loading

neon-izm commented Mar 11, 2023

MrkKSen commented Mar 14, 2023 •

edited

Loading

neon-izm Mar 23, 2023

neon-izm commented Mar 23, 2023

MrkKSen commented Mar 24, 2023 •

edited

Loading

neon-izm commented Mar 24, 2023

MrkKSen commented Mar 27, 2023

neon-izm commented Mar 27, 2023

neon-izm commented Jun 9, 2023

MrkKSen commented Jun 13, 2023

Feature/android api level31 #266

Feature/android api level31 #266

Conversation

neon-izm commented Mar 10, 2023 • edited Loading

議論ポイント

neon-izm Mar 10, 2023

Choose a reason for hiding this comment

neon-izm Mar 10, 2023

Choose a reason for hiding this comment

MrkKSen commented Mar 10, 2023 • edited Loading

neon-izm commented Mar 11, 2023

MrkKSen commented Mar 14, 2023 • edited Loading

Unityのバージョンアップはしない

手元にある Android 11 (API 30) のスマホで試した結果

その他

neon-izm Mar 23, 2023

Choose a reason for hiding this comment

neon-izm commented Mar 23, 2023

FBを受けての変更点

AndroidRuntimePermission導入は無くても良いか

Unityバージョンアップについて

残件について

MrkKSen commented Mar 24, 2023 • edited Loading

neon-izm commented Mar 24, 2023

このPRはマージせず、toio SDKはTarget API=30を維持する

このPRをマージして、Target API=31にする

MrkKSen commented Mar 27, 2023

neon-izm commented Mar 27, 2023

neon-izm commented Jun 9, 2023

MrkKSen commented Jun 13, 2023

neon-izm commented Mar 10, 2023 •

edited

Loading

MrkKSen commented Mar 10, 2023 •

edited

Loading

MrkKSen commented Mar 14, 2023 •

edited

Loading

MrkKSen commented Mar 24, 2023 •

edited

Loading