Upgrade to humbug/box 4 and remove support builds for php < 8.1

[icanhazstring]

Replace #592

[theofidry]

Hmm I'm looking at the produced PHAR there is definitely something very weird, need to check some more. It's the sort of error that if it was produced on the repo, the entire CI would fail. So really wondering what caused this

[theofidry]

Nevermind it was me being on the right branch but the wrong commit. Can you retry with 4.6? It should just work (also you can remove the --no-parallel)

[icanhazstring]

Will check, thx

[theofidry]

Hm weird that you get the same error, can you reproduce it locally? (I couldn't)

[icanhazstring]

I think there is another problem.
As I "quickly" removed PHP 8.1.

Will have to check 😅

[theofidry]

btw I see you have some annoyances with the required builds, if I may I highly suggest to opt for something like here. It offers much more flexibility and avoid to have to update the protection rules whenever you upgrade the matrix

[icanhazstring]

@theofidry still same problem, but only in the action itself.
Locally it runs fine.

DateError was just defined with php 8.3 so the error does not make sense.
Seems like the PHP_VERSION_ID constant has a weird value when the action runs.

[icanhazstring]

Nevermind, found some more information about what could go wrong.
I think it has something todo with the

humbug_phpscoper_expose_class('DateError', '__ComposerUnused__\DateError');

inside the phar vendor/autoload.php.
Will dig a little deeper.

[icanhazstring]

Here are the autoload.php from local and action
autoload-action.txt
autoload-local.txt

The only difference I find is that the DateError expose has a different order.
Don't know if that might be a problem.

[theofidry]

Got a laptop back so will try to check it out ASAP