Uploaded image for project: 'Spring Framework'
  1. Spring Framework
  2. SPR-15930

Ensure spring-jcl is used in the generated Eclipse classpath instead of commons-logging

    Details

    • Type: Task
    • Status: Closed
    • Priority: Minor
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: 5.0 RC4
    • Component/s: [Build]
    • Labels:
      None
    • Last commented by a User:
      false

      Description

      Status Quo

      Changes in SPR-11836 pushed all project dependencies to the end of the Eclipse classpath.

      However, with the recent introduction of the spring-jcl module, log4j is no longer used when executing tests within the Eclipse IDE. Consequently, we see thousands of lines of unwanted log statements written to the console using STD_ERR (since JUL logs INFO messages to STD_ERR by default).

      Manual Work-around

      Manually move the spring-jcl classpath entry before any third-party JARs in the generated Eclipse .classpath file.

      Deliverables

      • Modify the ide.gradle script so that spring-jcl comes before any third-party JAR in the generated Eclipse .classpath
        • Implemented differently

        Issue Links

          Activity

          Hide
          sbrannen Sam Brannen added a comment -

          Resolved by excluding commons-logging in the spring-test Gradle build in commit: https://github.com/spring-projects/spring-framework/commit/8acb8aef1f4b87ecfb965084f3f19ce2e188201c

          Show
          sbrannen Sam Brannen added a comment - Resolved by excluding commons-logging in the spring-test Gradle build in commit: https://github.com/spring-projects/spring-framework/commit/8acb8aef1f4b87ecfb965084f3f19ce2e188201c

            People

            • Assignee:
              sbrannen Sam Brannen
              Reporter:
              sbrannen Sam Brannen
              Last updater:
              St├ęphane Nicoll
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:
                Days since last comment:
                10 weeks, 6 days ago