@jbedo I'd love to merge this... but after reading the description and github page I still have no idea what this software does 🤷‍♂️

Maybe if we call in some python experts like @FRidh and @jonringer they can review the package, ignoring anything about the software they don't know.

python2 is already EOL, I don't think we should be using it in any new expressions

however, it looks like they only support python2:
https://github.com/Illumina/manta/blob/75b5c38d4fcd2f6961197b28a41eb61856f2d976/src/python/libexec/denovo_scoring.py#L1
Illumina/manta#180

I'm fine with this usage of python2, it's just for some python integration, and upstream is already aware that it's an issue

would you need python at runtime? if so, we need to wrap the executable

        wrapProgram $out/bin/manta \
        --prefix PATH : "${python2}/bin"

the package needed is nativeBuildInputs = [ makeWrapper ];

All references to python end up as absolute paths after the fixup phase, so a wrapper ends up unnecessary.

@GrahamcOfBorg build manta