Skip to content

Commit 5cdd039

Browse files
author
Robin Chalas
committed
minor #27858 [Console] changed warning verbosity; fixes typo (adrian-enspired)
This PR was merged into the 4.2-dev branch. Discussion ---------- [Console] changed warning verbosity; fixes typo | Q | A | ------------- | --- | Branch? | master | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes* | Fixed tickets | n/a | License | MIT | Doc PR | n/a * Tests pass, but I do not have an installation of MacOS to run tests on. Tests should be unaffected (the test is simply [skipped on MacOS](https://github.com/symfony/console/blob/master/Tests/Command/CommandTest.php#L345)). When a Console Command fails to change the process title on MacOS, a warning is issued to output. This warning is relevant to developers of Console applications, but to end users is largely meaningless and potentially confusing. This PR changes the verbosity of the warning to "very verbose" so it does not interrupt normal usage. I've also fixed a typo in the message ("get" vs. "set"). Commits ------- 86c771a changed warning verbosity; fixes typo
1 parent d96cc7c commit 5cdd039

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

Command/Command.php

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -210,7 +210,10 @@ public function run(InputInterface $input, OutputInterface $output)
210210
if (function_exists('cli_set_process_title')) {
211211
if (!@cli_set_process_title($this->processTitle)) {
212212
if ('Darwin' === PHP_OS) {
213-
$output->writeln('<comment>Running "cli_get_process_title" as an unprivileged user is not supported on MacOS.</comment>');
213+
$output->writeln(
214+
'<comment>Running "cli_set_process_title" as an unprivileged user is not supported on MacOS.</comment>',
215+
OutputInterface::VERBOSITY_VERY_VERBOSE
216+
);
214217
} else {
215218
cli_set_process_title($this->processTitle);
216219
}

0 commit comments

Comments
 (0)