Op een grimmige herfstdag (Sunday 14 November 2004 22:02),schreef  Craig A. 
Berry:
> Abe Timmerman wrote:
> > Op een grimmige herfstdag (Sunday 14 November 2004 01:28),schreef  Craig
> > A. Berry:
> >> I don't think Test::Smoke knows how to add
> >>qualifiers to MMS or MMK, such as with the /MACRO="__DEBUG__"=1 case
> >>to do a vmsdebug build.
> >
> > I'll make a new snapshot this weekend. My version already knows about
> > -"Dusevmsdebug" and /macro=("__DEBUG__=1"), but I must admit that if
> > there are other qualifiers needed by certain configure options they will
> > not be set. (I'll need to do more digging in configure.com)
>
> That may actually the only macro we need to worry about, though it's
> worth checking if there are others.

Will do (soonish)...

>                                      I thought it had not been
> implemented because it refuses to run the tests for -Dusevmsdebug
> thinking it can't find a successfully built perl. I think the problem is
> just that it's looking for perl.exe when what it needs in this case is
> ndbgperl.exe. The following ought to take care of it, though I haven't
> tested this yet.

That's a good catch! I've applied it and deployed it on both VMS machines (now 
we have to wait for another change :-)
If this solves it, I will do a new snapshot (1.19_53).

> --- lib/Test/Smoke/Smoker.pm;-0 Sat Oct 30 07:42:49 2004
> +++ lib/Test/Smoke/Smoker.pm    Sun Nov 14 14:57:12 2004
> @@ -381,6 +381,7 @@ sub make_ {
>       my $exe_ext  = $Config{_exe} || $Config{exe_ext};
>       my $miniperl = "miniperl$exe_ext";
>       my $perl     = "perl$exe_ext";
> +    $perl = "ndbg$perl" if $self->{is_vms} &&
> $config->has_arg('-Dusevmsdebug');
>       -x $miniperl or return BUILD_NOTHING;
>       return -x $perl
>           ? $self->{_run_exit} ? BUILD_MINIPERL : BUILD_PERL

Many thanks for your work +
Good luck,

Abe
-- 
NI-S> All in all not one of my better bits of code.
EINSUFFICIENTPARANOIA?

Remember, just because you don't believe that everyone is out to get

you, it doesn't actually mean that they are not.
                                        -- Nicholas Clark on p5p @ 20020522

Reply via email to