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ディレクトリを整理したい #2267

Open
3 tasks
sevenc-nanashi opened this issue Sep 11, 2024 · 3 comments
Open
3 tasks

buildディレクトリを整理したい #2267

sevenc-nanashi opened this issue Sep 11, 2024 · 3 comments

Comments

@sevenc-nanashi
Copy link
Member

内容

build/下が混沌としてきたので整理したいです。

Pros 良くなる点

すっきりする

Cons 悪くなる点

ないはず?

実現方法

軽く分類しました。

配布周り:
icons
README.txt

electron-builder周り:
afterAllArtifactBuild.js
afterNsisWebArtifactBuild.js
codesign_cleanup.bash
codesign_setup.bash
funcs.nsh
installer.nsh
installer_linux.sh

ダウンロード:
vendored
download7z.js

スクリプト:
generateLicenses.js
mergeLicenses.js
splitNsisArchive.js

その他:
patches

VOICEVOXのバージョン

0.20.0

OSの種類/ディストリ/バージョン

  • Windows
  • macOS
  • Linux

その他

(なし)

@Hiroshiba
Copy link
Member

Hiroshiba commented Sep 11, 2024

整理したいところですね!!

個人的には、まあファイル2個ぐらいだったらディレクトリに分けなくても良いのでは、という気がしなくもないです!
というルールに従って僕も整理してみました。と言っても真似してelectron-builderディレクトリを作るだけですが!

  • build/
    • README.txt
    • icons/
    • vendored/
    • patches/
    • electron-builder/ ← new !
      • afterAllArtifactBuild.js
      • afterNsisWebArtifactBuild.js
      • funcs.nsh
      • installer.nsh
      • splitNsisArchive.js
    • codesign_cleanup.bash
    • codesign_setup.bash
    • download7z.js
    • generateLicenses.js
    • mergeLicenses.js
    • installer_linux.sh

(コードサイン系とinstaller_linux.shelectron-builderから外してみました)
うーーーーーーーん。
scriptsは結局そこが太っていく気もするのと、スクリプトじゃないものを置きたくなってくるんですよね~。(エンジンの経験談)

追加でコードサイン系のcodesigin/と、ライセンス系のlicenses/はアリ・・・・かも・・・?これでファイル数は半分ぐらいになるはず。。
とりあえず、少なくともelectron-builder周りをどけるのはとても良さそう!!これだけでも結構見やすくなりそう。

@sevenc-nanashi
Copy link
Member Author

そもそもbuild/ディレクトリ自体がなんかビルド成果物感があるんですよね

@Hiroshiba
Copy link
Member

あ、ですね!
というかビルドに関係ないものも含まれているので、toolsとかにするのが良いのかな~と。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants