-
-
Notifications
You must be signed in to change notification settings - Fork 257
更新uv,使之能在win系统上使用 #311
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
更新uv,使之能在win系统上使用 #311
Conversation
Hi @Mikachu2333 ❤️ 感谢你的贡献!我们将在最少半小时,最多5天内阅读此 PR 并回复你 |
目前 目前的想法是我先写死一个镜像源,后面靠大佬把这个新的镜像源(可以叫 |
commits的地狱命名转正式再说,现在不想动它 |
1153c7a
to
2500b04
Compare
理论上来说是能用的,本地测试也没问题。但是由于逻辑太要命,检查实在是看不动了,还请帮忙看看逻辑漏没漏…… |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
The PR updates the UV configuration handling to enable proper functionality on Windows systems and adds support for UV Python mirror switching. The changes implement a comprehensive configuration file management system that respects UV's configuration file priority order and handles both uv.toml
and pyproject.toml
files.
Key changes:
- Enhanced cross-platform Windows support with proper path handling
- Added comprehensive UV configuration parsing for both Python install mirrors and PyPI sources
- Implemented configuration file priority system following UV's specification
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.
File | Description |
---|---|
src/recipe/lang/Python/uv.c | Complete rewrite of UV configuration handling with new data structures, path resolution, and configuration management functions |
src/recipe/lang/Python/rawstr4c.md | Updated documentation to reflect new configuration format for both uv.toml and pyproject.toml files |
src/recipe/lang/Python/rawstr4c.h | Updated raw strings to match new configuration templates, removing grep-dependent commands |
我认可你的毅力和精神,但是这种改动已经如同我们上次提到的情况:过于复杂,受益甚小。
|
我觉得你应当考虑做减法而不是做加法,这样费劲去做解析,不但维护非常非常困难,以现在的人手来看,也不会有人有足够的耐心和时间去审查这么多代码,这导致最终维护者只有你一人。 如果你真的想做到解析,我认为你应当认真考虑使用别的语言去做,你用别的语言做这件事,是事半功倍的。你甚至可以单独写一个小的exe文件, 当你开始一个工作的时候,你应当从宏观的角度去先权衡一下对整个项目的利弊以及成本和收益。 |
|
我之前考虑过这事来着,但是考虑到调用安全的问题于是放弃……目前在想我是不是应该写个库出来,然后chsrc编译的时候直接静态编译进去…… c在不调库的情况下确实太难用了( |
这玩意先搁置吧,等我再想想怎么办…… 目前这解析着实是令人有些撑不住了,无论是逻辑、代码量还是代码的屎山程度对于一个不熟悉c的贡献者来说都是在有些难绷 |
问题描述
uv
无法在 win 上使用且uv python
换源缺失uv
换源不完整,还需考虑Python
解释器本身下载地址的换源 #309uv
换源需要grep
#308方案与实现
PlPythonUvConfig
、pl_python_uv_config_define_python
、pl_python_uv_config_define_pypi
、PlPythonUvConfigSummary
、pl_python_uv_config_summary_struct_is_full
、pl_python_uv_config_find
等)