-
Notifications
You must be signed in to change notification settings - Fork 92
refactor: source cache into a separate crate #1860
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
base: main
Are you sure you want to change the base?
Conversation
Resolved conflicts: - Combined workspace members (rattler_git, rattler_build_source_cache, rattler_build_recipe_generator) - Updated rattler dependency versions to match main - Kept source cache refactoring with cache_adapter instead of duplicate fetch_source function - Regenerated Cargo.lock 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
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.
Maybe we can add in rattler_build_source_cache
another module, named cache
where we can put cache.rs
and builder.rs
?
authors.workspace = true | ||
license.workspace = true | ||
repository.workspace = true | ||
description = "Source cache management for rattler-build" |
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.
only for GIT and url right?
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.
yeah but we don't cache "path" sources (as they are on disk already) :)
|
||
## Usage | ||
|
||
```rust |
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.
should we make it as a doctest for example in lib.rs?
This PR splits up
rattler-build
to add additional crates:rattler_build_source_cache
contains a module to fetch and cache sources (URL and Git).rattler_git
is a copy frompixi_git
(which comes fromuv_git
) and takes care of checking out git repositories. This should ideally – if things work out – make it's way torattler
.