From 531ff352c90dbc03033b4717780cffdaf975deee Mon Sep 17 00:00:00 2001 From: Anthony Islas Date: Thu, 1 May 2025 17:46:41 -0700 Subject: [PATCH] Correctly read user input for MPI configuration prompt Note that while the typo of the initial variable is fixed, this has no practical effect as the scope of the assigned variable within the if statements extends beyond the control of these blocks. --- arch/configure_reader.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/configure_reader.py b/arch/configure_reader.py index 3a41c86e..545f34ae 100755 --- a/arch/configure_reader.py +++ b/arch/configure_reader.py @@ -583,12 +583,12 @@ def projectSpecificOptions( options, stanzaCfg ) : ############################################################################## # Decompose the weird way to write the logic for DM/SM - USE_MPI = False + useMPI = False if ( stanzaCfg.serialOpt_ or stanzaCfg.smparOpt_ ) and ( stanzaCfg.dmparOpt_ or stanzaCfg.dmsmOpt_ ) : # togglable # we can safely check this since the user would not have been able to select this stanza if it couldn't be disabled if stanzaCfg.dmCompilersAvailable() : - useMPI = not( input( "[DM] Use MPI? Default [N] [y/N] : " ).lower() in yesValues ) + useMPI = input( "[DM] Use MPI? Default [N] [y/N] : " ).lower() in yesValues else : useMPI = False else: