13

I would like to make a script that deletes a directory with rmdir after confirming with a password using read to set the variable.

So far I have this:

#!/bin/bash -x
echo "Password:"
read -t 30 S1
    S2='55555'
if [ $S1=$S2 ]; then
    rmdir /home/william/test
else
    echo "fail"
sleep 10
fi

So, I have the -x to try to debug it but every time the script either fails to echo (if I put the password in wrong) or it wont remove the directory needed.

If someone has a modifiable script that I could use or if you could point out the problems with the current script that would be great.

3 Answers 3

17

The right way to compare those two strings (S1 and S2) using if is:

if [ "$S1" = "$S2" ]

Do not be stingy in use spaces in this case.

See: http://tldp.org/LDP/Bash-Beginners-Guide/html/sect_07_01.html

1
  • TLDP is not an authentic site as it has a lot of inconsistencies and errors. I would advise that we don't use it as a reference. Mar 4, 2018 at 18:12
6

In bash scripting you need to compare two variable with below method.

if [ "var1" != "var2" ]; then
  Do something
fi;

Spaces are important

4

You may also use GNU test, for example:

test s1 = s2 && echo Equal || echo Not equal

In your context, it is:

test "$S1" = "$S2" && rmdir -v /home/william/test

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Not the answer you're looking for? Browse other questions tagged or ask your own question.