diff options
| author | Kyle Edwards <kyle.edwards@kitware.com> | 2023-06-02 20:02:23 (GMT) |
|---|---|---|
| committer | Kyle Edwards <kyle.edwards@kitware.com> | 2023-06-08 18:56:27 (GMT) |
| commit | 891b60d691812c45b92d4ad2127e1559f80f8332 (patch) | |
| tree | 835fea3aab501b9a2d1f431093f58e6083a8c56f /Source/cmBinUtilsWindowsPEObjdumpGetRuntimeDependenciesTool.cxx | |
| parent | 5be0cd9f3c07b60b1f60ea678c1d82c38185ca27 (diff) | |
| download | CMake-891b60d691812c45b92d4ad2127e1559f80f8332.zip CMake-891b60d691812c45b92d4ad2127e1559f80f8332.tar.gz CMake-891b60d691812c45b92d4ad2127e1559f80f8332.tar.bz2 | |
cmUVProcessChain: Add Status::SpawnResult field
Diffstat (limited to 'Source/cmBinUtilsWindowsPEObjdumpGetRuntimeDependenciesTool.cxx')
| -rw-r--r-- | Source/cmBinUtilsWindowsPEObjdumpGetRuntimeDependenciesTool.cxx | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/Source/cmBinUtilsWindowsPEObjdumpGetRuntimeDependenciesTool.cxx b/Source/cmBinUtilsWindowsPEObjdumpGetRuntimeDependenciesTool.cxx index f14de55..2139b4b 100644 --- a/Source/cmBinUtilsWindowsPEObjdumpGetRuntimeDependenciesTool.cxx +++ b/Source/cmBinUtilsWindowsPEObjdumpGetRuntimeDependenciesTool.cxx @@ -34,7 +34,7 @@ bool cmBinUtilsWindowsPEObjdumpGetRuntimeDependenciesTool::GetFileInfo( builder.AddCommand(command); auto process = builder.Start(); - if (!process.Valid()) { + if (!process.Valid() || process.GetStatus(0).SpawnResult != 0) { std::ostringstream e; e << "Failed to start objdump process for:\n " << file; this->SetError(e.str()); @@ -57,8 +57,7 @@ bool cmBinUtilsWindowsPEObjdumpGetRuntimeDependenciesTool::GetFileInfo( this->SetError(e.str()); return false; } - auto status = process.GetStatus(); - if (!status[0] || status[0]->ExitStatus != 0) { + if (process.GetStatus(0).ExitStatus != 0) { std::ostringstream e; e << "Failed to run objdump on:\n " << file; this->SetError(e.str()); |