Anders Hammar commented on Improvement MJAXB-71

I'm struggling here. I think that any plugin adding a non-existing source directory is just plain bad and should be banned. But, on the other hand, I don't see any harm with the check here. If schemagen would have quietly ignored any specified non-existing source dir I think we wouldn't argue about if that was a wrong behavior.

The logging about ignoring a non-existing source dir is currently at info level (not warn as the patch). I changed that to comply with how the m-resources-p logs non-existing resource directory. We could bump that to warn if it would make it more clear to the user (but most users just ignores warnings anyways as long as the build passed, in my experience).

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators.
For more information on JIRA, see: http://www.atlassian.com/software/jira
--------------------------------------------------------------------- To unsubscribe from this list, please visit: http://xircles.codehaus.org/manage_email

Reply via email to