search

ddev / ddev-redis-7

Redis 7 service for DDEV
Apache License 2.0
3 stars 6 forks source link

sed fails on macos because of missing option -e bevor the script #24

Closed medienagent-user closed 1 year ago

medienagent-user commented 1 year ago

ddev get ddev-redis-7 fails. On macos sed fails, when "-e" is missing before the script. Instead of 

sed -i "s/REPLACE_ME/${DDEV_SITENAME}/g" redis/snapshots.conf

on mac to run it has to be 

sed -i -e "s/REPLACE_ME/${DDEV_SITENAME}/g" redis/snapshots.conf

On Linux both versions work.

Bildschirmfoto 2023-10-18 um 19 05 57
Morgy93 commented 1 year ago

Just ran into this issue. I hope it can be resolved soon. Thanks for providing the PR to it. 🙌

axmue commented 1 year ago

I messed up my pull request. Sorry. The first commit was the correct one.