At 12:01 AM -0500 3/13/06, John E. Malmberg wrote: >Craig A. Berry wrote: >>The attached provides the special handling needed for C compiler defines >>in ExtUtils::CBuilder. This is against blead but I'm hoping the right >>number after -p supplied to patch will apply cleanly to the CPAN version >>(which is separately maintained, right?). > >I just applied it against [EMAIL PROTECTED] and got the following result: > >EAGLE> mcr [-]ndbgperl "-I[-.LIB]" [-.lib.extutils.CBuilder.t]01-basic.t >1..11 ># Running under perl version 5.009004 for VMS ># Current time local: Sun Mar 12 23:41:46 2006 ># Current time GMT: Mon Mar 13 04:41:46 2006 ># Using Test.pm version 1.25 >Global symbol "$args" requires explicit package name at >../../ExtUtils/CBuilder/Platform/VMS.pm line 24. >Compilation failed in require at (eval 3) line 2.
I can't reproduce this problem, even after backing out my change and reapplying the patch. Please try reapplying the patch in verbose mode and/or visually inspecting the result and/or the patch itself. $ delete/log [.lib.extutils.cbuilder.platform]vms.pm; %DELETE-I-FILDEL, D0:[CRAIG.perl.lib.ExtUtils.CBuilder.Platform]VMS.pm;2 deleted (33 blocks) $ gpatch --verbose -p0 -i [-]cbuilder_defines.patch Hmm... Looks like a unified diff to me... The text leading up to this was: -------------------------- |--- lib/ExtUtils/CBuilder/Platform/VMS.pm;-0 Wed Sep 28 19:54:07 2005 |+++ lib/ExtUtils/CBuilder/Platform/VMS.pm Sat Mar 11 17:53:09 2006 -------------------------- Patching file lib/ExtUtils/CBuilder/Platform/VMS.pm using Plan A... Hunk #1 succeeded at 9. done $ perl -c [.lib.ExtUtils.CBuilder.Platform]VMS.pm [.lib.extutils.cbuilder.platform]vms.pm syntax OK -- ________________________________________ Craig A. Berry mailto:[EMAIL PROTECTED] "... getting out of a sonnet is much more difficult than getting in." Brad Leithauser