Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 359645 - Checks needed on Excalibur Logkit Version: 1.0.1
Summary: Checks needed on Excalibur Logkit Version: 1.0.1
Status: RESOLVED FIXED
Alias: None
Product: Hudson
Classification: Technology
Component: Core (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Winston Prakash CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-10-01 20:16 EDT by Duncan Mills CLA
Modified: 2011-10-07 14:02 EDT (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Duncan Mills CLA 2011-10-01 20:16:17 EDT
Question from CQ http://dev.eclipse.org/ipzilla/show_bug.cgi?id=5566
Can you let us know if you can use this package with the following files
removed?

Logger.java
Errorhandler.java
NormalizedJDBCTarget.java
Comment 1 Winston Prakash CLA 2011-10-07 14:01:57 EDT
I searched through core sources. The classes from Logkit does not seem to be used any where in the core. Seems to be brought in by velocity as a compile time dependency. BTW the version seems to be Logkit 2.0 (not 1.0.1). I see this comment in the core pom.xml

        <!--
        | these are now optional in commons-logging:1.1.1,
        | keeping them around in case any plugins use them
        -->
        <dependency>
            <groupId>log4j</groupId>
            <artifactId>log4j</artifactId>
            <version>${log4j.version}</version>
        </dependency>
        <dependency>
            <groupId>logkit</groupId>
            <artifactId>logkit</artifactId>
            <version>${logkit.version}</version>
        </dependency>
        <dependency>
            <groupId>avalon-framework</groupId>
            <artifactId>avalon-framework</artifactId>
            <version>${avalon-framework.version}</version>
        </dependency>
        <!--
        | ^ end of optional commons-logging dependencies
        -->
Comment 2 Winston Prakash CLA 2011-10-07 14:02:22 EDT
Relevant comments added.