ci: use local sources when building with Bake

[dunglas]

Fixes #1993

Because of a change introduced in the latest version of the Docker Bake Action, Bake downloads the sources from the main Git branch instead of using the local Git tree.

[henderkes]

It's failing with what already cost me some headaches. I forgot when this was failing and what caused it, but just commented out the checkout logic.

I feel like we should not do a checkout for a release version and instead let SPC handle that (it supports downloading a specific version). In case no version is specified, just stay on the current branch (aka don't do anything, also if there's no git folder).

What do you think?

[dunglas]

Good idea, but we should also fix the "dev" workflow. Maybe we could let SPC check out the current Git commit?

[dunglas]

Btw, there are shallow cloning-related errors in the GNU build, which don't make the build fail.

[henderkes]

We don't need to! See: #1968 we can set FRANKENPHP_SOURCE_PATH to build with the current directory. If we don't have the version of the code we want to build with available, spc can download it with the download command (tags, not commits).

[henderkes]

Btw, there are shallow cloning-related errors in the GNU build, which don't make the build fail.

Yeah, they were caused by cross-device copying or something, can be ignored.

[henderkes]

Can you rebase on top of main please?

[dunglas]

@henderkes the infamous "#30 ERROR: write /go/src/app/dist/static-php-cli/buildroot/bin/frankenphp: no space left on device" is back

[dunglas]

Let's try this: https://github.com/php/frankenphp/pull/1997/files#diff-63d95b30841a27f0b6dda7d474c3c82144d75edd5cc6c3d4e022ea9e26629ba6R230

[henderkes]

Oh I think I get it, it doesn't use the build-static-gnu.Dockerfile, does it?

The last step is the one responsible for deleting the source and download folders.