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

licenses.jsonの生成スクリプトを追加 #125

Merged
merged 18 commits into from
Sep 28, 2021

Conversation

aoirint
Copy link
Member

@aoirint aoirint commented Sep 28, 2021

内容

https://github.com/Hiroshiba/voicevox/issues/219#issuecomment-927917044 のコードをベースに、
ENGINEの依存関係のライセンス情報を格納したlicenses.jsonの生成スクリプトを追加します。

  • 生成スクリプト generate_licenses.py の追加
  • ENGINEのDockerイメージに同梱
  • ENGINEの実行バイナリに同梱
  • ライセンス情報がpip-licensesで取得できないライブラリの例外処理実装を確認するCIステップの追加
    • ライセンス情報が取得できない場合、例外を投げる

関連 Issue

@aoirint aoirint changed the title licenses.jsonを追加 licenses.jsonの生成スクリプトを追加 Sep 28, 2021
Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

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

LGTM!! EULAの取得方法も明確でとても良さそうに感じました。

2点、とても細かい部分で気づいたことがあったので、ちょっと修正コミットを追加します。

Dockerfile Outdated Show resolved Hide resolved
@Hiroshiba
Copy link
Member

diff --git a/README.md b/README.md
--- README.md
+++ README.md
@@ -144,8 +144,9 @@
     --include-package=uvicorn \
     --include-package-data=pyopenjtalk \
     --include-package-data=resampy \
     --include-data-file=VERSION.txt=./ \
+    --include-data-file=licenses.json=./ \
     --include-data-file=C:/path/to/cuda/*.dll=./ \
     --include-data-file=C:/path/to/libtorch/*.dll=./ \
     --include-data-file=C:/音声ライブラリへのパス/*.bin=./ \
     --include-data-file=C:/音声ライブラリへのパス/metas.json=./ \

@Hiroshiba
Copy link
Member

(実験報告です)
READMEにライセンス.jsonを追加したかったけど、変更箇所に含まれていないためかcommitができませんでした。なるほど。
そこまで大事ではなかったので、諦めてマージします!

@aoirint aoirint deleted the patch-license branch October 9, 2023 03:00
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