-
1. Re: logging 2.2.0.CR1 and OnDemandDependencyUninstallTestCase
adrian.brock Mar 4, 2010 6:49 AM (in response to adrian.brock)There's a similar issue for this new test -
3. Re: logging 2.2.0.CR1 and OnDemandDependencyUninstallTestCase
alesj Mar 4, 2010 8:24 AM (in response to adrian.brock)I upgraded to logging 2.2.0.CR1 since I was getting some strange dependency/classpath issues
due to a bizarre circular dependency in logging 2.1.0.GA.
logging-log4j -> jboss-cl -> jboss-dependency -> logging-log4j
This is my "fault". :-)
The reason is the actual implementation of the TCLMCFilter:
Which does deployment matching based on proper API, instead of wild string guessing (previous impl).
But afaik, main code (not tests) should only depend on logging-spi,
where for tests you exclude the "jboss-cl" part -- then it should work with np.
Its a bit of a bizarre test to me (playing about a private logger to see if something gets logged), why not just attach an appender?
I've removed the test for now. If you think it is an important test, I'll let you figure out how to recreate it using public api. ;-)
This is explained in more details in some other thread.
The issue is that we suppress the exception, only posting a warning -- the only way to know if the wanted behavior occured.
Kabir, can you please have a look at this?
ps: what's the issue with the new test, I didn't understand that?
-
4. Re: logging 2.2.0.CR1 and OnDemandDependencyUninstallTestCase
dmlloyd Mar 4, 2010 10:36 AM (in response to adrian.brock)The reason the constructor is gone is to force logger creation to happen viaLogger.getLogger(String)
, which allows the possibility of reusing logger instances. It should be fairly trivial to switch things over though. -
5. Re: logging 2.2.0.CR1 and OnDemandDependencyUninstallTestCase
kabirkhan Mar 4, 2010 10:46 AM (in response to alesj)alesj wrote:
Kabir, can you please have a look at this?
Done