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