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.

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

Look for civicrm.standalone.php to bootstrap standalone #193

Merged
merged 1 commit into from
Jun 7, 2024

Conversation

ufundo
Copy link
Contributor

@ufundo ufundo commented Apr 26, 2024

Attempt to get cv playing nice with an updated directory structure for standalone in this PR: civicrm/civicrm-core#29960

I think I've essentially merged the flag file and the settings file... but I noticed the old flag file had some "config" type script in it, and wondering if they were split out in the first place?

To my mind it seems to make sense that this file says "Hi, I'm a standalone installation, run me to bootstap the install (classloader/settings etc)". But I might be missing something.

@civicrm-builder
Copy link

Can one of the admins verify this patch?

@ufundo ufundo marked this pull request as ready for review May 8, 2024 09:10
@ufundo
Copy link
Contributor Author

ufundo commented May 8, 2024

Have updated this to work with the new format as well as the old format - hope this is easier as a transition step so it wouldn't have to merge in sync with civicrm/civicrm-core#29960

Also gone through a bit more thoroughly on other places the directory structure is referenced

@artfulrobot
Copy link
Contributor

I've tested this alongside an install including civicrm/civicrm-core#29960 and it works; cv runs as it should.

@totten
Copy link
Member

totten commented Jun 7, 2024

civibot, test this please

@totten totten merged commit aa0cb7c into civicrm:master Jun 7, 2024
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants