From 10e8c922d30528cbf864001130265b91d4f8b627 Mon Sep 17 00:00:00 2001 From: Remi Collet Date: Wed, 8 Oct 2014 07:25:29 +0200 Subject: php-phpunit-PHPUnit: new upstream patch for "no colors" patch, raise dependency on sebastian/environment >= 1.1 --- phpunit-colors.patch | 77 ++++++++++++++++++++++++++++------------------------ 1 file changed, 41 insertions(+), 36 deletions(-) (limited to 'phpunit-colors.patch') diff --git a/phpunit-colors.patch b/phpunit-colors.patch index 847d46c..9c0a8f9 100644 --- a/phpunit-colors.patch +++ b/phpunit-colors.patch @@ -1,42 +1,47 @@ -From 591efd8fad55a526f021cad9c6cbcd9081df8969 Mon Sep 17 00:00:00 2001 -From: Remi Collet -Date: Mon, 6 Oct 2014 10:54:54 +0200 -Subject: [PATCH] only enable colors when output to a terminal (not to a file) +From 3d56c46afb9ed10b0d248fb0ea698aa22d939421 Mon Sep 17 00:00:00 2001 +From: Sebastian Bergmann +Date: Tue, 7 Oct 2014 11:29:29 +0200 +Subject: [PATCH] Rework 591efd8fad55a526f021cad9c6cbcd9081df8969 --- - src/TextUI/Command.php | 7 +++++-- - src/Util/Configuration.php | 3 +++ - 2 files changed, 8 insertions(+), 2 deletions(-) + composer.json | 2 +- + src/TextUI/ResultPrinter.php | 6 +++++- + 2 files changed, 6 insertions(+), 2 deletions(-) -diff --git a/src/TextUI/Command.php b/src/TextUI/Command.php -index 929ca36..42b08b6 100644 ---- a/src/TextUI/Command.php -+++ b/src/TextUI/Command.php -@@ -278,8 +278,11 @@ protected function handleArguments(array $argv) +diff --git a/composer.json b/composer.json +index 9dc9950..ae6ff27 100644 +--- a/composer.json ++++ b/composer.json +@@ -30,7 +30,7 @@ + "symfony/yaml": "~2.0", + "sebastian/comparator": "~1.0", + "sebastian/diff": "~1.1", +- "sebastian/environment": "~1.0", ++ "sebastian/environment": "~1.1", + "sebastian/exporter": "~1.0", + "sebastian/version": "~1.0", + "ext-dom": "*", +diff --git a/src/TextUI/ResultPrinter.php b/src/TextUI/ResultPrinter.php +index ca16c8a..e701e33 100644 +--- a/src/TextUI/ResultPrinter.php ++++ b/src/TextUI/ResultPrinter.php +@@ -43,6 +43,8 @@ + * @since File available since Release 2.0.0 + */ - foreach ($this->options[0] as $option) { - switch ($option[0]) { -- case '--colors': { -- $this->arguments['colors'] = true; -+ case '--colors': -+ if (function_exists("posix_isatty") && !posix_isatty(STDOUT)) { -+ $this->arguments['colors'] = false; -+ } else { -+ $this->arguments['colors'] = true; - } - break; - -diff --git a/src/Util/Configuration.php b/src/Util/Configuration.php -index 9ddcfb8..0fcc5fb 100644 ---- a/src/Util/Configuration.php -+++ b/src/Util/Configuration.php -@@ -582,6 +582,9 @@ public function getPHPUnitConfiguration() - $result['colors'] = $this->getBoolean( - (string) $root->getAttribute('colors'), false - ); -+ if ($result['colors'] && function_exists("posix_isatty") && !posix_isatty(STDOUT)) { -+ $result['colors'] = false; -+ } ++use SebastianBergmann\Environment\Console; ++ + /** + * Prints the result of a TextUI TestRunner run. + * +@@ -147,7 +149,9 @@ public function __construct($out = null, $verbose = false, $colors = false, $deb } - /** + if (is_bool($colors)) { +- $this->colors = $colors; ++ $console = new Console; ++ ++ $this->colors = $colors && $console->hasColorSupport(); + } else { + throw PHPUnit_Util_InvalidArgumentHelper::factory(3, 'boolean'); + } -- cgit