增加uv的python下载镜像 - #359
Conversation
|
Hi @Mikachu2333, ❤️ 感谢你的贡献!你的 PR 当前基于 |
1 similar comment
|
Hi @Mikachu2333, ❤️ 感谢你的贡献!你的 PR 当前基于 |
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Updates the Python uv recipe to support additional metadata and to configure both PyPI index sources and Python interpreter download mirrors via uv.toml.
Changes:
- Updated recipe metadata (last-updated dates, sauciers).
- Improved
getsrcbehavior with better missing-config handling and printingpython-install-mirrorwhen present. - Refactored
setsrcto write both[[index]]andpython-install-mirror, adding multiple GitHub Release mirror candidates for auto-selection.
|
@ccmywish READY. |
|
另,由于我的配置文件没有其他设置,所以我没有对复杂场景的edge case做过多考虑。遇到了再说 |
ccmywish
left a comment
There was a problem hiding this comment.
所有的 free() 都不要,recipe 里全部不用 free
只考虑 xy.h 和 core.c 去 free()
|
@ccmywish 正在修改。主要想问一下能不能给GitHub的copilot设置一下prompt,让它别挑每个配方的内存泄漏的问题……我就是因为一开始GPT疯狂提示泄露才改的 |
|
这里的 Copilot 留下的只是评论,可以完全忽视。
我们有这个文件,但是似乎这里的 Copilot 没读,可能要加一句 |
关于这里,我在想是否应该这么表述
另,如果你同意我的表述,上述要求应该写在doc中的代码贡献部分才是。要是你坚持recipe中所有内存泄漏一刀切不考虑我就改回去。不过无论怎样代码风格提示文件里面都应该写明 |
其实理论就应该按照你说的这样。但是我希望 recipe 里一个
综上,在我们是一个命令行程序无需管理内存的大前提下,不如一刀切不要在 recipe 里 free,你可以在文档里简要说一下这些。 |
|
@ccmywish READY FOR REVIEW |
|
虽然ready了,但是逻辑还是有些问题的,但是我确实不会改了…… 如果使用 |
|
是的,有点困难,这几天有点事,我稍微空闲了看一下,在你的PR上改 |
|
感激不尽,c我是真写不明白( 另,不着急改的,拖几个月也无所谓,反正理论上来讲镜像文件改一次就完事了,我现在直接手动改了配置也能凑合用 |
|
分享我今天刚看的一篇文章给你:https://mp.weixin.qq.com/s/EDM8IahEzDgyQnbJ8oYUQw
我把你设置为 collaborator 就是因为你持之以恒地在尝试改进,尽管C语言并不是你的强项,但是你依然是一个很好的贡献者。👍
是的,我打算这个功能完成后就发下一个版本 |
b3f6b9a to
d61de4f
Compare
1. 增加uv的python下载镜像 2. 现在uv会同时对pypi和python进行换源
|
这个pr实在太乱了,关了重开一个,顺便把commit msg整理一下 |
|
关于 ## script to set pypi mirror
- input-file = `uv-pypi.py`
## script to set python binary file mirror
- input-file = `uv-python-binary.py`将这个 python 文件直接翻译为C语言,用C语言调用执行这个 python 文件 因为我记得 @happy-game 说 TOML 已经是 Python 默认的库了 |
|
这样能大幅简化 C 语言的工作量。 由于 |
是的,我刚才查了文档,但这样带了来一个致命的问题——先有鸡还是先有蛋 试想一个场景
这时候他想通过chsrc对uv的python下载进行换源,而chsrc提示:你的系统中不存在python,请先下载python再换源…… 此外,这样还会带来python的版本兼容问题,python是从3.11版本加入的toml解析功能,但是现在很多ai学习和学校还在教授python3.8,而要是分别为两个版本适配显然会更显著地增加工作量与复杂度…… 我本人也不支持纯c写解析, 仔细看了看,没有用,只是读取而已,没有写回,果然还是得手动实现…… 我还是再加一堆测试用例凑合用吧,实在是没招了 |
你说的对,我没有想到这个情况。👍
|
考虑过,但pwsh不属于linux自带配置,甚至也不是win自带的配置(win10以上内置5.0),如果借助 PowerShell(不是pwsh),又会出现好多兼容问题
|
👍👍👍 我们去新的 PR 处讨论 |
-scope问题描述
uv换源不完整,还需考虑Python解释器本身下载地址的换源 #309uv换源不完整,还需考虑Python解释器本身下载地址的换源 #358方案与实现
~/.config/uv/uv.toml中配置的改写,相关参考已写在代码内。