groovy

Error message when trying to execute a directory should be improved

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 1.0-RC-1
  • Fix Version/s: 1.5.2
  • Component/s: None
  • Labels:
    None
  • Environment:
    groovy svn head
  • Number of attachments :
    0

Description

When trying to execute a directory, e.g.
groovy c:\temp

the error message is not very clear:

org.codehaus.groovy.control.MultipleCompilationErrorsException: startup failed, c:\temp: c:\temp (Access is denied)

1 error

What it actually should say is:

c:\temp is a directory and can not be executed

Note that if the patch attached to GROOVY-1596 is not applied, the above results in NPE.

Activity

Hide
Paul King added a comment -

Now says:

Caught: java.lang.IllegalArgumentException: c:\temp (c:\temp) is a directory not a Groovy source file.
Show
Paul King added a comment - Now says:
Caught: java.lang.IllegalArgumentException: c:\temp (c:\temp) is a directory not a Groovy source file.
Hide
Paul King added a comment -

close off release 1.5.4

Show
Paul King added a comment - close off release 1.5.4

People

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: