remove GO_PTTBBS_VERSION in scripts/docker_build.sh
chhsiao1981 opened this issue · 1 comments
chhsiao1981 commented
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.
chhsiao1981 commented
- GO_PTTBBS_VERSION is from go.mod
- use
go mod download
in Dockerfile. No need GO_PTTBBS_VERSION anymore.