lcpu-club / overleaf

PKU LaTeX
https://latex.pku.edu.cn
GNU Affero General Public License v3.0
38 stars 4 forks source link

feat: select compiler image closes #6 #5

Closed HolgerHuo closed 1 month ago

HolgerHuo commented 1 month ago

Description

Related issues / Pull Requests

Contributor Agreement

wzf03 commented 1 month ago

https://github.com/lcpu-club/overleaf/issues/6

leavelet commented 1 month ago

这些镜像在哪里构建?24.10好理解,22.10是啥意思?要不只保留年份?

HolgerHuo commented 1 month ago

这些镜像在哪里构建?

目前没有ci,底包(特别是旧版本)应该无须更新,可以打包完直接上传。

24.10好理解,22.10是啥意思?要不只保留年份?

https://github.com/lcpu-club/overleaf/blob/5181fd30cb5f681c564a06acc91d73eef4b3e440/services/clsi/app/js/DockerRunner.js#L237 上游使用了这样的匹配规则,保留年份需要也修改一下,我猜测应该是打包时的月份。

leavelet commented 1 month ago

这些镜像在哪里构建?

目前没有ci,底包(特别是旧版本)应该无须更新,可以打包完直接上传。

24.10好理解,22.10是啥意思?要不只保留年份?

https://github.com/lcpu-club/overleaf/blob/5181fd30cb5f681c564a06acc91d73eef4b3e440/services/clsi/app/js/DockerRunner.js#L237 上游使用了这样的匹配规则,保留年份需要也修改一下,我猜测应该是打包时的月份。

上游的规则要不就按上游来?底包可能需要改Dockerfile,增加构建脚本

HolgerHuo commented 1 month ago

上游的规则要不就按上游来? 好的。感觉上游不会对这些模块再有更新了,更新的应该在pro版里

底包可能需要改Dockerfile,增加构建脚本

是指deploy.sh? ci直接部署到服务器上?目前有打包好字体的版本吗

wzf03 commented 1 month ago

上游使用了这样的匹配规则,保留年份需要也修改一下,我猜测应该是打包时的月份。

或许只是一个小的标识符,来防止tag冲突的,比如2024.0、2024.1这样递增下去的,没有啥意义

HolgerHuo commented 1 month ago

或许只是一个小的标识符,来防止tag冲突的,比如2024.0、2024.1这样递增下去的,没有啥意义

确实,应该是这样,https://github.com/overleaf/overleaf/wiki/Server-Pro:-Sandboxed-Compiles 示例是从1开始的