Sed not working inside bash for loop

Question

I want to replace an image path inside all html files recursively in all folders.

#!/bin/bash

for image in images/*.png
do

  echo "sed -i .bak -e 's|$image|image2/$image|g' app/www/*.html"
  sed -i .bak -e 's|$image|image2/$image|g' app/www/*.html

done

The weird thing is that when I manually execute the command:

  sed -i .bak -e 's|images/add.png|image2/add.png|g' app/www/*.html

everything works fine and the path is replaces. However inside the for loop this doesn't work while the echo part echoes the same line as mentioned above.

Based on the answer it could be a duplicate. But was the question itself a duplicate as well? — Bas van Dijk, Mar 13 '17 at 06:15

score 2 · Accepted Answer · answered Mar 12 '17 at 20:43

2

You need to use " and not ' Otherwise the $image is not evaluated

Edit: it works in the echo because the ' surrounded string is surrounded by " ;)

answered Mar 12 '17 at 20:43

Denny Weinberg

2,294
1
18
33

This fixed it thanks! – Bas van Dijk Mar 12 '17 at 20:48

Sed not working inside bash for loop

1 Answers1