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

update COPYRIGHT THIRD PARTY SOFTWARE NOTICES. #184

Merged
merged 4 commits into from
Jul 6, 2021
Merged

update COPYRIGHT THIRD PARTY SOFTWARE NOTICES. #184

merged 4 commits into from
Jul 6, 2021

Conversation

tatsu-jp
Copy link
Contributor

Issue 番号 / Issue ID

目的 / Purpose

ライセンス内容を確認して記述不足や文字化けを修正することで、ライセンス内容を最新の状態にします。

破壊的変更をもたらしますか / Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request の種類 / Pull Request type

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[x] Documentation content changes
[ ] Other... Please describe:

検証方法 / How to test

コードの入手 / Get the code

git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
dotnet restore

確認事項 / What to check

  • GitHubのビューアで開いて、文字化け等が解消されていることを確認

その他 / Other information

参考


Copyright (c) 2014, Austin Andrews (http://materialdesignicons.com/), with Reserved Font Name Material Design Icons.
Licensed under the Apache License, Version 2.0 (the "License");
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Apache Licenseへ変更するのではなく、フォントを追加した当時のライセンスであるSILの本文を追加する修正が正しいはずです。

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ご指摘ありがとうございます。すみません、時系列を勘違いしていました。
修正しました。Copyright Holderはgoogle/material-design-iconsに"Material design icons is the official icon set from Google. "とあったので、Googleにしています。

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

COCOAで使われているMaterial Design Iconsは https://github.com/Templarian/MaterialDesign-Webfont からのもので、Copyright Holderが Austin Andrews 氏になっているのはそのためと思われます。
ここは原文通り、2014, Austin Andrews (http://materialdesignicons.com/) でお願いします。

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ありがとうございます。理解が不十分でした。
今夜修正します。

@keiji keiji changed the base branch from master to develop June 7, 2021 07:16
Copy link
Collaborator

@keiji keiji left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

カスタムフォント廃止に伴い、ライセンスの削除が発生しています。
よろしくお願いします。

@@ -443,27 +443,209 @@ We've kept attribution comments terse, so we ask that you do not actively work t
---
## (Font) Noto Sans CJK JP
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

v1.2.4 でNoto Sans CJK JPは削除されたので、合わせてエントリーの削除お願いします:bow:

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#210 で削除されていることを確認して、Noto Sans CJK JPのライセンス記述を削除しました。
Info.plist上の指定も削除した方が良いように思うのですが、残している意図等あれば教えて頂けると助かります。

<array>
<string>fa-brands-400.otf</string>
<string>fa-regular-400.otf</string>
<string>fa-solid-900.otf</string>
<string>NotoSansCJKjp-Medium.otf</string>
<string>NotoSansCJKjp-Regular.otf</string>
<string>Roboto-Regular.ttf</string>
</array>

Copy link
Collaborator

@keiji keiji Jun 12, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

おそらく消し忘れですね。ありがとうございます。
あとでIssueにします! しました #231

@tatsu-jp
Copy link
Contributor Author

#116 (Font Awesome, Robotoの削除)に関してv1.2.4で実施済みとコメントありましたが、#210 では削除されていることを確認できませんでした。develop/mainブランチのCovid19Radar.iOS以下にフォントファイルが残っていました( #116 の修正が反映されていない)。

内部リポジトリ側では修正済み、GitHub側にマージする際に不備があったと理解すれば良いでしょうか。
ご確認お願いします。

@keiji
Copy link
Collaborator

keiji commented Jun 12, 2021

Font Awesome, Robotoは削除していません。今回使用をしなくなったのはNoto Sans CJK JPだけなので、削除するライセンスもNoto Sans CJK JPに関するものだけという認識です。

@tatsu-jp
Copy link
Contributor Author

ご確認ありがとうございます!
了解しました。

Copy link
Collaborator

@keiji keiji left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@keiji keiji requested a review from cocoa-dev June 13, 2021 00:30
@keiji keiji merged commit 42ef2c3 into cocoa-mhlw:develop Jul 6, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

サードパーティライセンスを確認して最新に反映する
4 participants