juaml/junifer
Watch
3
0
Fork
You've already forked junifer
 0
Code Issues 14 Pull requests 2 Packages Activity Actions

Fix check for FSL #454

Merged
fraimondo merged 2 commits from fix/fsl_check into main 2025-09-16 17:28:53 +00:00
Conversation 3 Commits 2 Files changed 2 +4 -1
2 changed files with 4 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

1
docs/changes/newsfragments/454.bugfix Normal file
View file

@ -0,0 +1 @@
Fix FSL presence checking status code by `Fede Raimondo`_

4
junifer/pipeline/utils.py
View file

@ -155,7 +155,9 @@ def _check_fsl(commands: Optional[list[str]] = None) -> bool:
shell=True, # is unsafe but kept for resolution via PATH shell=True, # is unsafe but kept for resolution via PATH
check=False, check=False,
) )
fsl_found = completed_process.returncode == 1 fsl_found = (
completed_process.returncode == 1 or completed_process.returncode == 0
)
# Check for specific commands # Check for specific commands
if fsl_found and commands is not None: if fsl_found and commands is not None: