David Golden via RT wrote:
Show quoted text> This simple patch does that if --verbose is given, at least for
> ACTION_*. Is that sufficient? Or too annoying when running tests with
> verbose? If so, do we just add a "--debug" property and document it?
> If we do, we should probably create a log_debug method like log_verbose
> and then we can sprinkle it anywhere else we want to add debugging
> diagnostics.
I think its cutting too fine a hair to separate them. At that point you can
grep and the like.
This is a good start, thanks!
Show quoted text> Index: lib/Module/Build/Base.pm
> ===================================================================
> --- lib/Module/Build/Base.pm (revision 13014)
> +++ lib/Module/Build/Base.pm (working copy)
> @@ -1593,7 +1593,10 @@
> local $self->{action} = $action;
> my $method = $self->can_action( $action );
> die "No action '$action' defined, try running the 'help' action.\n"
> unless $method;
> - return $self->$method();
> + $self->log_verbose("Starting ACTION_$action\n");
> + my $rc = $self->$method();
> + $self->log_verbose("Finished ACTION_$action\n");
> + return $rc;
> }
>
> sub can_action {
>
>
>
--
Stabbing you in the face so you don't have to.
