mafft version check still failing - bug in regexp found

[tseemann]

% roary -a

2017/10/31 11:18:12 Looking for 'mafft' - found /home/linuxbrew/.linuxbrew/bin/mafft
Use of uninitialized value in concatenation (.) or string at /home/linuxbrew/.linuxbrew/Cellar/perl/5.26.1/lib/perl5/site_perl/5.26.1/Bio/Roary/External/CheckTools.pm line 129.
2017/10/31 11:18:12 Determined mafft version is

% mafft --help

------------------------------------------------------------------------------
  MAFFT v7.305b (2016/Aug/16)

The bug is the space at the end of REGEXP ... the b in the version ruins this. You can move the space safely.

    'mafft' => {
        GETVER => "mafft --version < /dev/null 2>&1",
        REGEXP => qr/(\d+\.\d+) /,
        NEEDED => 1,
    },

These hashes look very familiar :-)
We need a central Perl repo for this module which knows all the tools?

[andrewjpage]

Fixed thanks