Skip to content

Commit 49b70a7

Browse files
committed
Add code comment about an exit() in Drupal installer.
1 parent 5a1cc7b commit 49b70a7

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

src/Commands/core/SiteInstallCommands.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -138,6 +138,8 @@ public function install(array $profile, $options = ['db-url' => self::REQ, 'db-p
138138
require_once Path::join(DRUSH_BASE_PATH, 'includes/site_install.inc');
139139

140140
require_once DRUSH_DRUPAL_CORE . '/includes/install.core.inc';
141+
// This can lead to an exit() in Drupal. See install_display_output() (e.g. config validation failure).
142+
// @todo Get Drupal to not call that function when on the CLI.
141143
drush_op('install_drupal', $class_loader, $settings);
142144
if (empty($options['account-pass'])) {
143145
$this->logger()->success(dt('Installation complete. User name: @name User password: @pass', ['@name' => $options['account-name'], '@pass' => $account_pass]));

0 commit comments

Comments
 (0)