-
Notifications
You must be signed in to change notification settings - Fork 105
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
cargo raze
fails on macOS if /tmp/cargo-raze/doesnt/exist
doesn't exist
#545
Comments
I got this error in Linux as well. Additionally this failure has been occurring in one of the CI jobs in my local fork: https://github.com/yatbear/tensorboard/actions/runs/4491201369/jobs/7899333584, but weirdly not in the upstream CI. |
Hi cargo-raze team, This error is now occurring in our main repo now: https://github.com/tensorflow/tensorboard/actions/runs/4574349463/jobs/8078646809 Could you please provide some pointers for this? Thanks! |
This seems to be coming from Keats/tera#819, as far as I can tell. |
As mentioned in Keats/tera#819, there was a regression in how non-existent glob paths are handled. Previously, invalid globs would be expanded to the empty set, but now they lead to an error. Since this implementation just wants a bogus template directory, we just load the `Tera::default()` template. This should address google#545 Signed off by: Nick Groszewski <[email protected]>
Note, we avoid this error in the meantime by specifying |
I saw this when running on Circle CI too, I had to add a line to |
Probably just need to first create the path instead of relying on certain behavior at Tera's level? |
Problematic code:
cargo-raze/impl/src/rendering/bazel.rs
Line 78 in afe9b48
But if I create
/tmp/cargo-raze/doesnt/exist
it works:env:
The text was updated successfully, but these errors were encountered: