On Thu, May 3, 2012 at 2:52 AM, barisyanar <[email protected]> wrote: > hi all, > > https://github.com/dhubler/sipxecs/blob/master/sipXpbx/bin/sipx-chkspace.in > > at line 63 Clean as `pwd` did not work for our company system where > voicemail_clean is searched in the path //voicemail_clean. > i think pwd there, assigns the current working directory to the Clean at > line 35,not the directory where sipx-chkspace is present. > i think it would be better to replace `pwd` with @sysconfdir@/cron.daily > should i open a tracker and resolve this?
The unfortunate design of the script is that if a user moves voicemail_clean to a weekly cron, then your fixed directory assumption will be wrong. Proper fix, all scripts should install into $(bindir) or $(libexecdir) and something else should register the cron in /etc/cron.* with symlinks. > > regards > baris > > _______________________________________________ > sipx-dev mailing list > [email protected] > List Archive: http://list.sipfoundry.org/archive/sipx-dev/ _______________________________________________ sipx-dev mailing list [email protected] List Archive: http://list.sipfoundry.org/archive/sipx-dev/
