ucfopen/Materia-Widget-Dev-Kit

Docker install option is broken with 2.5.0.0

clpetersonucf opened this issue · 6 comments

docker inspect <image> is now prepending host_mnt to the Source path of the mounts. This breaks the host directory path MWDK uses to install widgets.

/path/to/materia/fuel/app/tmp/widget_packages

is now

/host_mnt/path/to/materia/fuel/app/tmp/widget_packages

MWDK is also no longer correctly identifying the Docker container that Materia is running in (as of https://github.com/ucfopen/Materia/releases/tag/v8.0.0)
Here:

let targetImage = execSync('docker ps -a --format "{{.Image}} {{.Names}}" | grep -e ".*materia-web-base:.* materia-phpfpm" | head -n 1 | cut -d" " -f2');

.*materia-web-base:.* is no longer a thing - can probably simplify this line to

let targetImage = execSync('docker ps -a --format "{{.Image}} {{.Names}}" | grep -e "materia-phpfpm" | head -n 1 | cut -d" " -f2');

make sense. I keep messing with the docker stuff and this is the fragile bit

I think 'materia-phpfpm' is here to stay ... until we switch to python or something

For what it's worth, widget installation seems to otherwise work just fine between Materia v8.0.0 and MWDK v2.5.0 once I made the change in the above comment.

Looks like this is already fixed in #74.

Fixed in 2.5.1.