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.ymlでgnuのsedやsplitなどを.bashrc経由で使えるようにする #701

Conversation

Hiroshiba
Copy link
Member

@Hiroshiba Hiroshiba commented Jun 19, 2023

内容

で出ていたエラーの解決です。

以前にmac環境でbsdではなくgnuのsedなどを使えるようにしたとき、actionsなどが利用するchmodなども置き換わってしまってエラーが発生するようになっていました。
グローバルなPATHを変えるのではなく、bashrcに書き込んでbash実行時のみPATHが変わるようにしてみました。

関連 Issue

close #698

スクリーンショット・動画など

その他

actions内でbashを使っている場合にエラーになりますが・・・大丈夫かな~と踏んでいます。
${{ matrix.sed }}と書かなくて良くなる・初学者の転ばぬ先の杖になる・他にbashrcを利用したいときが出てきたときに便利かなと思ってこうしてみました。

なにか問題が起こったらmatrixを使う方に書き換えましょう・・・!!

追記:問題になっているsetup-pythonはbashを使っているので、効果がないかもしれません・・・。
https://github.com/actions/setup-python/blob/3f824b7ca6388f5e27e362d31352e6456c8e3bfb/src/install-python.ts#L67

@Hiroshiba Hiroshiba requested a review from a team as a code owner June 19, 2023 19:33
@Hiroshiba Hiroshiba requested review from y-chan and removed request for a team June 19, 2023 19:33
@github-actions
Copy link

github-actions bot commented Jun 19, 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 152 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 1415 284 coverage-80%

@Hiroshiba Hiroshiba force-pushed the sedとsplitをmatrixに含めて明示的に実行 branch from 754f150 to 4b45658 Compare June 19, 2023 19:48
@Hiroshiba
Copy link
Member Author

Hiroshiba commented Jun 19, 2023

buildしてみたのですが、関係なさそうなところで落ちました。。。
https://github.com/Hiroshiba/voicevox_engine/actions/runs/5315489665/jobs/9624016274

Traceback (most recent call last):
  File "<string>", line 1, in <module>
ModuleNotFoundError: No module named 'pyopenjtalk'
Error: Process completed with exit code 1.

3.11.3系でもpyopenjtalkに落ちるのかと思ったのですが、落ちていなかったので謎です。
https://github.com/VOICEVOX/voicevox_engine/actions/runs/5303548815/jobs/9599647796

落ちていた方はimageが11.7.7で、落ちてない方は11.7.6でした。
先程マージしたこちらのPRが関係しているかも・・・?

@Hiroshiba
Copy link
Member Author

bash -lすると環境変数が受け継がれなくなったりすることを思い出しました。
この戦略は実現が困難な気がしてきたので、matrixやenvでsedを切り替える方式を模索します。。

@Hiroshiba Hiroshiba closed this Jun 19, 2023
@Hiroshiba Hiroshiba deleted the sedとsplitをmatrixに含めて明示的に実行 branch June 19, 2023 20:59
@Hiroshiba Hiroshiba removed the request for review from y-chan June 19, 2023 21:29
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.

CI: Runner Image macos-11 20230612.1 (macOS 11.7.7)でバイナリビルドに失敗する
1 participant