Commons Logging
  1. Commons Logging
  2. LOGGING-113

pom.xml in maven repository does not list dependencies as optional

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.1.0
    • Fix Version/s: 1.1.1
    • Labels:
      None
    • Environment:

      maven 2

      Description

      Dear Commons Logging Developers,

      I don't know if this is the right place, but I have a problem with the maven 2 artifact for commons-logging, available at:

      ftp://ibiblio.org/pub/packages/maven2/commons-logging/commons-logging/1.1/
      ftp://ibiblio.org/pub/packages/maven2/commons-logging/commons-logging/1.1/commons-logging-1.1.pom

      The pom lists avalon-framework, servlet-api, junit, log4j, logkit as REQUIRED dependencies, while instead they should be optional dependencies.

      How to fix:
      add <optional>true</optional> to all dependencies that are optional, e.g.
      <dependency>
      <groupId>log4j</groupId>
      <artifactId>log4j</artifactId>
      <version>1.2.12</version>
      <optional>true</optional>
      </dependency>

      Thanks

        Issue Links

          Activity

          Max Berger created issue -
          Joerg Schaible made changes -
          Field Original Value New Value
          Link This issue duplicates LOGGING-105 [ LOGGING-105 ]
          Hide
          Henri Yandell added a comment -

          This has been fixed in HEAD (the version that has been published is not allowed to be edited).

          Just a question of a 1.1.1 release being made - c-logging releases seem pretty painful to do and no one is volunteering currently.

          Show
          Henri Yandell added a comment - This has been fixed in HEAD (the version that has been published is not allowed to be edited). Just a question of a 1.1.1 release being made - c-logging releases seem pretty painful to do and no one is volunteering currently.
          Henri Yandell made changes -
          Resolution Fixed [ 1 ]
          Status Open [ 1 ] Closed [ 6 ]
          Fix Version/s 1.1.1 [ 12312160 ]

            People

            • Assignee:
              Unassigned
              Reporter:
              Max Berger
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development