fix: don't throw error if ProcessStartFailedException

Gemorroj · Jul 27, 2024 · b777f6a · b777f6a
1 parent 7333dbb
commit b777f6a
Show file tree

Hide file tree

Showing 10 changed files with 50 additions and 27 deletions.
diff --git a/src/Parsers/Apcaccess.php b/src/Parsers/Apcaccess.php
@@ -3,16 +3,18 @@
 namespace Ginfo\Parsers;
 
 use Ginfo\Common;
+use Symfony\Component\Process\Exception\ProcessFailedException;
+use Symfony\Component\Process\Exception\ProcessStartFailedException;
 use Symfony\Component\Process\Process;
 
 class Apcaccess implements ParserInterface
 {
     public static function work(): ?array
     {
         $process = new Process(['apcaccess', 'status'], null, ['LANG' => 'C']);
-        $process->run();
-
-        if (!$process->isSuccessful()) {
+        try {
+            $process->mustRun();
+        } catch (ProcessFailedException|ProcessStartFailedException $e) {
             return null;
         }
 

diff --git a/src/Parsers/Free.php b/src/Parsers/Free.php
@@ -2,6 +2,8 @@
 
 namespace Ginfo\Parsers;
 
+use Symfony\Component\Process\Exception\ProcessFailedException;
+use Symfony\Component\Process\Exception\ProcessStartFailedException;
 use Symfony\Component\Process\Process;
 
 class Free implements ParserInterface
@@ -17,9 +19,9 @@ private function __clone()
     public static function work(): ?array
     {
         $process = new Process(['free', '-bw'], null, ['LANG' => 'C']);
-        $process->run();
-
-        if (!$process->isSuccessful()) {
+        try {
+            $process->mustRun();
+        } catch (ProcessFailedException|ProcessStartFailedException $e) {
             return null;
         }
 

diff --git a/src/Parsers/Lpstat.php b/src/Parsers/Lpstat.php
@@ -2,6 +2,8 @@
 
 namespace Ginfo\Parsers;
 
+use Symfony\Component\Process\Exception\ProcessFailedException;
+use Symfony\Component\Process\Exception\ProcessStartFailedException;
 use Symfony\Component\Process\Process;
 
 /**
@@ -12,8 +14,9 @@ class Lpstat implements ParserInterface
     public static function work(): ?array
     {
         $process = new Process(['lpstat', '-p'], null, ['LANG' => 'C']);
-        $process->run();
-        if (!$process->isSuccessful()) {
+        try {
+            $process->mustRun();
+        } catch (ProcessFailedException|ProcessStartFailedException $e) {
             return null;
         }
 

diff --git a/src/Parsers/Sensors/Ipmi.php b/src/Parsers/Sensors/Ipmi.php
@@ -3,6 +3,8 @@
 namespace Ginfo\Parsers\Sensors;
 
 use Ginfo\Parsers\ParserInterface;
+use Symfony\Component\Process\Exception\ProcessFailedException;
+use Symfony\Component\Process\Exception\ProcessStartFailedException;
 use Symfony\Component\Process\Process;
 
 /**
@@ -15,9 +17,9 @@ class Ipmi implements ParserInterface
     public static function work(): ?array
     {
         $process = new Process(['ipmitool', 'sdr'], null, ['LANG' => 'C']);
-        $process->run();
-
-        if (!$process->isSuccessful()) {
+        try {
+            $process->mustRun();
+        } catch (ProcessFailedException|ProcessStartFailedException $e) {
             return null;
         }
 

diff --git a/src/Parsers/Sensors/Nvidia.php b/src/Parsers/Sensors/Nvidia.php
@@ -3,6 +3,8 @@
 namespace Ginfo\Parsers\Sensors;
 
 use Ginfo\Parsers\ParserInterface;
+use Symfony\Component\Process\Exception\ProcessFailedException;
+use Symfony\Component\Process\Exception\ProcessStartFailedException;
 use Symfony\Component\Process\Process;
 
 /**
@@ -15,8 +17,9 @@ class Nvidia implements ParserInterface
     public static function work(): ?array
     {
         $process = new Process(['nvidia-smi', '-L'], null, ['LANG' => 'C']);
-        $process->run();
-        if (!$process->isSuccessful()) {
+        try {
+            $process->mustRun();
+        } catch (ProcessFailedException|ProcessStartFailedException $e) {
             return null;
         }
 

diff --git a/src/Parsers/Sensors/Sensors.php b/src/Parsers/Sensors/Sensors.php
@@ -3,6 +3,8 @@
 namespace Ginfo\Parsers\Sensors;
 
 use Ginfo\Parsers\ParserInterface;
+use Symfony\Component\Process\Exception\ProcessFailedException;
+use Symfony\Component\Process\Exception\ProcessStartFailedException;
 use Symfony\Component\Process\Process;
 
 class Sensors implements ParserInterface
@@ -18,8 +20,9 @@ private function __clone()
     public static function work(): ?array
     {
         $process = new Process(['sensors'], null, ['LANG=C']);
-        $process->run();
-        if (!$process->isSuccessful()) {
+        try {
+            $process->mustRun();
+        } catch (ProcessFailedException|ProcessStartFailedException $e) {
             return null;
         }
 

diff --git a/src/Parsers/Sestatus.php b/src/Parsers/Sestatus.php
@@ -3,6 +3,8 @@
 namespace Ginfo\Parsers;
 
 use Ginfo\Common;
+use Symfony\Component\Process\Exception\ProcessFailedException;
+use Symfony\Component\Process\Exception\ProcessStartFailedException;
 use Symfony\Component\Process\Process;
 
 class Sestatus implements ParserInterface
@@ -18,9 +20,9 @@ private function __clone()
     public static function work(): ?array
     {
         $process = new Process(['sestatus'], null, ['LANG' => 'C']);
-        $process->run();
-
-        if (!$process->isSuccessful()) {
+        try {
+            $process->mustRun();
+        } catch (ProcessFailedException|ProcessStartFailedException $e) {
             return null;
         }
 

diff --git a/src/Parsers/Smbstatus.php b/src/Parsers/Smbstatus.php
@@ -2,6 +2,8 @@
 
 namespace Ginfo\Parsers;
 
+use Symfony\Component\Process\Exception\ProcessFailedException;
+use Symfony\Component\Process\Exception\ProcessStartFailedException;
 use Symfony\Component\Process\Process;
 
 /**
@@ -12,9 +14,9 @@ class Smbstatus implements ParserInterface
     public static function work(): ?array
     {
         $process = new Process(['smbstatus'], null, ['LANG' => 'C']);
-        $process->run();
-
-        if (!$process->isSuccessful()) {
+        try {
+            $process->mustRun();
+        } catch (ProcessFailedException|ProcessStartFailedException $e) {
             return null;
         }
 

diff --git a/src/Parsers/Systemd.php b/src/Parsers/Systemd.php
@@ -2,6 +2,8 @@
 
 namespace Ginfo\Parsers;
 
+use Symfony\Component\Process\Exception\ProcessFailedException;
+use Symfony\Component\Process\Exception\ProcessStartFailedException;
 use Symfony\Component\Process\Process;
 
 class Systemd implements ParserInterface
@@ -17,9 +19,9 @@ private function __clone()
     public static function work(): ?array
     {
         $process = new Process(['systemctl', 'list-units', '--type', 'service', '--all'], null, ['LANG' => 'C']);
-        $process->run();
-
-        if (!$process->isSuccessful()) {
+        try {
+            $process->mustRun();
+        } catch (ProcessFailedException|ProcessStartFailedException $e) {
             return null;
         }
 

diff --git a/src/Parsers/Who.php b/src/Parsers/Who.php
@@ -2,6 +2,8 @@
 
 namespace Ginfo\Parsers;
 
+use Symfony\Component\Process\Exception\ProcessFailedException;
+use Symfony\Component\Process\Exception\ProcessStartFailedException;
 use Symfony\Component\Process\Process;
 
 class Who implements ParserInterface
@@ -17,9 +19,9 @@ private function __clone()
     public static function work(): ?array
     {
         $process = new Process(['who', '--count'], null, ['LANG' => 'C']);
-        $process->run();
-
-        if (!$process->isSuccessful()) {
+        try {
+            $process->mustRun();
+        } catch (ProcessFailedException|ProcessStartFailedException $e) {
             return null;
         }