ZooKeeper
  1. ZooKeeper
  2. ZOOKEEPER-1249

jline should be an optional maven dependency

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Trivial Trivial
    • Resolution: Duplicate
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: build
    • Labels:
      None

      Description

      When a project adds a maven dependency to zookeeper, they probably don't want the jline dependency. jline should have <optional>true</optional> in zookeeper's maven pom.

        Issue Links

          Activity

          Hide
          Mahadev konar added a comment -

          @David,
          Good point. Mind uploading a patch?

          Show
          Mahadev konar added a comment - @David, Good point. Mind uploading a patch?
          Hide
          David Smiley added a comment -

          I'm a downstream user of zookeeper (via Solr trunk) and don't have the source tree to Zookeeper. The change you need to make to the pom is trivial.

          Show
          David Smiley added a comment - I'm a downstream user of zookeeper (via Solr trunk) and don't have the source tree to Zookeeper. The change you need to make to the pom is trivial.
          Hide
          Patrick Hunt added a comment -

          FYI: the poms are generated, not hand coded. Perhaps some ivy magic is involved?

          (will be great to move to Maven in 3.5)

          Show
          Patrick Hunt added a comment - FYI: the poms are generated, not hand coded. Perhaps some ivy magic is involved? (will be great to move to Maven in 3.5)
          Hide
          Patrick Hunt added a comment -

          Fixed in ZOOKEEPER-1655

          Show
          Patrick Hunt added a comment - Fixed in ZOOKEEPER-1655

            People

            • Assignee:
              Unassigned
              Reporter:
              David Smiley
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development