We could use java.util.logging, then no dependencies. We definitely need some kind of logging in the components, getting a message on the UI that says:
"Validation Error "Segment": Value is not a valid option.", isn't exactly useful. A simple log message would have told me exactly what the problem is. When I finally broke down and put a logger into the component and found out in two seconds that it was because the value wasn't in my select list after spending all day thinking it was something completely different. For our users sake, we need this. Travis On 12/6/05, Simon Kitching <[EMAIL PROTECTED]> wrote: > Hi, > > Is it really ok for stuff in the "api" and "impl" subdirs to depend on > commons-logging? > > Does the spec say anything about dependency requirements for the JSF > implementation? In particular, I'm concerned that j2ee.jar will > apparently require a JSF implementation to be included in the future; if > MyFaces is that implementation and it uses org.apache.* libs then those > libs must also be bundled in the j2ee.jar file, or be bundled by every > container that provides that j2ee file. And exposing libraries via the > container like that can cause pain, as we all found out when a buggy > version of org.apache.xerces was bundled with java 1.4 :-( > > I see that currently 3 classes in the "api" project use JCL, and lots of > classes in "impl". > > JBoss and Tomcat always provide JCL jars at the container level (which > is a bad idea IMHO, but that's another topic). However not all > containers may. > > Cheers, > > Simon > > Martin Marinschek wrote: > > yes. > > > > You can do whatever you want, as long as it is private ;) > > > > regards, > > > > Martin > > > > On 12/7/05, Travis Reeder <[EMAIL PROTECTED]> wrote: > >> Is it safe to put loggers into Spec components, for instance in > >> UISelectMany, to show why something is invalid? > >> > >> Travis > >> > > > > > > -- > > > > http://www.irian.at > > > > Your JSF powerhouse - > > JSF Consulting, Development and > > Courses in English and German > > > > Professional Support for Apache MyFaces > > > > > >
