[ 
http://jira.codehaus.org/browse/MARTIFACT-10?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=138539#action_138539
 ] 

Benjamin Bentmann commented on MARTIFACT-10:
--------------------------------------------

bq. btw, feel free to apply yourself for others
Sorry, but there are some parts of Maven that I really don't like to touch. 
Please don't take it as an offense, but the almost complete lack of proper API 
docs for maven-artifact (and other vital core components) makes me perceive 
such code as voodoo and I am on the other side not skilled enough to call 
myself a voodoo priest.

> Fix error handling for system-scoped artifacts
> ----------------------------------------------
>
>                 Key: MARTIFACT-10
>                 URL: http://jira.codehaus.org/browse/MARTIFACT-10
>             Project: Maven Artifact
>          Issue Type: Improvement
>            Reporter: Benjamin Bentmann
>            Assignee: Brett Porter
>            Priority: Trivial
>             Fix For: 3.0-alpha-1
>
>         Attachments: file-error-handling.patch
>
>
> The current order of checking for error cases checks {{!File.isFile()}} 
> before {{!File.exists()}}. However, {{!File.isFile()}} might also fail if the 
> file does not exist at all, hence capturing the same case that is intended to 
> be captured by the later check with {{!File.exists()}}. The order needs to be 
> reversed to get the desired exception messages.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to