tiny pdumper fix

tiny pdumper fix

[Charles A. Roelli]

I'd like to push this change to the pdumper branch to allow building
on older macOS (less than 10.11, I think), where only MAP_ANON is
defined:

--- a/src/pdumper.c
+++ b/src/pdumper.c
@@ -4101,6 +4101,11 @@ dump_anonymous_allocate_w32 (void *base,
 #endif
 }
 
+/* Old versions of macOS only define MAP_ANON, not MAP_ANONYMOUS.  */
+#ifndef MAP_ANONYMOUS
+#define MAP_ANONYMOUS MAP_ANON
+#endif
+
 static void *
 dump_anonymous_allocate_posix (void *base,
                                size_t size,

Is it well placed here, or does it belong in the header file instead?

Re: tiny pdumper fix

[Stefan Monnier]

> I'd like to push this change to the pdumper branch to allow building
> on older macOS (less than 10.11, I think), where only MAP_ANON is
> defined:

Hmm... looks like none of those people who know about pdumper or macOS
was available.  Sorry.
I just installed you patch as is.

> Is it well placed here, or does it belong in the header file instead?

It probably belongs elsewhere, but I pushed it as-is,


        Stefan

Re: tiny pdumper fix

[Charles A. Roelli]

> From: Stefan Monnier <monnier@iro.umontreal.ca>
> Date: Fri, 28 Sep 2018 16:34:03 -0400
> 
> > I'd like to push this change to the pdumper branch to allow building
> > on older macOS (less than 10.11, I think), where only MAP_ANON is
> > defined:
> 
> Hmm... looks like none of those people who know about pdumper or macOS
> was available.  Sorry.
> I just installed you patch as is.
> 
> > Is it well placed here, or does it belong in the header file instead?
> 
> It probably belongs elsewhere, but I pushed it as-is,

ACK, thanks.  From the autoconf documentation, it looks like
AC_CHECK_DECL may be useful here.