Skip to content

Commit

Permalink
fix: use 1 as return code for successful FSL commands in pipeline.utils
Browse files Browse the repository at this point in the history
  • Loading branch information
synchon committed Oct 31, 2023
1 parent 4f2772f commit fef93f9
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions junifer/pipeline/utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -135,7 +135,7 @@ def _check_fsl(commands: Optional[List[str]] = None) -> bool:
shell=True, # is unsafe but kept for resolution via PATH
check=False,
)
fsl_found = completed_process.returncode == 0
fsl_found = completed_process.returncode == 1

# Check for specific commands
if fsl_found and commands is not None:
Expand All @@ -155,7 +155,7 @@ def _check_fsl(commands: Optional[List[str]] = None) -> bool:
shell=True, # is unsafe but kept for resolution via PATH
check=False,
)
command_found = command_process.returncode == 0
command_found = command_process.returncode == 1
commands_found_results[command] = (
"found" if command_found else "not found"
)
Expand Down

0 comments on commit fef93f9

Please sign in to comment.