On Sat, 14 Dec 2002, Stas Bekman wrote:
[ .. ]
> Please check that this work for you and I'll commit it
> (verified on linux).  You need to run a full build to check,
> unless you patch the Makefile manually.
> 
> I also see that my copy-n-paste has translated \t into spaces,
> so replace those with tabs (between the + and $)
> 
> I've replaced \"\" with qq{} because shells and quotes, even
> escaped ones are a big trouble. At least your suggestion didn't
> work for me. qq{} works just fine.
> 
> Index: lib/Apache/Build.pm
> ===================================================================
> RCS file: /home/cvs/modperl-2.0/lib/Apache/Build.pm,v
> retrieving revision 1.108
> diff -u -r1.108 Build.pm
> --- lib/Apache/Build.pm 28 Nov 2002 16:31:36 -0000      1.108
> +++ lib/Apache/Build.pm 14 Dec 2002 08:43:32 -0000
> @@ -1057,6 +1057,9 @@
>   lib: $(MODPERL_LIB)
> 
>   install:
> +       $(MODPERL_PERLPATH) -e "exit ! -d qq{$(MODPERL_AP_LIBEXECDIR)}" || \
> +       $(MODPERL_PERLPATH) -MExtUtils::Command  \
> +       -e mkpath $(MODPERL_AP_LIBEXECDIR)
>          $(MODPERL_TEST_F) $(MODPERL_LIB_DSO) && \
>          $(MODPERL_CP) $(MODPERL_LIB_DSO) $(MODPERL_AP_LIBEXECDIR)

This works fine on Win32, both if the modules directory
exists, and in creating it if it doesn't. Thanks!

-- 
best regards,
randy

Reply via email to