Browse Source

Make sure platform-check returns a non-0 exit code if it fails

Jordi Boggiano 4 years ago
parent
commit
d494df61ff

+ 2 - 1
src/Composer/Autoload/AutoloadGenerator.php

@@ -660,7 +660,8 @@ if (!(PHP_VERSION_ID $lowestOperator $lowestPhpVersionId && PHP_VERSION_ID $high
 }
 $requiredExtensions
 if (\$issues) {
-    die('Composer detected issues in your platform:' . "\\n\\n" . implode("\\n", \$issues));
+    echo 'Composer detected issues in your platform:' . "\\n\\n" . implode("\\n", \$issues);
+    exit(104);
 }
 
 PLATFORM_CHECK;

+ 2 - 1
tests/Composer/Test/Autoload/Fixtures/platform/no_extensions_required.php

@@ -9,5 +9,6 @@ if (!(PHP_VERSION_ID >= 70200 && PHP_VERSION_ID < 80000)) {
 }
 
 if ($issues) {
-    die('Composer detected issues in your platform:' . "\n\n" . implode("\n", $issues));
+    echo 'Composer detected issues in your platform:' . "\n\n" . implode("\n", $issues);
+    exit(104);
 }

+ 2 - 1
tests/Composer/Test/Autoload/Fixtures/platform/no_php_lower_bound.php

@@ -9,5 +9,6 @@ if (!(PHP_VERSION_ID >= 0 && PHP_VERSION_ID < 80000)) {
 }
 
 if ($issues) {
-    die('Composer detected issues in your platform:' . "\n\n" . implode("\n", $issues));
+    echo 'Composer detected issues in your platform:' . "\n\n" . implode("\n", $issues);
+    exit(104);
 }

+ 2 - 1
tests/Composer/Test/Autoload/Fixtures/platform/no_php_required.php

@@ -17,5 +17,6 @@ if ($missingExtensions) {
 }
 
 if ($issues) {
-    die('Composer detected issues in your platform:' . "\n\n" . implode("\n", $issues));
+    echo 'Composer detected issues in your platform:' . "\n\n" . implode("\n", $issues);
+    exit(104);
 }

+ 2 - 1
tests/Composer/Test/Autoload/Fixtures/platform/no_php_upper_bound.php

@@ -9,5 +9,6 @@ if (!(PHP_VERSION_ID >= 70200 && PHP_VERSION_ID < 99999)) {
 }
 
 if ($issues) {
-    die('Composer detected issues in your platform:' . "\n\n" . implode("\n", $issues));
+    echo 'Composer detected issues in your platform:' . "\n\n" . implode("\n", $issues);
+    exit(104);
 }

+ 2 - 1
tests/Composer/Test/Autoload/Fixtures/platform/specific_php_release.php

@@ -9,5 +9,6 @@ if (!(PHP_VERSION_ID >= 70208 && PHP_VERSION_ID < 80000)) {
 }
 
 if ($issues) {
-    die('Composer detected issues in your platform:' . "\n\n" . implode("\n", $issues));
+    echo 'Composer detected issues in your platform:' . "\n\n" . implode("\n", $issues);
+    exit(104);
 }

+ 2 - 1
tests/Composer/Test/Autoload/Fixtures/platform/typical.php

@@ -17,5 +17,6 @@ if ($missingExtensions) {
 }
 
 if ($issues) {
-    die('Composer detected issues in your platform:' . "\n\n" . implode("\n", $issues));
+    echo 'Composer detected issues in your platform:' . "\n\n" . implode("\n", $issues);
+    exit(104);
 }