<html>#2831: ExternalLibraries/MPI creates MPI_INC_DIRS that have double quotes in their value
<table style='border-spacing: 1ex 0pt; '>
<tr><td style='text-align:right'> Reporter:</td><td>Roland Haas</td></tr>
<tr><td style='text-align:right'> Status:</td><td>new</td></tr>
<tr><td style='text-align:right'>Milestone:</td><td></td></tr>
<tr><td style='text-align:right'> Version:</td><td></td></tr>
<tr><td style='text-align:right'> Type:</td><td>bug</td></tr>
<tr><td style='text-align:right'> Priority:</td><td>minor</td></tr>
<tr><td style='text-align:right'>Component:</td><td>EinsteinToolkit thorn</td></tr>
</table>
<p>ExternalLibraries/MPI’s <code>detect.pl</code> script creates entries in <code>make.MPI.defn</code> like this (on stampede3):</p>
<div class="codehilite"><pre><span></span><code><span>MPI_DIR = /opt/intel/oneapi/mpi/2021.11</span>
<span>MPI_INC_DIRS = "/opt/intel/oneapi/mpi/2021.11/include"</span>
<span>MPI_LIB_DIRS = "/opt/intel/oneapi/mpi/2021.11/lib"</span>
<span>MPI_LIBS = mpicxx mpifort mpi rt pthread dl</span>
</code></pre></div>
<p>which is incorrect since make makes the double quotes part of the variables value. This is usually safe when used in the shell since <code>-L"Foo"</code> is the same as <code>-LFoo</code> but fails with Silo which stores the options in a string (and does not escape embedded quotes).</p>
<p>Since make does not handle spaces well anyway <code>detect.pl</code> should not output any quotes since the paths will never contain one.</p>
<p>--<br/>
Ticket URL: <a href='https://bitbucket.org/einsteintoolkit/tickets/issues/2831/externallibraries-mpi-creates-mpi_inc_dirs'>https://bitbucket.org/einsteintoolkit/tickets/issues/2831/externallibraries-mpi-creates-mpi_inc_dirs</a></p>
</html>