Dan Tran wrote:
Since there is little interest with this vote, I am going the extend
it 2 more days, if there is no objection I will go forward with the
release after that.
Sorry, I just had some a look at it to give you my "go" but it ended in a -1
due to MBUILDHELPER-6. I attached a patch to the issue for your review in
case you had a good reason to do the path construction manually.
Some non-critical side notes:
docck:check complains about a missing <description> element in the POM, I
guess that could be fixed as well. The same text could then be copied to the
index.apt as the introduction to have it look a little nicer.
Also, you could consider to add an expression="${removeAll}" annotation or
similar to the mojo parameter "removeAll" to allow its configuration from
the CLI.
Benjamin
---------------------------------------------------------------------
To unsubscribe from this list, please visit:
http://xircles.codehaus.org/manage_email
