crond: build tweaks

Using 'depends on FEATURE_SYSLOG' is unreliable because
FEATURE_SYSLOG can't be set directly.  A POSIX build with all
applets that need syslog disabled *except* crond may fail.
Revert to using 'select FEATURE_SYSLOG' in crond and exclude
the unused syslog code on Windows.

If crond is unable to run sendmail report this in the error
message.

Update all default Windows configurations.

(GitHub PR #561)

1 files changed, 2 insertions, 2 deletions

diff --git a/miscutils/crond.c b/miscutils/crond.c
index bbb312882..c1e90440a 100644
--- a/miscutils/crond.c
+++ b/miscutils/crond.c
@@ -11,7 +11,7 @@
 //config:config CROND
 //config:       bool "crond (15 kb)"
 //config:       default y
-//config:   depends on FEATURE_SYSLOG || PLATFORM_MINGW32
+//config:   select FEATURE_SYSLOG
 //config:       help
 //config:       Crond is a background daemon that parses individual crontab
 //config:       files and executes commands on behalf of the users in question.
@@ -761,7 +761,7 @@ fork_job(const char *user, int mailFd, CronLine *line, bool run_sendmail)
                 /* Using the fd as input doesn't appear to work with external programs */
                 shell_arg = (char *)"sh -c \""SENDMAIL" "SENDMAIL_ARGS"\"";
                 if ((fdpipe = mingw_popen_fd("sh", shell_arg, "w", -1, &pid)) == -1)
-                        bb_perror_msg("can't execute '%s'", "sh");
+                        bb_perror_msg("can't execute '%s'", SENDMAIL);
 
                 bb_copyfd_eof(mailFd, fdpipe);
                 close(fdpipe);