Fixes dev/core#4060: Fix cv failure on J3.

[aydun]

Overview

cv fails on J3 - see dev/core#4060

Before

Failed to start application error

After

No error

Technical Details

This code was changed by #24796 for J4 compatibility.

This PR simply reverts to the previous command for J3 leaving J4 as is.

[civibot]

(Standard links)

• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).

[demeritcowboy]

Can you put against 5.58? It sounds like this broke in 5.56.

[demeritcowboy]

@aydun Can you rebase? It would be good to get this into a point release.

[aydun]

@demeritcowboy - see #25306

[totten]

+1 for 5.58-rc variant.

Patch looks plausible from a distance. Any concerns @seamuslee001 @monishdeb?

[seamuslee001]

This seems fine to me just needs to be on the 5.58 branch

[monishdeb]

looks good to me too and yes it should be on 5.58-rc branch

[monishdeb]

@seamuslee001 @totten I have submitted 5.58-rc PR #25320 on behalf of @aydun

[lcdservices]

I think this should be included in a 5.57.x release (and maybe 5.56.x). A lot of people will be upgrading because of the security patches and find their cron breaks.

[aydun]

@seamuslee001 @totten I have submitted 5.58-rc PR #25320 on behalf of @aydun

Thanks @monishdeb

So we have:

• Fixes dev/core#4060: Fix cv failure on J3. #25287 on master
• Fix cv failure on J3. #25306 on 5.57
• Fix cv failure on J3. #25320 on 5.58

Should cover most bases!

[demeritcowboy]

5.57 now merged. This was magically upmerged into master already via 5b8696f