Bug 1260605 - Remove hack from bug 967556. r=nalexander
authorMike Hommey <mh+mozilla@glandium.org>
Wed, 30 Mar 2016 11:08:06 +0900
changeset 291160 01b372afc3fc2d0bcd7aceede1c4f9b3c0180d01
parent 291159 0f5f41c619be9ede89c093a7b465ea45e2c6aa21
child 291161 04e7a688a7dff3813b4b391fe0e4a484f117a431
push id19656
push usergwagner@mozilla.com
push dateMon, 04 Apr 2016 13:43:23 +0000
treeherderb2g-inbound@e99061fde28a [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs1260605, 967556
Bug 1260605 - Remove hack from bug 967556. r=nalexander The hack was supposed to avoid clobbers when landing bug 967556, which might have worked somehow back then, but doesn't do anything useful anymore. In fact, it now breaks the display of some results in old-configure.
--- a/build/autoconf/hotfixes.m4
+++ b/build/autoconf/hotfixes.m4
@@ -11,14 +11,13 @@ dnl prints is replaced with:
 dnl   #define posix_memalign innocuous_posix_memalign
 dnl   #include "theinclude"
 dnl   #undef posix_memalign
 dnl This avoids double declaration of that function when the header normally
 dnl declares it, while the test itself is just expecting the function not to be
 dnl declared at all, and declares it differently (which doesn't matter for the
 dnl test itself).
 dnl More recent versions of autoconf are essentially doing this.
-define([ac_cv_func_], [ac_cv_func2_])dnl
 define([AC_CHECK_FUNC], [dnl
 patsubst(_AC_CHECK_FUNC($@), [#include.*$], [#define $1 innocuous_$1
 #undef $1])])dnl