This repository has been archived by the owner on Apr 12, 2023. It is now read-only.
ソースコードにMPLのライセンスを追加 #83
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
aaa439c to
bb6744a
Compare
ghost
reviewed
Mar 28, 2021
Covid19Radar/Covid19Radar/App.xaml
Outdated
Comment on lines
1
to
5
| <!-- This Source Code Form is subject to the terms of the Mozilla Public | ||
| - License, v. 2.0. If a copy of the MPL was not distributed with this | ||
| - file, You can obtain one at https://mozilla.org/MPL/2.0/. --> | ||
|
|
||
| <?xml version="1.0" encoding="utf-8" ?> |
There was a problem hiding this comment.
XML宣言は先頭にないとダメなので宣言より前にコメントを追加してはいけないと思います
There was a problem hiding this comment.
お、やってみます。
xlfの方は先頭でビルド通ったので油断してましたね。
There was a problem hiding this comment.
ありがとうございます。手元でビルド通りました。pushしてCIで試します。
修正ファイルが多いのでforce pushします。
bb6744a to
dd31f47
Compare
|
ソースコードにコメント入れてなかったのは、何か意図があったというよりは、単に時間の余裕がなかったという事情がありまして、今なら含めても良いのでは無いかなと思います。 |
|
ありがとうございます。確かにロゴマークについては切り分けて考えなくてはいけませんね（ #3 ）。 ひとまずいくつかライセンス表記漏れのファイルが見つかっているので、そちらを追加しながらビルドチェックをしていきます。 |
|
@halsk @heykuro @kazgoto @masanork @cocoa-dev Issueの解決のため、各ファイルにライセンス通知を追加しました。 参考までにライセンス通知を 追加していない ファイル一覧を示します。 |
heykuro
approved these changes
Mar 30, 2021
ghost
reviewed
Mar 30, 2021
Comment on lines
+1
to
+3
| # This Source Code Form is subject to the terms of the Mozilla Public | ||
| # License, v. 2.0. If a copy of the MPL was not distributed with this | ||
| # file, You can obtain one at https://mozilla.org/MPL/2.0/. |
There was a problem hiding this comment.
マージされてしまったのですが、.bat ファイルのコメントは # ではエラーになります
PS > .\CleanBuild.bat
># This Source Code Form is subject to the terms of the Mozilla Public
'#' は、内部コマンドまたは外部コマンド、
操作可能なプログラムまたはバッチ ファイルとして認識されていません。
REM がコメント記法です。
There was a problem hiding this comment.
そちらを見落としていました。ありがとうございます！
Closed
この Issue がまだ提出されていないので、@keiji さんお願いできますか。 |
4 tasks
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue 番号 / Issue ID
目的 / Purpose
「各ファイルにライセンス通知を配置していないためCOCOAのコードはMPLの適用対象とみなされない」との指摘を受けて取り急ぎ対応しました。
XML, xaml, cs, xlfの各ファイルの先頭に機械的にライセンス表記を追加してあります。
ほかにも追加の必要があればコマンド一つで追加で安心です。
破壊的変更をもたらしますか / Does this introduce a breaking change?
Pull Request の種類 / Pull Request type
確認事項 / What to check
すべてコメントであるのでビルドが通るかを確認する。
ライセンスヘッダーに追加漏れがないかを確認する。
COCOAのfork元であるCovid19Radarに「ライセンスを各ファイルに記載しなかったのかのは意図的か」。また、COCOAでは記載することを望むかについて確認する。
その他 / Other information
廣瀬（ @kazumihirose ）さん。
Covid19RadarからforkしたCOCOAですが、各ファイルにライセンス表記をしていないためCOCOAのコードはMPLの適用対象とみなされないとの指摘がありました。
#78 (comment)
当該箇所はCovid19Radarからforkしたときからの状態である認識していて、またMPLの規定では
つまり
ともされていて、Covid19Radarが開発時に「各ファイルにライセンス表記を含めることが不可能または望ましくない」と判断した可能性を考えています。
これについては判断した理由を聞きたいわけではありません。Original Authorsの意思を最大限に尊重すべく、COCOAではどうすることを望むか。ご意見をお聞かせ願えますと幸いです。
UPDATE
ぼくの勘違いでした！ 各ファイル修正したらビルド通りました。
本PRのCI結果をご覧いただければわかるとおり、Xamlや、一部のXMLファイルはライセンス表記を追加すると（コメント形式であっても）ビルドできなくなるようです。どのファイルがよくてどのファイルがダメなのか。考えながら開発することは建設的でなく、Covid19Radarでもこの出来事からすべてのファイルにライセンス表記は不可能、または望ましくないと判断したのではないかと愚考します。ひとまず次のコメントにあるとおり、IssueとしてはInvalidとしてcloseの方向で考えていますが、引き続きご意見お待ちしています。#80 (comment)