Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.1.0
    • Fix Version/s: 1.1.0
    • Component/s: build
    • Labels:
      None

      Description

      Older versions of the download task fail with Gradle 2.6.

      Upgrading to version 2.0.0 fixes the problem.

        Activity

        Hide
        rnowling RJ Nowling added a comment -

        This patch updates to the download task to version 2.0.0.

        Newer versions of the plugin are referenced using the plugin block rather than a dependency. The plugin block has certain requirements with respect to ordering in the Gradle files, so I had to rearrange the buildscript and plugin blocks.

        I tested this on Gradle 2.6. I would encourage additional testing before committing.

        Show
        rnowling RJ Nowling added a comment - This patch updates to the download task to version 2.0.0. Newer versions of the plugin are referenced using the plugin block rather than a dependency. The plugin block has certain requirements with respect to ordering in the Gradle files, so I had to rearrange the buildscript and plugin blocks. I tested this on Gradle 2.6. I would encourage additional testing before committing.
        Hide
        jayunit100 jay vyas added a comment -

        Does it work with our gradlew script? That's the best test IMO

        Show
        jayunit100 jay vyas added a comment - Does it work with our gradlew script? That's the best test IMO
        Hide
        cos Konstantin Boudnik added a comment -

        It doesn't seem like we have to switch to Gradle 2.6 just because of that, but won't be that a good idea nonetheless?

        +1 on the patch if it works with current ./gradlew, as Jay suggested. If so - please commit. Thanks!

        Show
        cos Konstantin Boudnik added a comment - It doesn't seem like we have to switch to Gradle 2.6 just because of that, but won't be that a good idea nonetheless? +1 on the patch if it works with current ./gradlew, as Jay suggested. If so - please commit. Thanks!
        Hide
        rnowling RJ Nowling added a comment -

        Test with gradlew. Committed.

        Show
        rnowling RJ Nowling added a comment - Test with gradlew. Committed.

          People

          • Assignee:
            rnowling RJ Nowling
            Reporter:
            rnowling RJ Nowling
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development