Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

LICENSEファイルを追加する #470

Merged
merged 1 commit into from
Sep 23, 2018

Conversation

berryzplus
Copy link
Contributor

目的

GitHubのリポジトリ一覧でライセンス表示が出るようにします。
zlib licenseOther と表示されます。)

インストーラに同梱するライセンスファイルの話 #41 と関連します。

@m-tmatma m-tmatma added license ライセンス関連 document ドキュメント labels Sep 23, 2018
@m-tmatma m-tmatma added this to the next release milestone Sep 23, 2018
@m-tmatma
Copy link
Member

http://woshidan.hatenadiary.jp/entry/2017/09/27/004114

改変した場合は改変したことがわかるようにする
に関しては、 #210 があるから、改変版を作りたい人は何もしなくていいのかな?

@berryzplus
Copy link
Contributor Author

改変した場合は改変したことがわかるようにする
に関しては、 #210#210, GitHub の commit に対する URL, PR の Head commit に対する URL へのリンクをバージョンダイアログに追加 があるから、改変版を作りたい人は何もしなくていいのかな?

それで合ってる認識です。
GitHub上でForkしたリポジトリをビルドすれば、forkされたほうのリポジトリURLが表示されるようになってますよね。

これの話は、単にリポジトリ一覧で表示されるときに「ライセンス不詳のアプリ」と表示されていることへの対策です。「Otherってどうなの?」は別問題として 😄

@m-tmatma
Copy link
Member

この LICENSE ってどこから持ってきました?

@berryzplus
Copy link
Contributor Author

この LICENSE ってどこから持ってきました?

ここ
http://zlib.net/zlib_license.html

wikipediaで公式と説明されているページですね。

oss ライセンスのまとめみたいなとこにある文書とも一致している感じです。
https://opensource.org/licenses/Zlib

追加してるのは先頭と末尾の表記のみです。

@KENCHjp KENCHjp merged commit 359e69e into sakura-editor:master Sep 23, 2018
@berryzplus berryzplus deleted the feature/add_license_file branch September 23, 2018 12:04
@KENCHjp KENCHjp mentioned this pull request Sep 24, 2018
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 11, 2019
…nse_file

LICENSEファイルを追加する
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
document ドキュメント license ライセンス関連
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants