-
Notifications
You must be signed in to change notification settings - Fork 15
5.24 and earlier don’t have .git/ ? #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
Comments
It looks like the https://github.com/FGasper/p5-TOML-XS/runs/2302718053?check_suite_focus=true#step:3:25 Not quite sure what's going on there, since they all use the same base Ubuntu. |
Weird. Yeah, even when I put |
Actually the 5.24 ones report being on stretch; 5.26 report buster. @oalders |
The checkout action cannot find git >= 2.18 and so use the API to download a tarball You would have to initialize the git repo and set the submodule if needed manually https://github.com/FGasper/p5-TOML-XS/runs/2313744629?check_suite_focus=true#step:6:25
|
IMO this is not an issue with the image but the way git checkout action works... We probably want to rebuild the images with a more modern git binary... I guess that GitHub does not use older version of git due to CVEs |
https://github.com/FGasper/p5-TOML-XS/actions/runs/731758471
^ If you look at the failures here, they all fail because there’s no .git/ directory.
5.26:

https://github.com/FGasper/p5-TOML-XS/runs/2302718045?check_suite_focus=true
5.24:

https://github.com/FGasper/p5-TOML-XS/runs/2302718053?check_suite_focus=true
Is this an intentional difference between the images?
Thank you!
The text was updated successfully, but these errors were encountered: