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

Xamarin.ExposureNotification を NuGet から参照する様に変更する #241

Closed
Takym opened this issue Jun 15, 2021 · 9 comments
Closed
Labels
wontfix 今のところ対応予定がないもの

Comments

@Takym
Copy link
Contributor

Takym commented Jun 15, 2021

その機能リクエストは何らかの問題に関連しますか / Is your feature request related to a problem?

Xamarin.ExposureNotification のソースコードは丸ごとコピーされていますが、何か特別な理由はあるのでしょうか。
特に無ければ NuGet から参照する様に変更した方が無難だと思います。

解決策についてお書きください / Describe the solution you'd like

Xamarin.ExposureNotification を NuGet から参照する様に変更します。

あなたが考える代替案についてご説明ください / Describe alternatives you've considered

現在のままでも特に問題は無いのでこのままにしておきます。

その他 / Additional context

@b-wind
Copy link

b-wind commented Jun 15, 2021

古いバージョンをforkした物を取り込み、そのままと言う理解です。
バージョンアップは計画されていたと思いますが、現状は分かりませんね。

@keiji
Copy link
Collaborator

keiji commented Jun 15, 2021

現在のXamarin.ExposureNotificationはオリジナルから結構変わってるんですよね。
たとえば、バックグラウンド処理(WorkManager)のExistingPeriodicWorkPolicyが、今のCOCOAではKeepですが、オリジナル版ではReplaceです。

xamarin/XamarinComponentsにはPull Requestを出していますが反応がありません(あと、CLAの提出機構がなぜか働かない)
https://github.com/xamarin/XamarinComponents/pull/1108/files

ほかにも、変更に伴って既存のWorkManagerのジョブをキャンセルする仕組みなどが加わっています(これはコードを工夫すれば取り除けると思いますが)。

@b-wind
Copy link

b-wind commented Jun 15, 2021

CLA ってここから同意するのですかね。本来botがURLを提示してくるようですけれど。
https://cla.dotnetfoundation.org/xamarin/XamarinComponents?pullRequest=1108

@Takym
Copy link
Contributor Author

Takym commented Jun 15, 2021

ExistingPeriodicWorkPolicy は外側からは書き換える事はできないのでしょうか。

@Takym
Copy link
Contributor Author

Takym commented Jun 15, 2021

今ソースコードを読みましたが、外側からの書き換えはできないみたいですね。

@keiji
Copy link
Collaborator

keiji commented Jun 20, 2021

こちらひとまずwon't fixに指定した上でcloseとはせず「条件が整えばNuGet参照に切り換えたいね」という扱いで行きたいと思います。

@keiji keiji added the wontfix 今のところ対応予定がないもの label Jun 22, 2021
@tmurakami
Copy link

(すでにご存知かもしれませんが) Fhi.Smittestopp.App ではXamarin.ExposureNotification.Init()を利用せずに自前でバックグランド処理を実装しています。
Android: https://github.com/folkehelseinstituttet/Fhi.Smittestopp.App/blob/7c8e687a24d7e9d0f27f0885ee6d48d8a2c2e41e/NDB.Covid19/NDB.Covid19.Droid/Utils/BackgroundFetchScheduler.cs
iOS: https://github.com/folkehelseinstituttet/Fhi.Smittestopp.App/blob/7c8e687a24d7e9d0f27f0885ee6d48d8a2c2e41e/NDB.Covid19/NDB.Covid19.iOS/Utils/BackgroundServiceHandler.cs

COCOA でも、自前でバックグランド処理を持つことができれば NuGet を利用することができるのではないかと思っています。
いかがでしょうか?

@keiji
Copy link
Collaborator

keiji commented Nov 23, 2021

Xamarin.ExposureNotificationはExposureWindow mode対応のCOCOA2で取り除く予定です。

Xamarin.ExposureNotificationがベースにしているXamarin.GooglePlayServices.Nearby.ExposureNotification課題があります。課題を解決するPull Requestは出しており、取り込まれるのを待っている状態です。

Xamarin.ExposureNotificationのメンテナンスが継続するのが望ましいと考えていますが、次善の策として別のExposureNotificationライブラリを使った実装をfeature/cocoa2で進めています。
本Issueについてはfeature/cocoa2がdevelopブランチに取り込まれたタイミングでクローズとする予定です。

@keiji
Copy link
Collaborator

keiji commented Jan 8, 2022

時が過ぎ、現在のdevelopからはXamarin.ExposureNotificationがdevelopは含まれていませんので本IssueをCloseします。

@keiji keiji closed this as completed Jan 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
wontfix 今のところ対応予定がないもの
Projects
None yet
Development

No branches or pull requests

4 participants