Skip to content

Commit a03b327

Browse files
author
Marius Burkard
committed
Merge branch '5885-fix-cron-not-found-on-install' into 'develop'
Resolve "New install finishes with [ERROR] Cron not found" Closes #5885 See merge request ispconfig/ispconfig3!1306
2 parents 1fd891d + bf140b2 commit a03b327

File tree

2 files changed

+7
-5
lines changed

2 files changed

+7
-5
lines changed

install/install.php

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -172,6 +172,11 @@
172172
//** Detect the installed applications
173173
$inst->find_installed_apps();
174174

175+
//* crontab required by ISPConfig
176+
if(!$conf['cron']['installed']) {
177+
die("crontab not found; please install a compatible cron daemon before ISPConfig\n\n");
178+
}
179+
175180
//** Select the language and set default timezone
176181
$conf['language'] = $inst->simple_query('Select language', array('en', 'de'), 'en','language');
177182
$conf['timezone'] = get_system_timezone();
@@ -607,10 +612,7 @@
607612

608613
//* Configure ISPConfig
609614
swriteln('Installing ISPConfig crontab');
610-
if($conf['cron']['installed']) {
611-
swriteln('Installing ISPConfig crontab');
612-
$inst->install_crontab();
613-
} else swriteln('[ERROR] Cron not found');
615+
$inst->install_crontab();
614616

615617
swriteln('Detect IP addresses');
616618
$inst->detect_ips();

install/lib/installer_base.lib.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -192,7 +192,7 @@ public function find_installed_apps() {
192192
// if(is_installed('vlogger')) $conf['vlogger']['installed'] = true;
193193
// ISPConfig ships with vlogger, so it is always installed.
194194
$conf['vlogger']['installed'] = true;
195-
if(is_installed('cron') || is_installed('anacron')) $conf['cron']['installed'] = true;
195+
if(is_installed('crontab')) $conf['cron']['installed'] = true;
196196

197197
if (($conf['apache']['installed'] && is_file($conf['apache']["vhost_conf_enabled_dir"]."/000-ispconfig.vhost")) || ($conf['nginx']['installed'] && is_file($conf['nginx']["vhost_conf_enabled_dir"]."/000-ispconfig.vhost"))) $this->ispconfig_interface_installed = true;
198198
}

0 commit comments

Comments
 (0)