Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Fix cv failure on J3. #25306

Merged
merged 1 commit into from
Jan 11, 2023
Merged

Fix cv failure on J3. #25306

merged 1 commit into from
Jan 11, 2023

Conversation

aydun
Copy link
Contributor

@aydun aydun commented Jan 10, 2023

Overview

cv fails on J3 - see dev/core#4060

Before

Failed to start application error

After

No error

Technical Details

Rebased version of #25287

Comments

This code was changed by #24796 for J4 compatibility.

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

@civibot
Copy link

civibot bot commented Jan 10, 2023

(Standard links)

@civibot civibot bot added the 5.57 label Jan 10, 2023
@aydun aydun mentioned this pull request Jan 10, 2023
Closed
@demeritcowboy
Copy link
Contributor

Thanks @aydun this will get merged but we also need a 5.58 version.

@demeritcowboy demeritcowboy merged commit 1c8b9f5 into civicrm:5.57 Jan 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
5.57
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@aydun @demeritcowboy