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

build: C APIにもLICENSEファイルを同梱 #965

Merged
merged 1 commit into from
Feb 4, 2025

Conversation

qryxip
Copy link
Member

@qryxip qryxip commented Feb 4, 2025

内容

#947 に続き。

関連 Issue

Refs: #938

その他

ちなみに今C APIだけREADME.mdはREADME.txtとして入っているようです。
(blameしたら #196 の初案段階からあって、誰も言及していないっぽい)

なのでPython APIやRust APIとはちょっと違う形になってるかと思います。

[追記] あと今Android用にだけリリースしているJava APIですが、そっちはちょっと調査が必要そう。どうもそういう文化が無い(!?)ようなので…

@qryxip qryxip requested a review from Hiroshiba February 4, 2025 03:04
@Hiroshiba
Copy link
Member

ちなみに今C APIだけREADME.mdはREADME.txtとして入っているようです。

あっ これ個人的には.txtの方が良い気もしています!
全てのtxtエディタで開けるので。.mdはmarkdownだとわからない人もいそう。

ORTとかのはmdでも良いけど、少なくともvvmのは.txtにした方が良さそうかも。voicevox_coreのも.txtで良さそう。
(voicevox_vvmのGithubがレンダリングしてくれるのでREADMEは.mdでも問題ない)

voicevox_vvm側の.mdを.txtにするのやらないとだ。
ダウンローダで変換かませるのと、voicevox_vvm側のリリース作成時に.txtにしてreleasesにアップロードするのと2つ方法ありそうですけど、前者で良いですかね・・・?
releases作るのもありな気がしてきたなぁ。

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!!

@qryxip qryxip merged commit 7224847 into VOICEVOX:main Feb 4, 2025
29 checks passed
@qryxip qryxip deleted the pr/build-include-license-file-in-whl branch February 4, 2025 11:30
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