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

refactor: Makefile を解体 #1269

Closed
2 tasks done
tarepan opened this issue May 18, 2024 · 3 comments · Fixed by #1335
Closed
2 tasks done

refactor: Makefile を解体 #1269

tarepan opened this issue May 18, 2024 · 3 comments · Fixed by #1335
Assignees

Comments

@tarepan
Copy link
Contributor

tarepan commented May 18, 2024

質問の内容

概要: Makefile の用途が知りたい

現在の VOICEVOX ENGINE は pyinstaller ベースのビルドをおこなっており、GitHub Actions での自動ビルドでも pyinstaller を呼び出している。
一方、レポジトリルートには Makefile も定義されており、Docker 関連らしきコードが書かれている。
make コマンドが見つからなかったため、この Makefile の用途は判然としない。

このような背景から、以下の質問があります:

  • Makefile は現在も機能しているファイルか否か
  • Makefile を削除することは可能か

VOICEVOXのバージョン

0.19.0

@tarepan tarepan added the 要議論 実行する前に議論が必要そうなもの label May 18, 2024
@Hiroshiba
Copy link
Member

昔docker版を作るときに便利コマンドとして配置された、という感じの記憶が微かにあります!
ただテストされていないため、いつの間にか使えなくなってることがあるかもです。

コマンド集として便利そうですが、もしかしたら然るべきところにドキュメントとして書く方が良いのかもとか思いました。
docs/docker.mdを作るとか。

ビルドノウハウがあるかもなので、コメントや引数などをworkflows/build-docker.ymlへの移植を検討しても良いかも。

作成当時から2年が経ち、世間的にmakeユーザーも減少傾向かもなので運用を見直す機会かもしれません。
個人的にはREADMEにコマンドを移すか検討しつつ、

@aoirint さんも良かったら気軽にご意見伺えると🙏

@aoirint
Copy link
Member

aoirint commented May 23, 2024

方針としては、機能しておらず、いろいろと問題を抱えているので、消していいと思います。

  • ドキュメントが存在しない
  • 非サポートになったUbuntu 18.04の消し忘れ
  • arm64版の未サポート

用途としては、以下のようなものがあると思います。

  • Dockerイメージをローカルでビルドするため、Dockerイメージの各バリエーションごとのビルド時オプション--build-argの既定値を提供する
  • Dockerイメージのビルド不具合の調査のため、ステージ--targetごとにタグを付けてビルドする機能を提供する

ドキュメントで代替できるといいと思います。対応関係としては、開発者・貢献者向けガイドの内容になると思います。

@tarepan tarepan changed the title Makefile の用途が知りたい refactor: Makefile を解体 May 25, 2024
@tarepan tarepan added 機能向上 状態:実装者募集 実装者を募集している状態 and removed 要議論 実行する前に議論が必要そうなもの labels May 25, 2024
@tarepan
Copy link
Contributor Author

tarepan commented May 26, 2024

着手します。

@tarepan tarepan added 状態:実装 実装をおこなっている状態 and removed 状態:実装者募集 実装者を募集している状態 labels May 26, 2024
@tarepan tarepan self-assigned this May 26, 2024
@tarepan tarepan removed the 状態:実装 実装をおこなっている状態 label Jun 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants