Infrastructure
  1. Infrastructure
  2. INFRA-6633

consider update to Clojure 1.5.1 as minimum version

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Invalid
    • Fix Version/s: Initial Clearing
    • Component/s: None
    • Labels:
      None

      Description

      Currently `core.incubator` is required as a Clojure dependency. The `-?>` function is available in Clojure 1.5.1 as `some->`.

      * https://github.com/clojure/core.incubator/blob/master/src/main/clojure/clojure/core/incubator.clj#L29
      * https://github.com/clojure/clojure/blob/master/changes.md#24-new-threading-macros

      ```
      % ack "\-\?>"
      apis/ec2/src/main/clojure/org/jclouds/ec2/ebs2.clj
      22: (:use org.jclouds.compute2 [clojure.core.incubator :only (-?>)])
      229: size (-?> (get-string options :size) as-int)

      compute/src/main/clojure/org/jclouds/compute2.clj
      62: (org.jclouds predicate) [clojure.core.incubator :only (-?>)])
      308: (-?> node .getLocation .getId))
      ```

      I can put in a pull request for the updates if requiring the newer version of Clojure is acceptable.

        Activity

        Trevor Wennblom created issue -
        Hide
        David Nalley added a comment -
        Refiled as JCLOUDS-228
        Show
        David Nalley added a comment - Refiled as JCLOUDS-228
        David Nalley made changes -
        Field Original Value New Value
        Status Waiting for Infra [ 10011 ] Closed [ 6 ]
        Resolution Invalid [ 6 ]
        Gavin made changes -
        Fix Version/s Initial Clearing [ 12325964 ]

          People

          • Assignee:
            Unassigned
            Reporter:
            Trevor Wennblom
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development