SED and MV question

wendtbd · 08-22-2006, 08:12 AM

I am a Linux newbie and have a question about scripting using SED and MV. Below is a set of command I am running and would like to combine into one if possible.

for i in *; do cat "$i"|sed -e 's/ns3.domain.com./ns1.newdomain.com./' >>"$i".new; done

for i in *; do cat "$i"|sed -e 's/ns4.domain.com./ns2.newdomain.com./' >>"$i".new; done

Second I am trying to overwrite the original files with the .new files I am creating above but can't figure out how to remove the .new extension. An example of what I want to do, in a for loop of course, is below.

mv zone.dns.new zone.dns

Here is what I have which doesn't work of course.

for i in *; do mv "$i" "$i"; done

Any suggestions would be appreciated,

Barrett

unSpawn · 08-22-2006, 08:35 AM

Hello and welcome to LQ, hope you like it here.

Combine like this?:

Code:

#!/bin/sh
for i in *; do cat "$i"|sed -e 's/ns3.domain.com./ns1.newdomain.com./' \
-e 's/ns4.domain.com./ns2.newdomain.com./' >>"$i".new; mv "$i.new" "$i" 
done
# always clean exit a script
exit 0

BTW, you don't want this:

Code:

for i in *; do

but something similar to:

Code:

find /some/dir -type f -name "domain*.conf"|while read i; do

wendtbd · 08-22-2006, 09:01 AM

Yeah, looks like a great forum and a great place for a newbie to learn a lot...

I guess when you combine all the commands it is pretty easy to overwrite the original file. I have ran this and it worked great, thanks for the help.

Just for future reference how would I do this with the mv command by itself? Using the command;

for i in *; do mv "$i" "$i"; done

Can I add something to the second "$i" to remove (in my case) the .new extension? I will use the find /some/dir code you suggested above from now on, seems a little safer way to go.

Thanks again
Barrett

unSpawn · 08-22-2006, 02:46 PM

Can I add something to the second "$i" to remove (in my case) the .new extension?
Maybe something like:

Code:

find /some/dir -type f -name "*.new"|while read i; do
mv "$i" "${i%.new}"; done

wendtbd · 08-23-2006, 11:22 AM

Thanks again unSpawn, this works great.