i think we can do this at least for default, since
https://github.com/dhubler/sipxecs/blob/release-4.4/sipXpbx/sipxpbx.spec.in
contains
the lines:
%attr(755,root,root) %{_sysconfdir}/cron.daily/sipx-chkspace
%attr(755,root,root) %{_sysconfdir}/cron.daily/voicemail_clean
%attr(755,root,root) %{_sysconfdir}/cron.daily/sipx-core-clean
%attr(755,root,root) %{_sysconfdir}/cron.daily/sipx-cert-check
On Thu, May 3, 2012 at 12:32 PM, Douglas Hubler <[email protected]> wrote:
> 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/
>
_______________________________________________
sipx-dev mailing list
[email protected]
List Archive: http://list.sipfoundry.org/archive/sipx-dev/
