Skip to content

Conversation

@kimushun1101
Copy link
Member

@kimushun1101 kimushun1101 commented Dec 1, 2024

  • Add git and openssh in devcontainer #69 でDevcontainer上でも開発が行えるようになりましたが、Devcontainer上でビルドしてしまうと生成物がroot権限で作られます。
  • Devcontainer外で、例えばmise run generateなどをするとファイルが上書きできずにエラーとなってしまいます。
  • そのため https://code.visualstudio.com/remote/advancedcontainers/add-nonroot-user を参考に、rootでない一般ユーザーであるtypst-jpユーザーを作成しました。
  • mise run previewに合わせてWebサーバーのコマンドをpython3 -m http.server -d distに変更しました。それに伴いNode.jsのインストールは削除しました。 chore: textlintを導入 #83 が終わったら後追いでまた導入するかもしれませんが。
  • npx serve -n ./distではポート番号が3000番でしたが、python3 -m http.server -d distでは8000番に変更されています。問題ないでしょうか?
  • Ubuntu 22.04 ネイティブでテストした際に https://gist.github.com/Seasawher/18f5781e372e4fbe691ee8942828bd8e と同じ問題が発生したため、"postCreateCommand": "git config --global --add safe.directory ${containerWorkspaceFolder}",を追加しました。

Signed-off-by: Shunsuke Kimura <kimushun1101@gmail.com>
Signed-off-by: Shunsuke Kimura <kimushun1101@gmail.com>
Signed-off-by: Shunsuke Kimura <kimushun1101@gmail.com>
Signed-off-by: Shunsuke Kimura <kimushun1101@gmail.com>
@kimushun1101 kimushun1101 requested a review from 3w36zj6 December 1, 2024 08:09
@kimushun1101 kimushun1101 self-assigned this Dec 1, 2024
@kimushun1101
Copy link
Member Author

Devcontainer前提で、一般ユーザー名をvscodeとしましたが、Dockerfile単体で使うことも想定するとtypst-jpなどのユーザー名の方が良さそうでしょうか?Dockerfile単体で使える人はmiseも使えそうな気もしますが。

@3w36zj6
Copy link
Member

3w36zj6 commented Dec 11, 2024

Development ContainersはVS Codeに固有のものではないので、ユーザー名はtypst-jptypst-jp.github.ioなどの命名がより適切だと思います。

Signed-off-by: Shunsuke Kimura <kimushun1101@gmail.com>
@kimushun1101
Copy link
Member Author

コメントありがとうございます。変更いたしました。

@kimushun1101 kimushun1101 merged commit b565060 into typst-jp:main Dec 12, 2024
3 checks passed
@kimushun1101 kimushun1101 deleted the fix/devcontainer-to-non-root-user branch December 12, 2024 15:48
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