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

Replace license-file with license #1

Merged

Conversation

qryxip
Copy link
Contributor

@qryxip qryxip commented Jan 3, 2024

Replaces the license-file fields with license.

As described in Cargo Book, license-file is generally used for "nonstandard" licenses that are not in the SPDX list.

In actual, on Crates.io, the license section would be:
image

@White-Green
Copy link
Owner

PRありがとうございます
world crateに関しては提案の通りで良いと思います
world_sys crateについてこの書き方にしてしまうとオリジナルのWorldのライセンスが機械可読な形で載らないようになってしまうことが気になっています
オリジナルのWorldのライセンスではMITと同様"表示"を求めているため、こちらのライセンスもツールで集計できる機械可読な形で含まれるほうが良いと考えているのですが、良い方法は無いのでしょうか

@qryxip
Copy link
Contributor Author

qryxip commented Jan 3, 2024

あまりよい方法は今のところないはずなので、GitのsubmoduleでWorldを含んでいるこのリポジトリならMIT AND {ライブラリのライセンス}とするのがちょうどよいかなと思っています。
(ちなみに私はcargo-denyで「依存に含まれるbuild.rs」を全部チェックするようにしています。あとbottlerocket-license-scanも併用しています。例: rust-lang-ja/atcoder-proposal)

あとWorldのライセンスは「修正BSDライセンス」/ "modified BSD License"であると明記されており、さらにそれはLICENSE.txtの内容的にBSD-3-Clauseとみなして間違いなさそうです。

@@ -2,7 +2,7 @@
name = "world_sys"
version = "0.1.0"
edition = "2021"
license-file = "WORLD/LICENSE.txt"
license = "MIT"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Suggested change
license = "MIT"
license = "MIT AND BSD-3-Clause"

@White-Green
Copy link
Owner

なるほど、確かにビルドスクリプトをチェックすれば外部ライブラリをリンクしているところはわかりますね
私が調査をした感じでもこれを実現するスマートな方法は見当たりませんでした
World本体まで含めた全自動でのライセンス情報集計を諦めたときworld-sys crateにWorld本体のライセンスは含めないほうが良いと思うので、MITのみ(AND BSD-3-Clauseは無し)のライセンス表記としようと思います

現状のコードでそうなっているのでこの状態でマージします
何かあればコメントをください

@White-Green White-Green merged commit 37c0d11 into White-Green:main Jan 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants