Re: mod_perl.c:61: `my_perl' undeclared under Cygwin

[Stas Bekman <st...@stason.org>]

> I have one request. I don't know who I have to ask for this. It's about the 'dllexport' issue with cygwin. I'd like to supply a patch, but for this patch to work I need the compiler flag '-DCYGWIN' to be set for all  .c files in the WrapXS dir when compiling (or src/modules/perl when building with MP_STATIC_EXTS=1)
> 
> When I try to build as a dso, -DCYGWIN is set, but when I build static MP2 with MP_STATIC_EXTS=1 - the flag isn't there (I haven't tested static mp2 without MP_STATIC_EXTS=1).
> Can somebody "fix" this, please? I need the -DCYGWIN to be set everywhere not only when building dso. Sorry for not doing this myself, but someone more familar with mp2 can do it much faster and better.

Nick, please try this patch:

Index: lib/Apache/Build.pm
===================================================================
--- lib/Apache/Build.pm (revision 158003)
+++ lib/Apache/Build.pm (working copy)
@@ -29,6 +29,7 @@

  use constant AIX     => $^O eq 'aix';
  use constant DARWIN  => $^O eq 'darwin';
+use constant CYGWIN  => $^O eq 'cygwin';
  use constant IRIX    => $^O eq 'irix';
  use constant HPUX    => $^O eq 'hpux';
  use constant OPENBSD => $^O eq 'openbsd';
@@ -549,6 +550,10 @@
         }
      }

+    if (CYGWIN) {
+        $cflags .= " -DCYGWIN";
+    }
+
      $cflags;
  }



-- 
__________________________________________________________________
Stas Bekman            JAm_pH ------> Just Another mod_perl Hacker
http://stason.org/     mod_perl Guide ---> http://perl.apache.org
mailto:stas@stason.org http://use.perl.org http://apacheweek.com
http://modperlbook.org http://apache.org   http://ticketmaster.com