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:
Materia-Widget-Dev-Kit/express.js
Line 468 in 420f5c0
.*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.