Skip to content

fix: remove committed developer-local 'output' symlink - #42

Open
armatrix wants to merge 1 commit into
alibaba:mainfrom
armatrix:fix/remove-output-symlink
Open

fix: remove committed developer-local 'output' symlink#42
armatrix wants to merge 1 commit into
alibaba:mainfrom
armatrix:fix/remove-output-symlink

Conversation

@armatrix

Copy link
Copy Markdown

修复 #41 中的问题 1:移除被提交进仓库的 output 软链。

该条目指向开发者本机路径(/Users/hoshinoren/.../tron-comic/output),在任何其他机器上都是死链,导致后端启动时 os.makedirs("output", exist_ok=True)FileExistsError(路径存在但不是目录),uvicorn 直接退出。

  • .gitignore 第 93 行已有 output/ 规则,删除跟踪后不会再被意外提交
  • src/apps/comic_gen/api.py 启动时会自行创建该目录,无需在仓库中保留任何占位

#41 中的问题 2(lockfile 指向内网 registry)建议由维护者在公网环境重新生成 lockfile 修复,不在本 PR 范围内。

The tracked 'output' entry is a symlink pointing to a developer's
local machine path (/Users/hoshinoren/.../tron-comic/output), which
is broken on any other machine and crashes backend startup:
os.makedirs("output", exist_ok=True) in src/apps/comic_gen/api.py
raises FileExistsError because the path exists but is not a directory.

'output/' is already in .gitignore (line 93); the runtime creates the
directory itself, so nothing needs to be tracked here.
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants