Thanks for the update. I'll add the the first part to my script to make sure the path exists. My mistake again.
The reason I don't use the :
at the top is because sometimes people have their php elsewhere which then breaks their script and they have to modify it.<edit>
could use /root/ispconfig/php/bin/php which everyone should have, i used typical functions to keep it simple so should work. let me know your thoughts</edit>
So I usually prefer that they use their own call in crontab and let it find their own path :
10 * * * * root php /root/clamavupdate.php
this way no need to use `which php` or anything. up to the user really.
Also, haven't you just complicated the install a bit too much?
All they need to do is....
wget http://www.djtremors.com/downloads/clamavupdate.php.txt -O /root/clamavupdate.php
and then put their email in the file, then do a test run `php /root/clamavupdate.php` and if all is well, add it to their crontab.
echo "35 */3 * * * root php /root/clamavupdate.php" >> /var/spool/cron/root
# Debian/ubuntu (I think, make sure you check yours)
echo "35 */3 * * * root php /root/clamavupdate.php" >> /var/spool/cron/crontabs/root
thanks for the input...