Bug#766764: Support PATH ~ expansion?

[Yuri D'Elia]

On 10/26/2014 10:52 PM, Alexandre Detiste wrote:
> While cron allways use a shell:
> http://anonscm.debian.org/cgit/pkg-cron/pkg-cron.git/tree/do_command.c#n375 ,
> 
> from version 1.3.2, systemd-cron tries to avoid it:
> 
> *) it strips "> /dev/null" of the end of the command line and replace it by
>     "StandardOutput=null"
> 
> *) it translates "[ -x /usr/lib/command ] && /usr/lib/command" into
>    - ExecStart=/usr/lib/command
>    - ConditionFileIsExecutable=/usr/lib/command

This is nice, but I'm just wondering how robust this can possibly be.

If you decide to perform substitutions on the command, then you should
also ensure SHELL is a ksh derivative where this syntax makes sense.

Again, I like the idea, but it's a slippery slope implementation wise.