[ 
http://jira.codehaus.org/browse/MEXEC-19?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_87587
 ] 

Ryan Sonnek commented on MEXEC-19:
----------------------------------

i disagree.

The job of this plugin *is* to execute a java class, but why force developers 
to jump through extra hoops.  Here's a simple scenerio that i think is totally 
legit:

* developer runs the exec:java plugin for his class
* developer makes changes
* developer re-runs the exec:java plugin

Ooops!  since they didn't run {{mvn compile}}, they'll be running the *old* 
version of the class.  More troubleshooting is required, and some people might 
initially think it's a "bug" in the exec:java plugin for not picking up their 
changes.

My opinion is that *if* the exec:java plugin adds the target/classes directory 
to the classpath, it should also ensure that those classes are uptodate.  If 
the exec:java plugin changes to only use precompiled dependencies, then it's 
fine as it is today.

and *please* don't introduce yet another flag to make this option configurable. 
 we just need more stuff that "works as expected".

> exec:java plugin should bind to the process-classes phase
> ---------------------------------------------------------
>
>                 Key: MEXEC-19
>                 URL: http://jira.codehaus.org/browse/MEXEC-19
>             Project: Mojo Exec Plugin
>          Issue Type: Improvement
>    Affects Versions: 1.1
>         Environment: maven-exec-plugin-1.1-SNAPSHOT
>            Reporter: Ryan Sonnek
>            Priority: Trivial
>
> the exec:java plugin uses adds target/classes to the classpath, but relies on 
> the user to run a specific maven goal (ex: install, compile, etc) before 
> running the plugin.  It would be better if the exec:java plugin automatically 
> bound to the process-classes phase to always compile the classes in need.

-- 
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

        

---------------------------------------------------------------------
To unsubscribe from this list please visit:

    http://xircles.codehaus.org/manage_email

Reply via email to