Track RPL_PATS environment variable and pattern files#110
Track RPL_PATS environment variable and pattern files#110TheVeryDarkness merged 2 commits intoRPL-Toolchain:masterfrom
Conversation
|
TODO: add some basic CI for this |
There was a problem hiding this comment.
Pull request overview
Tracks RPL_PATS (env var value + referenced pattern paths) in rustc dep-info so Cargo/rustc incremental correctly invalidates when RPL’s configuration changes.
Changes:
- Capture
RPL_PATSas aResultin the driver and pass both the raw env var value and parsed paths intoRplCallbacks. - Add
RPL_PATSenv dep tracking and pattern-path file dep tracking inRplCallbacks.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| src/driver.rs | Plumbs RPL_PATS env var value and parsed pattern paths into RplCallbacks. |
| crates/rpl_interface/src/callbacks.rs | Tracks RPL_PATS in env_depinfo and adds pattern-path entries to file_depinfo. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| let rpl_pats_var = env::var("RPL_PATS"); | ||
| let pattern_paths = match &rpl_pats_var { |
There was a problem hiding this comment.
The driver hard-codes the env var name ("RPL_PATS") instead of using a shared constant (similar to rpl_interface::RPL_ARGS_ENV). Consider re-exporting/using RPL_PATS_ENV so changes to the env var name can’t silently diverge across crates.
| let rpl_args_var = self.rpl_args_var.take(); | ||
| let rpl_pats_var = self.rpl_pats_var.take(); | ||
| let rpl_pats = self.pattern_paths.clone(); |
There was a problem hiding this comment.
let rpl_pats = self.pattern_paths.clone(); needlessly clones the full pattern path list just to feed dep-info tracking. Since patterns are already being collected into PATTERNS in this method, you can derive the tracked path list once from that result (or take() the option) and avoid the extra allocation/copy.
This should be able to fix the interaction issue between RPL and rustc's incremental mechanism.