Page MenuHomePhabricator

Docker config - use civix from repo
Open, Needs TriagePublic

Description

By default buildkit downloads civix but because I patch it from time to time I think it's better to check it out from git

If we just added this to the path

/srv/civicrm-buildkit/bin/civix/bin

then we wouldn't really need to change anything else as I could switch to checked out civix for the lifetime of the container (which I don't normally change 'all the time') - but without that in the PATH it's too short-lived

@AndyRussG - I assume adding to the path is super trivial?

Here is the upstream repo https://github.com/totten/civix

Event Timeline

Change 663294 had a related patch set uploaded (by AndyRussG; author: AndyRussG):

[releng/dev-images@master] Fundraising images: Separate Mediawiki and Civi setups

https://gerrit.wikimedia.org/r/663294

Change 704396 had a related patch set uploaded (by AndyRussG; author: AndyRussG):

[wikimedia/fundraising/dev@master] Update civi image with updated PATH for civix from repo

https://gerrit.wikimedia.org/r/704396

The above patches are ready for review. Also, merging the dev-images patch should take us one step closer to using the WMF image registry.

I just took the simplest approach, as suggested above, and added /srv/civicrm-buildkit/bin/civix/bin to PATH in the image. However, if it's useful, we could eventually add the civix codebase to /src shared on the host, too.

Thanks!! :)

Change 663294 merged by Eileen:

[releng/dev-images@master] Fundraising images: Separate Mediawiki and Civi setups

https://gerrit.wikimedia.org/r/663294

Change 704396 merged by Eileen:

[wikimedia/fundraising/dev@master] Update civi image with PATH for civix from repo

https://gerrit.wikimedia.org/r/704396