fix(sources): catch FileNotFoundError for broken symlinks#1087
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
✅ Files skipped from review due to trivial changes (2)
📝 WalkthroughWalkthroughThe PR updates Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes 🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
@mergify rebase |
Add FileNotFoundError to except clause and add test for broken symlinks. Closes: python-wheel-build#1082 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Signed-off-by: Andre Lustosa <alustosa@redhat.com>
|
Deprecation notice: This pull request comes from a fork and was rebased using |
✅ Branch has been successfully rebased |
172b371 to
9589c5b
Compare
Closes: #1082
Pull Request Description
What
Add FileNotFoundError to the except clause in scan_compiled_extensions and add a regression test.
Why
Broken symlinks (e.g. .dockerignore → missing .gitignore in PyTorch tarballs) crash the bootstrap with FileNotFoundError since 0.80.0.