help with a ksh script


 
Thread Tools Search this Thread
Top Forums Shell Programming and Scripting help with a ksh script
# 1  
Old 05-03-2007
help with a ksh script

I am trying to substitute words within multiple files...I posted the first script (https://www.unix.com/showthread.php?t=37505) in tcsh and still for some reason getting nowhere so I tried it in ksh.

Code:
#!/bin/ksh
pattern1=$1
pattern2=$2

if [[ $# != 2 ]]
then
print "Usage: $0 $pattern1 $pattern2"
exit
fi


for file in $(ls)
do
sed "s/$pattern1/$pattern2/g" $file > $file
done

The problem is that when I run the script, it seems it gets to here
Code:
if [[ $# != 2 ]]
then
print "Usage: $0 $pattern1 $pattern2"
exit
fi

and never makes it to
Code:
for file in $(ls)
do
sed "s/$pattern1/$pattern2/g" $file > $file
done

because I don't see any changes made to the .txt files when I do
Code:
more test1.txt

cause I still see the old word there.
# 2  
Old 05-03-2007
you have to use a temp file

sed "s/$pattern1/$pattern2/g" $file > /tmp/file
mv /tmp/file $file

or something like that
# 3  
Old 05-03-2007
Quote:
Originally Posted by csnewbie84
I am trying to substitute words within multiple files...I posted the first script (https://www.unix.com/showthread.php?t=37505) in tcsh and still for some reason getting nowhere so I tried it in ksh.
...
The problem is that when I run the script, it seems it gets to here
Code:
if [[ $# != 2 ]]


Use -ne for numeric comparisons, and don't use the non-standard [[...]]:
Code:
if [ $# -ne 2 ]

Quote:
Code:
then
print "Usage: $0 $pattern1 $pattern2"
exit
fi

and never makes it to
Code:
for file in $(ls)


Not only is ls unnecessary, but it will break your script if any filenames contain spaces or other pathological characters. Use:

Code:
for file in *

Quote:
Code:
do
sed "s/$pattern1/$pattern2/g" $file > $file
done

because I don't see any changes made to the .txt files when I do
Code:
more test1.txt

cause I still see the old word there.

You're lucky to see anything at all. Normally, the shell truncates the file before the command is even started.

The standard method is to save the output to a temporary file, then move or copy it over the original file if the command succeeded:

Code:
sed -e '...' FILE > tempfile && mv tempfile FILE

Or copy it first and send the output to the original file:

Code:
cp FILE tempfile
sed -e '...' tempfile > FILE

Some version of sed have an option, -i, that edits the file in situ.

A trick that allows redirection to the same file is:

Code:
{ rm FILE; sed -e '...' > FILE; } < FILE

# 4  
Old 05-03-2007
Quote:
Originally Posted by cfajohnson

Use -ne for numeric comparisons, and don't use the non-standard [[...]]:
Code:
if [ $# -ne 2 ]


Not only is ls unnecessary, but it will break your script if any filenames contain spaces or other pathological characters. Use:

Code:
for file in *


You're lucky to see anything at all. Normally, the shell truncates the file before the command is even started.

The standard method is to save the output to a temporary file, then move or copy it over the original file if the command succeeded:

Code:
sed -e '...' FILE > tempfile && mv tempfile FILE

Or copy it first and send the output to the original file:

Code:
cp FILE tempfile
sed -e '...' tempfile > FILE

Some version of sed have an option, -i, that edits the file in situ.

A trick that allows redirection to the same file is:

Code:
{ rm FILE; sed -e '...' > FILE; } < FILE

SO I don't have to use "$" for FILE right?
# 5  
Old 05-03-2007
Quote:
SO I don't have to use "$" for FILE right?

If it's a variable, of course you do.

In the example, it is just a generic filename. You replace it with the name of your file. If you use a variable, quote it.


Last edited by cfajohnson; 05-03-2007 at 06:47 PM..
# 6  
Old 05-03-2007
Quote:
Originally Posted by cfajohnson
[INDENT]
If it's a variable, of course you do.

In the example, it is just a generic filename. You replace it with the name of your file. If you use a variable, quote it.
Thanks again! I will try that and let you know what happens.
# 7  
Old 05-03-2007
cfajohnson or anyone else,

personal opinion...would you use this in the script

Code:
if [[ $# != 2 ]]
then
print "Usage: $0 $pattern1 $pattern2"
exit
fi

cause I think it don't have to be there...I would thought the the do statement is the important part of the script.
Login or Register to Ask a Question

Previous Thread | Next Thread

10 More Discussions You Might Find Interesting

1. Shell Programming and Scripting

Script to replace lines in ksh Script

Hi All, I am novice to Unix and I need your expert advice for the below task. There is a KSH script file in which I need to replace few line as per the below expectations. So my file look like as # Host Setup Command: Line 1 Line 2 Line 3 Line 4 Line Any... (6 Replies)
Discussion started by: rupid0609
6 Replies

2. Shell Programming and Scripting

Deploy ksh script to file from other script

Hi all, I need to deploy two scripts on around ~100 machines and have only OPSware. Opsware have the option to execute a script, so I am trying to write a script which dose cat > script.ksh <<EOF script to be deployed EOF However the script between the two EOFs gets also executed which... (0 Replies)
Discussion started by: click
0 Replies

3. Shell Programming and Scripting

Help Create dynamic ksh script from a script

I am currently running 2 scripts to gather data for a 3rd script and would like to combine the 2 scripts into one. Having issues with the final output format. Note cannot post URL so replaced the http stuff with (name) in the examples All scripts contain #!/bin/ksh OS = Red Hat Enterprise... (0 Replies)
Discussion started by: pcpinkerton
0 Replies

4. Shell Programming and Scripting

KSH script to run other ksh scripts and output it to a file and/or email

Hi I am new to this Scripting process and would like to know How can i write a ksh script that will call other ksh scripts and write the output to a file and/or email. For example ------- Script ABC ------- a.ksh b.ksh c.ksh I need to call all three scripts execute them and... (2 Replies)
Discussion started by: pacifican
2 Replies

5. Shell Programming and Scripting

passing a variables value from the called script to calling script using ksh

How do i get the value of the variable from the called script(script2) to the calling script(script1) in ksh ? I've given portion of the script here to explain the problem. Portion of Script 1 ============= ----- ----- tmp=`a.ksh p1 p2 p3` if then # error processing fi -----... (10 Replies)
Discussion started by: rajarkumar
10 Replies

6. Shell Programming and Scripting

import var and function from ksh script to another ksh script

Ih all, i have multiples ksh scripts for crontab's unix jobs they all have same variables declarations and some similar functions i would have a only single script file to declare my variables, like: var1= "aaa" var2= "bbb" var3= "ccc" ... function ab { ...} function bc { ... }... (2 Replies)
Discussion started by: wolfhurt
2 Replies

7. Shell Programming and Scripting

tracing a ksh script within a ksh script

I normally trace a script with the ksh -x <script name> and redirect strderr to file. But if you have a script like the examble below...... vi hairy bear=`grep bear animals` if then ksh more_animals fi If I ksh -x hairy it won't trace "more_animals" unless I put a -x in it. Is... (1 Reply)
Discussion started by: shorty
1 Replies

8. Shell Programming and Scripting

how to convert unix .ksh script to windows .batch script

I am using awk in my .ksh script but when I am trying to run in windows its not recognising awk part of the ksh script , even when I changed it to gawk it does not work, this is how my .ksh and .bat files look like. thanx. #!/bin/ksh egrep -v "Rpt 038|PM$|Parameters:|Begin |Date: |End... (1 Reply)
Discussion started by: 2.5lt V8
1 Replies

9. Shell Programming and Scripting

executing a ksh script from another ksh script

Hi, I'm new to unix scripting.How can i call a script from another script. I have a.ksh and b.ksh .I have to call b.ksh from a.ksh after it is successfully exceuted. I tried using #!/bin/ksh -x in a.ksh and at the end i have used /path/b.ksh My problem is it is executing only a.ksh.it... (6 Replies)
Discussion started by: ammu
6 Replies

10. UNIX for Dummies Questions & Answers

SQL Script run in KSH Script

I've got a SQL script that is executed through a UNIX ksh script. It is working fine, but I wanted to add a line to put a date/time stamp in the log file that it generates. This is more of a SQL question, but I'm hoping someone can help me get the date/time...I've changed the script with the... (2 Replies)
Discussion started by: dstinsman
2 Replies
Login or Register to Ask a Question