fix: makeBinaryExecutable explicit filenames

[edkerk]

Main improvements in this PR:

• Fix:
  • checkInstallation correctly makes binaries executable on Ubuntu (solves bug: make binaries executable on Ubuntu #527)

I hereby confirm that I have:

• Selected develop as a target branch

[github-actions]

This PR has been automatically tested with GH Actions. Here is the output of the tests:

 > Installing from location             /home/m/actions-runner/_work/RAVEN/RAVEN
 > Checking RAVEN release               DEVELOPMENT
 > Checking MATLAB release              2020b
 > Checking system architecture         glnxa64
 > Set RAVEN in MATLAB path             Pass
 > Save MATLAB path                     Pass
   > Make binaries executable           Pass

 > Add Java paths for Excel format      Pass
 > Checking libSBML version             5.19.0
 > Checking model import and export
   > Import Excel format                Pass
   > Export Excel format                Pass
   > Import SBML format                 Pass
   > Export SBML format                 Pass

 > Checking for LP solvers
   > glpk                               Pass
   > gurobi                             Pass
   > scip                               Fail
   > cobra                              Pass
 > Set RAVEN solver                     gurobi

 > Checking BLAST+                      Pass
 > Checking DIAMOND                     Pass
 > Checking HMMER                       Pass

 > Checking function uniqueness         Pass

*** checkInstallation complete ***

Running blastPlusTests
.
Done blastPlusTests

Running cdhitTests
.
Done cdhitTests

Running checkTasksTests
.
Done checkTasksTests

Running diamondTests
.
Done diamondTests

Running fillGapsLargeTests
..
Done fillGapsLargeTests

Running fillGapsSmallTests
...
Done fillGapsSmallTests

Running hmmerTests
.
Done hmmerTests

Running importExportTests
....
Done importExportTests

Running mafftTests
.
Done mafftTests

Running miriamTests
.
Done miriamTests

Running modelAbilitiesTests
........
Done modelAbilitiesTests

Running modelConversionTests
.
Done modelConversionTests

Running modelCurationTests
.......... ........
Done modelCurationTests

Running modelSortingTests
..
Done modelSortingTests

Running solverTests
..
Error occurred in solverTests/testSCIP and it did not run to completion.
    ---------
    Error ID:
    ---------
    ''
    --------------
    Error Details:
    --------------
    Error using solverTests>testSCIP (line 80)
    SCIP MEX binary not installed or not functional, test skipped
..
Done solverTests

Running tinitTests
..........
Done tinitTests

Failure Summary:

     Name                  Failed  Incomplete  Reason(s)
    =====================================================
     solverTests/testSCIP    X         X       Errored.

Note: In the case of multiple test runs, this post will be edited.

[mihai-sysbio]

To me it looks like the meaningful change here is the position of the second ", which before was before the . and now is at the end. I'm also a bit unsure if quotation marks were needed at all - maybe it was a mechanism to deal with any spaces that would be found in the full path of binDir.