Philip Martin <philip.mar...@wandisco.com> writes:

> However this object code is dynamically loaded by Perl at runtime so
> it's not impossible that the Perl compiler flags need to be applied.  It
> seems to me that Subversion is responsible for ensuring that this is
> done and that we should be fixing this in Subversion.

I propose to compile with both Subversion and Perl flags using the
following patch:

Index: subversion/bindings/swig/perl/native/Makefile.PL.in
===================================================================
--- subversion/bindings/swig/perl/native/Makefile.PL.in (revision 1145056)
+++ subversion/bindings/swig/perl/native/Makefile.PL.in (working copy)
@@ -60,7 +60,7 @@
 my %config = (
     ABSTRACT => 'Perl bindings for Subversion',
     DEFINE => $cppflags,
-    CCFLAGS => $cflags,
+    CCFLAGS => join(' ', $cflags, $Config{ccflags}),
     INC  => join(' ',$apr_cflags, $apu_cflags, 
                  " -I$swig_srcdir/perl/libsvn_swig_perl",
                  " -I$svnlib_srcdir/include",

-- 
uberSVN: Apache Subversion Made Easy
http://www.uberSVN.com

Reply via email to