topic Re: Chaning a string using sed giving parse error in Operating System - HP-UX

Chaning a string using sed giving parse error

Enda Martin_1 — Fri, 24 Jan 2003 10:46:23 GMT

Hi,
I'm trying to chg a string in a file with another string within a loop, but am getting a parse error on the sed operation :

for e in `cat $hm/l1`
do
e0=`$hm/i1 $e`
sed -e 's/'$e'/'$e0'/g' $hm/dw > $hm/dw1
done

Error is
sed: Function s/030113/ Mon cannot be parsed.
sed: Function s/030114/ Tue cannot be parsed.

where file l1 contains
030113
030114
and job i1 returns a day

Any ideas - Appologies if it's something obvious

Enda

Re: Chaning a string using sed giving parse error

Dietmar Konermann — Fri, 24 Jan 2003 11:17:50 GMT

Looks like your "i1" jobs returns a trailing blank.

Try this instead:

$hm/i1 $e | read e0

Best regards...
Dietmar.

Re: Chaning a string using sed giving parse error

H.Merijn Brand (procura — Fri, 24 Jan 2003 11:28:51 GMT

Nah, I guess that either $e or $e0 contains a slash, making the sed command bogus.

My first suggestion is to use double quotes to make the command more readable:

sed -e "s/$e/$e0/g" $hm/dw >$hm/dw1

to be more lenient to the pattern, use perl's alternate syntax for s///, that would catch this:

perl -pe "s{$e}{$e0}g" $hm/dw >$hm/dw1

Enjoy, have FUN! H.Merijn

Re: Chaning a string using sed giving parse error

Jean-Louis Phelix — Fri, 24 Jan 2003 11:29:20 GMT

hi,

Or perhaps i1 returns '/' characters ... You can use any character as a separator instead of /. Try for example ';' :

sed -e 's;'$e';'$e0';g' $hm/dw > $hm/dw1

Regards.

Re: Chaning a string using sed giving parse error

Enda Martin_1 — Fri, 24 Jan 2003 11:31:20 GMT

Dietmar,
thanks for that. This has lead me to another problem in that the second string 030114 converts ok but the first string 030113 doesn't convert at all...it's as if sed is ignoring the first loop(if I put in a grep to see if the correct strings are been used and are in the dw file everything looks ok)

Cheers.

Re: Chaning a string using sed giving parse error

Rodney Hills — Fri, 24 Jan 2003 15:30:28 GMT

Enda,

It looks like if $hm is set before the loop, then $hm/dw and $hm/dw1 will be the same value for each iteration of the loop.

Thus $hm/dw1 will have the last sed command run, which is your 030114. The 030113 is overwritten.

HTH

-- Rod Hills

Re: Chaning a string using sed giving parse error

Rodney Hills — Fri, 24 Jan 2003 15:48:29 GMT

If you are trying to do multiple changes to a file with sed, then put the s/// commands in a file and use:

sed -f myfile $hm/dw > $hm/dw1

This would require building "myfile" with s/// commands from your $hm/l1 file.

If you used perl, then you could do something like

perl -en '$hm=ENV{"hm"};$x=`$hm/i1 $_`; print "s/$_/$x/g\n";}' $hm/l1 >myfile

to create "myfile".

But if you used perl, you could do it with using "sed".

HTH

-- Rod Hills

Re: Chaning a string using sed giving parse error

H.Merijn Brand (procura — Fri, 24 Jan 2003 16:38:10 GMT

Rodney, I hope you ment

perl -en '$hm=ENV{hm};chomp($x=`$hm/i1 $_`); print "s/$_/$x/g\n";}' $hm/l1 >myfile

backticks are not (yet) autochomped

Re: Chaning a string using sed giving parse error

Rodney Hills — Fri, 24 Jan 2003 21:17:06 GMT

I'm bad about chomp's sometime, especially on the one-liners. Thanks for keeping me on my toes.

-- Rod Hills