<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div>On Jun 24, 2010, at 1:52 PM, Alexander Neundorf wrote:</div><div><br class="Apple-interchange-newline"><blockquote type="cite"><span class="Apple-style-span" style="border-collapse: separate; font-family: Helvetica; font-size: medium; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; "><blockquote type="cite"><blockquote type="cite">And the FindMPI module claims that it can't find the '-inux-g++'<br></blockquote></blockquote><blockquote type="cite"><blockquote type="cite">library, or something similar. I was able to fix this by modifying<br></blockquote></blockquote><blockquote type="cite"><blockquote type="cite">FindMPI's search to look for -l only when it's followed by a space.<br></blockquote></blockquote><blockquote type="cite"><blockquote type="cite">This is a little worrisome because I think that is a standard gnu<br></blockquote></blockquote><blockquote type="cite"><blockquote type="cite">naming convention, so other platforms with compilers named blah-linux<br></blockquote></blockquote><blockquote type="cite"><blockquote type="cite">are going to have the same bug.<br></blockquote></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">That is a known bug. I've filed it in the bug tracker<br></blockquote><blockquote type="cite"><a href="http://*www.*vtk.org/Bug/view.php?id=9093.">http://*www.*vtk.org/Bug/view.php?id=9093.</a><span class="Apple-converted-space"> </span>I've also uploaded a patch<br></blockquote><blockquote type="cite">(take the second one), that fixes that problem.<br></blockquote><br>In the bug report you say "This patch is NOT heavily tested!"<br><br>Is this still true ?<br><br>Alex</span></blockquote></div><div><br></div>This patch does fix that particular issue with FindMPI on BlueGene/P. I'd suggest adding (^| ) to the regexes for the various other flag checks in FindMPI, too. As others have mentioned, I could imagine things on a link line that contain, say, "foo-Linux" or "foo-Darwin", which would break the -L and -D searches.<div><br></div><div>-Todd</div><div><br></div></body></html>