Skip to content

Commit

Permalink
Fix for mepo status bug
Browse files Browse the repository at this point in the history
  • Loading branch information
mathomp4 committed Nov 13, 2024
1 parent ddb369c commit 35dfe0e
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
3 changes: 3 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

### Fixed

- Fixed `mepo status` code to allow for quiet failures. There seems to be an odd scenario on non-internet-connected machines where
`mepo status` will fail unless first run on a node with internet access. We are as yet uncertain why this is the case.

### Removed

### Added
Expand Down
3 changes: 2 additions & 1 deletion esma_support/esma_mepo_status.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -20,10 +20,11 @@ function(esma_capture_mepo_status)
WORKING_DIRECTORY ${CMAKE_SOURCE_DIR}
OUTPUT_FILE "${OUTPUT_FILE}"
RESULT_VARIABLE MEPO_STATUS_RESULT
ERROR_QUIET
)

if(NOT MEPO_STATUS_RESULT EQUAL 0)
message(WARNING "mepo state and command were found but failed to run mepo status --hashes. This is odd.")
message(WARNING "mepo state and command were found but failed to run mepo status --hashes. This seems to happen when internet access is not available. Sometimes. We are not sure yet.")
else()
message(STATUS "mepo status output captured in ${OUTPUT_FILE_NAME}")

Expand Down

0 comments on commit 35dfe0e

Please sign in to comment.