Closed bermeitinger-b closed 8 months ago
I think it is indeed a regression in https://github.com/flathub/flatpak-external-data-checker/pull/387. The newly-added git rev-parse --show-toplevel
incantation is failing, because the checkout is not marked as safe – which is what we are trying to fix by calling git rev-parse
.
I opened #396 to revert that change. But the change was trying to fix a real bug.
I have the feeling that we are going about this the wrong way, and instead we should run the container with --user 1001
since the checkout is owned by that user, avoiding the whole "dubious ownership" issue.
OK, I believe that's fixed now. Thanks for the report!
Hi, our pipeline for org.signal.Signal breaks probably because of this commit #387 (or the update to bookworm in #393, which correlates to the dates of the failing jobs)
Action output for the update checker:
Do we need to adapt the workflow commands?