Berin Loritsch wrote:
BTW, The docs look nice, and I like the links to the mailing lists.
Two comments though (couldn't resist):
* We should probably have the mailing lists page centralized if we can.
This is auto generated content. We can centralize this at the build level later later.
Not a show stopper. I agree.
* We might want to enable the Test Report and the metrics report.
That way we can make that info a bit more accessible.
Yes - but lets do this later. Doing this with a multi-source project does not work will with maven in its current form. Lots of plugins dont respect the source path attribute so you are forced to do some shuffling of results and I would prefer to do that sort of thing - if - maven final does not take care of it properly.
Ok. Again, I agree. Not trying to force a ton of stuff here.
My comment is that we should have the AbstractLogEnabled class as part of the
interface/API portion. Several projects I know of almost depend on that class,
but they don't necessarily depend on anything else in the IMPL portion directly.
I would prefer *not* to have the AbstractLoggable interface in the API part for
the simple reason that it forces the API to have an explicit dependency on
LogKit--and it is deprecated so its use should be discouraged.
I agree. If nobody objects I'll cut a new release this weekend under 4.1.5 (i.e. drop the RC3 tag) containing AbstractLogEnabled inside the API.
Cool! Thanks.
I guess the last question would be do we want to start a "deprecated/compat"
JAR distributable to house all the stuff that is marked as deprecated? That
way new projects can simply focus on the current stuff and old projects will
have to either add a dependency (not hard) or upgrade (higher level of effort).
What do you think?
Can we worry about it later?
Yep. You did ask for comments, though ;P
Trust me, I am not as detailed as the IV&V (Independant Verification and Validation) guy I get to work with on the job. He would go so far as to mention that the exact text of a property that must be replaced at install time does not match the installation instructions. A non-issue, but mentioned nonetheless ;P
--
"They that give up essential liberty to obtain a little temporary safety
deserve neither liberty nor safety."
- Benjamin Franklin
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
