Fix mmap infinite recurse in memory patcher

[hkuno]

This commit fixes issue #6853 by commenting out lines in configure.m4
that define HAVE___MMAP, avoiding infinite recursion.
This commit also opportunistically converts tabs to spaces.

Signed-off-by: Harumi Kuno [email protected]

[hjelmn]

Other than the indentation changes 👍.

[jsquyres]

@hppritcha Cray CI's git seems to have gone off the rails -- can you please fix?

Caused by: hudson.plugins.git.GitException: Command "git init /global/homes/h/hpp/jenkins_cori/workspace/open-mpi.build.platforms/Platform/cray_cle/src" returned status code 127:
stdout: 
stderr: git: error while loading shared libraries: libcrypto.so.0.9.8: cannot open shared object file: No such file or directory

	at org.jenkinsci.plugins.gitclient.CliGitAPIImpl.launchCommandIn(CliGitAPIImpl.java:2042)
	at org.jenkinsci.plugins.gitclient.CliGitAPIImpl.launchCommandIn(CliGitAPIImpl.java:2010)
	at org.jenkinsci.plugins.gitclient.CliGitAPIImpl.launchCommandIn(CliGitAPIImpl.java:2006)
	at org.jenkinsci.plugins.gitclient.CliGitAPIImpl.launchCommand(CliGitAPIImpl.java:1638)
	at org.jenkinsci.plugins.gitclient.CliGitAPIImpl$5.execute(CliGitAPIImpl.java:811)
	... 12 more

[jsquyres]

@hkuno Can you squash down to one commit? Then this PR will be good to go.

[hppritcha]

had to fix up some things at nersc after upgrade to CLE 7
bot:ompi:retest

[hkuno]

@hkuno Can you squash down to one commit? Then this PR will be good to go.

Sure! I have done that. (Please let me know if the commit message is too terse, and I'll add more detail.) Thank you!

[hppritcha]

well there was another thing with our jenkins slave prepper installing an old autotools package. try again.

bot:ompi:retest

[hppritcha]

This PR also needs to be cherry-picked over to v4.0.x as it repairs #6720, which introduced an issue this PR addresses. Also needs to be patched in to v3.1.x and v3.0.x.

[markalle]

I'm just browsing this since I added the #6720 that introduced the problem by turning on the interception of mmap that had been considered unnecessary up till that point. I agree with this fix.