groovy

Groovy Ant task is too chatty

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 1.0-JSR-5
  • Fix Version/s: 1.0-JSR-6
  • Component/s: Ant integration
  • Labels:
    None
  • Number of attachments :
    0

Description

The Groovy Ant task logs "statements executed successfully" with the default log level. This log level should be changed to "Verbose".

The relevant file is in src/main/org/codehaus/groovy/ant/Groovy.java.

Activity

Hide
Russel Winder added a comment -

I am not sure what you are asking for here. Could you perhaps give an example of what you do get and what you would like to get?

Show
Russel Winder added a comment - I am not sure what you are asking for here. Could you perhaps give an example of what you do get and what you would like to get?
Hide
Jon Skeet added a comment -

Sure. At the moment, when I run a build I get "statements executed successfully" in the log. I only want to see that in verbose mode, as most of the time I don't care unless there's an error.

All I'm asking for is the call which does the log of "statements executed successfully" to have a log level of verbose in the same way that the one at the start of the script does.

Show
Jon Skeet added a comment - Sure. At the moment, when I run a build I get "statements executed successfully" in the log. I only want to see that in verbose mode, as most of the time I don't care unless there's an error. All I'm asking for is the call which does the log of "statements executed successfully" to have a log level of verbose in the same way that the one at the start of the script does.
Hide
Russel Winder added a comment -

galleon made the change of

log("statements executed successfully");

to

log("statements executed successfully", Project.MSG_VERBOSE);

in CVS HEAD a few days ago so this is now fixed. I shall mark the issue as fixed, when you get to try it and agree its fixed we can close it.

Show
Russel Winder added a comment - galleon made the change of log("statements executed successfully"); to log("statements executed successfully", Project.MSG_VERBOSE); in CVS HEAD a few days ago so this is now fixed. I shall mark the issue as fixed, when you get to try it and agree its fixed we can close it.
Hide
Russel Winder added a comment -

galleon committed fix but didn't update the issue.

Show
Russel Winder added a comment - galleon committed fix but didn't update the issue.
Hide
Jon Skeet added a comment -

That's great - thanks very much.

Show
Jon Skeet added a comment - That's great - thanks very much.
Hide
Russel Winder added a comment -

Assuming that being fixed and resolved and as no-one has said anything, we can close this one.

Show
Russel Winder added a comment - Assuming that being fixed and resolved and as no-one has said anything, we can close this one.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: