Ptt-official-app/go-pttbbsweb

remove GO_PTTBBS_VERSION in scripts/docker_build.sh

chhsiao1981 opened this issue · 1 comments

Provide the github link(s) of the file#line (請提供 github 關於 code 的 link)

https://github.com/Ptt-official-app/go-openbbsmiddleware/blob/main/scripts/docker_build.sh
https://github.com/Ptt-official-app/go-openbbsmiddleware/blob/main/docker/Dockerfile

What would you like to discuss (你想要討論什麼呢?~)

目前 MAX_USER / MAX_BOARD 在 go-pttbbs/ptttype/00-config.go 裡.
這會在 ptttype.Bid.IsValid 和 ptttype.UID.IsValid 時用到.

所以目前需要 go-pttbbs/01-config-docker.go.template 來讓 compile 時 MAX_USER / MAX_BOARD 是跟 docker 裡的 number 是一致的.

MAX_USER / MAX_BOARD 在 00-config.go 裡的原因是會被 cache.SHMRaw 這個 struct 用到.
所以是需要是 const 而無法在 runtime 時決定.

在設定 version 時常常會忘記 sciprts/docker_build.sh 也需要設定 go-pttbbs 的 version.

需要想個 work-around 的方式而不需要在 docker_build.sh 也需要設定 go-pttbbs 的 version.

#466

  1. GO_PTTBBS_VERSION is from go.mod
  2. use go mod download in Dockerfile. No need GO_PTTBBS_VERSION anymore.