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

Dockerfile (Linux): インデントの修正・entrypointにshebangを追加・entrypointの処理順交換 #99

Merged
merged 3 commits into from
Sep 16, 2021

Conversation

aoirint
Copy link
Member

@aoirint aoirint commented Sep 15, 2021

内容

Dockerfile (Linux) の細かな修正です。

  • インデントをスペース4個に統一
  • heredocにおいて、終端文字列のインデントを削除
    • 終端文字列はインデントをつけると終端文字列として動作しない
      • /entrypoint.shEOT以降の記述が書き込まれる状態になっていた
      • 自動で終了するコマンドを実行した場合、続けて、存在しないEOTコマンドを実行しようとして失敗・異常終了する
  • entrypointにshebangを追加
    • ENTRYPOINT [ "/entrypoint.sh" ] のような書き方ではイメージ実行に失敗したため、実際は使っていない
      • standard_init_linux.go:228: exec user process caused: exec format error
    • bashを想定していることを明示するため追加
  • entrypointのldconfigをREADME表示後に実行

@aoirint aoirint changed the title Dockerfile (Linux): インデントをスペース4個に統一・heredocのインデント削除・entrypointへのshebang追加 Dockerfile (Linux): インデントをスペース4個に統一・heredocのインデント削除・entrypointにshebangを追加 Sep 15, 2021
@aoirint aoirint marked this pull request as ready for review September 15, 2021 23:56
@aoirint aoirint changed the title Dockerfile (Linux): インデントをスペース4個に統一・heredocのインデント削除・entrypointにshebangを追加 Dockerfile (Linux): インデントの修正・entrypointにshebangを追加・entrypointの処理順交換 Sep 16, 2021
@aoirint aoirint marked this pull request as draft September 16, 2021 00:30
@aoirint aoirint marked this pull request as ready for review September 16, 2021 01:05
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! shebangと呼ぶんですね、勉強になりました。

@Hiroshiba Hiroshiba merged commit a5f1af2 into VOICEVOX:master Sep 16, 2021
@aoirint aoirint deleted the patch-linux-docker-fix-indent branch September 18, 2021 18:27
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