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

GPLライセンス違反確認を追加しました。 #679

Closed
wants to merge 6 commits into from

Conversation

tuna2134
Copy link
Contributor

@tuna2134 tuna2134 commented May 1, 2023

内容

GPLライセンス違反を確認する機能を追加しました。

関連 Issue

ref #546

@tuna2134 tuna2134 requested a review from a team as a code owner May 1, 2023 03:30
@tuna2134 tuna2134 requested review from y-chan and removed request for a team May 1, 2023 03:30
@Hiroshiba
Copy link
Member

Hiroshiba commented May 1, 2023

PRありがとうございます!
GPL以外にもちょっと検査しておきたいライセンスがあるのでついでに追加をお願いしてもよろしいでしょうか・・・!!

追加したいのはこちらです。

  "GPL",
  "GPL-2.0",
  "GPL-3.0",
  "AGPL",
  "NGPL",

参考

頂いたPRの形で望んだ動作をするのかが気になりました。
例えば、すでに含まれている別のライセンス(BSDなど)を指定した場合はエラーとなりますか?

@Hiroshiba
Copy link
Member

あるいは実際にGPLなpythonパッケージをインストールしてみてエラーになるかチェックしていただけると!
https://pypi.org/search/?q=gpl

@tuna2134
Copy link
Contributor Author

tuna2134 commented May 1, 2023

わかりました。

@Hiroshiba
Copy link
Member

discordでのやりとりの転記です。

tunaxl@なんかとびでた — 今日 16:32
pyinstallerがgnuライセンス使っていますが、どうします?

ちなみに対象から除外しますか?

ヒホ — 今日 16:37
エンジンのビルドスクリプト追いかけてみたんですが、ライセンス周りは
`bash build_util/create_venv_and_generate_licenses.bash`
が動けばOKですね!

でなかで pip install -r requirements-license.txt してそう
で、requirements-license.txt にはPyInstallerは含まれてない。ので除外する必要がないはず

@github-actions
Copy link

github-actions bot commented May 2, 2023

Coverage Result

Resultを開く
Name Stmts Miss Cover
voicevox_engine/init.py 2 1 coverage-50%
voicevox_engine/acoustic_feature_extractor.py 75 0 coverage-100%
voicevox_engine/dev/synthesis_engine/init.py 3 1 coverage-67%
voicevox_engine/dev/synthesis_engine/mock.py 36 2 coverage-94%
voicevox_engine/full_context_label.py 162 3 coverage-98%
voicevox_engine/kana_parser.py 86 1 coverage-99%
voicevox_engine/metas/Metas.py 33 0 coverage-100%
voicevox_engine/metas/MetasStore.py 29 14 coverage-52%
voicevox_engine/metas/init.py 2 0 coverage-100%
voicevox_engine/model.py 150 9 coverage-94%
voicevox_engine/mora_list.py 5 0 coverage-100%
voicevox_engine/part_of_speech_data.py 5 0 coverage-100%
voicevox_engine/preset/Preset.py 12 0 coverage-100%
voicevox_engine/preset/PresetError.py 3 1 coverage-67%
voicevox_engine/preset/PresetManager.py 81 2 coverage-98%
voicevox_engine/preset/init.py 4 0 coverage-100%
voicevox_engine/setting/Setting.py 11 0 coverage-100%
voicevox_engine/setting/SettingLoader.py 19 0 coverage-100%
voicevox_engine/setting/init.py 3 0 coverage-100%
voicevox_engine/synthesis_engine/init.py 5 0 coverage-100%
voicevox_engine/synthesis_engine/core_wrapper.py 201 159 coverage-21%
voicevox_engine/synthesis_engine/make_synthesis_engines.py 57 49 coverage-14%
voicevox_engine/synthesis_engine/synthesis_engine.py 130 11 coverage-92%
voicevox_engine/synthesis_engine/synthesis_engine_base.py 67 9 coverage-87%
voicevox_engine/user_dict.py 144 11 coverage-92%
voicevox_engine/utility/init.py 5 0 coverage-100%
voicevox_engine/utility/connect_base64_waves.py 37 0 coverage-100%
voicevox_engine/utility/core_version_utility.py 9 2 coverage-78%
voicevox_engine/utility/mutex_utility.py 11 1 coverage-91%
voicevox_engine/utility/path_utility.py 26 8 coverage-69%
TOTAL 1413 284 coverage-80%

@tuna2134
Copy link
Contributor Author

tuna2134 commented May 2, 2023

@Hiroshiba とりあえず完成しました

"GNU General Public License (GPL)",
"GNU General Public License v3 (GPLv3)",
"GNU Affero General Public License v3 (AGPL-3)",
"GNU Lesser General Public License v2 (LGPLv2)",
Copy link
Member

Choose a reason for hiding this comment

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

LGPLはOKなので外して頂ければ!
逆にAGPLはv1もv2もたぶんダメなので、含めて頂けると。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

わかりました!

@Hiroshiba
Copy link
Member

Hiroshiba commented Oct 17, 2023

@tuna2134 あ、こちらのプルリクエストどうしましょう 👀
せっかくなのでマージまで進められると楽しいのかなと!

@tarepan
Copy link
Contributor

tarepan commented Feb 25, 2024

👍 端的なコードで重要な機能を実現しており、より安全な開発が実現できそうです。

  • レビューコメント (対象ライセンス修正) の反映が approve に必要です。

@Hiroshiba
こちら優れたPRなので、誰かが引き継ぐ形で進行できるでしょうか?

@Hiroshiba
Copy link
Member

@tarepan
誰かが引き継いでの進行ですが、 @tuna2134 さんにもお聞きできればと思います!
しばらくお返事がなかったらたぶんお忙しいと思うので、誰かが引き継いでOKかなと!

@tarepan
Copy link
Contributor

tarepan commented Mar 25, 2024

#1141 にて commit log 含む引き継ぎがなされました。
本 PR は close とします。

@tarepan tarepan closed this Mar 25, 2024
@tarepan tarepan removed their assignment Mar 25, 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.

3 participants