增加uv的python下载镜像#359
Conversation
1. 增加uv的python下载镜像 2. 现在uv会同时对pypi和python进行换源
|
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,你可以在文档里简要说一下这些。 |
-scope问题描述
uv换源不完整,还需考虑Python解释器本身下载地址的换源 #309uv换源不完整,还需考虑Python解释器本身下载地址的换源 #358方案与实现
~/.config/uv/uv.toml中配置的改写,相关参考已写在代码内。