Issue Details (XML | Word | Printable)

Key: GROOVY-1597
Type: Improvement Improvement
Status: Closed Closed
Resolution: Fixed
Priority: Minor Minor
Assignee: Paul King
Reporter: Antti Karanta
Votes: 0
Watchers: 1
Operations

If you were logged in you would be able to see more operations.
groovy

Error message when trying to execute a directory should be improved

Created: 15/Dec/06 08:38 AM   Updated: 08/Feb/08 01:26 AM
Component/s: None
Affects Version/s: 1.0-RC-1
Fix Version/s: 1.5.2

Time Tracking:
Not Specified

Environment: groovy svn head


 Description  « Hide
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.



 All   Comments   Work Log   Change History      Sort Order: Ascending order - Click to sort in descending order
Paul King added a comment - 06/Jan/08 03:08 AM
Now says:
Caught: java.lang.IllegalArgumentException: c:\temp (c:\temp) is a directory not a Groovy source file.

Paul King added a comment - 08/Feb/08 01:26 AM
close off release 1.5.4