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

[Enhance] 对 Docker 支持 jqyq 实现自动换源 #100

Closed
ccmywish opened this issue Oct 12, 2024 · 3 comments · Fixed by #109
Closed

[Enhance] 对 Docker 支持 jqyq 实现自动换源 #100

ccmywish opened this issue Oct 12, 2024 · 3 comments · Fixed by #109
Assignees
Labels
改善加强 改善加强 软件 软件
Milestone

Comments

@ccmywish
Copy link
Contributor

#35 (comment)

@ccmywish ccmywish added 软件 软件 改善加强 改善加强 labels Oct 12, 2024
@ccmywish ccmywish added this to the v0.2.0 milestone Oct 12, 2024
@happy-game
Copy link
Contributor

既然目前的源文件只有docker是使用json格式,那我感觉直接使用sed替换"registry-mirrors": [xxx]为"registry-mirrors": [src"]更为合适,因为一些发行版(如debian)没有预装jq.
或许可以在换源前判断是否存在jq来决定使用jq还是sed换源.

@ccmywish
Copy link
Contributor Author

Hi, @happy-game 又见面了

我目前不用 docker,所以我不知道它的配置文件细节到底是什么样的,如果 sed 能做到,当然直接就用 sed 了。有兴趣解决这个问题吗?时间方面看你个人自由,没有任何要求。

@happy-game
Copy link
Contributor

我是看#35 (comment)中你提到

目前 chsrc 支持的60多个目标中,只有 docker 使用 JSON 作为配置文件(其他使用 JSON 的可以自动换源),另外有3个用 toml,1个用 yml,1个用xml,这些都无法自动换源。所以我更想引入 yq 而非 jq 来处理。但由于上述发现,并没有引入。

所以我觉得可以不用 jq, 而直接使用sed.

我会试着去解决这个问题.

@ccmywish ccmywish linked a pull request Oct 28, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
改善加强 改善加强 软件 软件
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants