Closed lenalebt closed 3 days ago
The relevant part of the log with debugging enabled, which shows it basically skips downloading the tool where it should download it:
##[debug]Evaluating condition for step: 'run'
##[debug]Evaluating: (success() && ((runner.os == 'Linux') || (runner.os == 'macOS')) && !steps.cache.outputs.cache-hit)
##[debug]Evaluating And:
##[debug]..Evaluating success:
##[debug]..=> true
##[debug]..Evaluating Or:
##[debug]....Evaluating Equal:
##[debug]......Evaluating Index:
##[debug]........Evaluating runner:
##[debug]........=> Object
##[debug]........Evaluating String:
##[debug]........=> 'os'
##[debug]......=> 'Linux'
##[debug]......Evaluating String:
##[debug]......=> 'Linux'
##[debug]....=> true
##[debug]..=> true
##[debug]..Evaluating Not:
##[debug]....Evaluating Index:
##[debug]......Evaluating Index:
##[debug]........Evaluating Index:
##[debug]..........Evaluating steps:
##[debug]..........=> Object
##[debug]..........Evaluating String:
##[debug]..........=> 'cache'
##[debug]........=> Object
##[debug]........Evaluating String:
##[debug]........=> 'outputs'
##[debug]......=> Object
##[debug]......Evaluating String:
##[debug]......=> 'cache-hit'
##[debug]....=> 'false'
##[debug]..=> false
##[debug]=> false
##[debug]Expanded: (true && (('Linux' == 'Linux') || (runner['os'] == 'macOS')) && !'false')
##[debug]Result: false
Another option would be to specifically check for the string-return case.
Seems to be an intended change, so checking for string 'false'
seems to be what they want us to do. I'll make a PR to fix it, as it breaks many of my workflows.
In case anyone needs a temporary workaround: Using lenalebt/setup-rclone@main
should work. I do not intend to do further updates over there, so switch back as soon as it's fixed over here :-)
Holy shit! It's 2024, and GitHub Actions still can't make their type system right! https://github.com/actions/cache/issues/1262#issuecomment-1765308602 https://github.com/actions/runner/issues/2238#issuecomment-2157289150 https://github.com/actions/runner/issues/1483 https://github.com/actions/cache/pull/1404
I'm keeping this open to track GitHub Actions' type system stupidity.
https://github.com/actions/cache/pull/1404 has now been reverted by https://github.com/actions/cache/pull/1467.
Yeah, well, the solution I started and you completed should work in both cases, so I think we're good :-). We'll see whether it will be a problem again.
I think they have now learned their lesson on not to push breaking changes on minor version.
I also plan to revert the temporary fix in order not to bloat the codebase.
They also plan to make cache-hit
to output as a string in the next major version.
Just FYI, a release to actions-cache v4.1.0 breaks this plugin. See https://github.com/actions/cache/issues/1466
This is the output I am seeing:
This could be fixed by setting it fixed in this plugin to 4.0.2, or wait for a release of actions/cache. Any idea for a workaround in the meantime?