add libclang in the nix development shell - #24
Conversation
Reviewer's guide (collapsed on small PRs)Reviewer's GuideThe flake now supplies a pinned libclang toolchain dependency, makes it available during package builds and in the development shell, and configures LIBCLANG_PATH so bindgen-based dependencies such as ffmpeg-sys-next do not rely on an incomplete host installation. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review. 📝 WalkthroughWalkthroughThe Nix flake adds an LLVM package binding. The ChangesLLVM Clang environment
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~5 minutes Change: Bug fix Merge Risk: ⚪ Minimal · up to The change directly addresses the reported ffmpeg-sys-next build environment issue without introducing an identified merge-blocking risk. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
same as title,
also this prevent
ffmpeg-sys-nextbuilds from failing when host libclang is incomplete.Summary by Sourcery
Provide a complete, reproducible libclang environment for Nix development and builds.
Bug Fixes:
Enhancements:
Build:
Summary by CodeRabbit