Well starting at the top, I would probably use the 'id' command instead of the variable as someone may set it on the command line to something else (if we assume someone is being naughty).
I am not understanding the point of the THISUSER? What I mean is, unless you are root you will not be able to go into another user's account (generally) so if
it is in 'this user' , ie the logged in user, why not simply use $USER, id or even '~' (synonym for home directory)??
I am next a little dubious about firing off a browser, does the accepting of the agreement close the browser?
You are testing your 'input' variable against a set string, "yes", what if the user enters "y" or "Yes"?
Irrelevant of the user's reply yuou do the following:
Code:
rm -r -f /home/$THISUSER/.pepper
I have 2 issues here:
1. If it is a given this will always occur then simply do it prior to testing 'input'
2. You never check to see if the directory actually exists and in the case of the 'else', ie likely the first time this has been run and the says "no", the user will now be presented with an error
from rm saying no such directory
Hope some of this helps