-
Notifications
You must be signed in to change notification settings - Fork 18
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
Caching not working #72
Comments
Per #59 this should only happen when a commit is specified on the version, not for normal releases, so I hope the impact is ultimately not that big (I don't have a good view of how many people are pinning a commit vs. using a stable release though.) That said it obviuosly still needs to be solved, because a small impact is still an impact. #60 contains a potential fix, it would be quite helpful if someone could validate the approach and finish it up. |
Many (if not most) people use Zig nightly releases rather than stable. I've been using the https://github.com/squeek502/resinator/actions/runs/8220331040/job/22479316486
|
I use the nightly.
|
Thanks for the input. I brought it up because i know setup-zig is used by many C projects for zig's convenient C (cross-)compilation. I'm giving this repo some attention today so this should finally be addressed. @kristoff-it since you are on the zig team ... do you think the ziglang github org or just an individual closer to zig would be interested in taking over this repo? I don't really have the time to maintain it actively (obviously) and also not really the expertise, since I'm not using zig actively at the moment. It would be good to have someone more knowledgeable maintain it |
@goto-bus-stop @kristoff-it I am not part of the zig team, but have a fair number of projects using zig now and would be willing to help. I have a fork started with support for nominated mach engines and I am fairly certain of the cause of the some of the issues cited here. |
I've noticed that all of my recent Actions run present the same message from setup-zig which sees a cache miss when trying to find Zig, and that fails to save because of a cache hit:
To diagnose the problem a bit I've deleted my cached Zig and run two deployments right after (in sequence, not in parallel):
First run (link):
Second run (link, triggered after the first had finished):
Somehow, it seems that the step fails to find existing caches.
This is a big problem because it means that the ZSF incurs in avoidable expenses.
The text was updated successfully, but these errors were encountered: