Angus Leeming wrote: >> < > Doesn't resolve symlinks on Mac >> < >> < and I repeat: this script is to be used to move .fig files into the >> temp directory only. The worst case is that the directories are flagged >> as different and so 'sed' is used rather than 'cp'. Explain to me why >> that is a problem in real life rather than in some hypothetical >> situation. >> >> Sorry, I didn't want to upset you. > > I wasn't upset. > >> Also I didn't realize it was about the >> choice between sed and cp, I didn't read the patch. I thought it was >> about not accidently overwriting some files. > > Ach! Should I test for that? Hmmm. As it stands the script will overwrite > an existing "$2" if 'sed' is used and will not if 'cp' is used. I guess > I'll change that to > 'cp' -f "$1" "$2"
Can you pick any other holes in the attached script? -- Angus
fig_copy.sh
Description: application/shellscript
