Uploaded image for project: 'Yetus'
  1. Yetus
  2. YETUS-936

test-patch.sh does not do gradle

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Blocker
    • Resolution: Fixed
    • 0.11.1
    • 0.12.0
    • build
    • MacBook, with in $HOME the .gradle folder. The wrapper/thereunder folder thereunder contains: gradle-2.13-bin, gradle-3.2.1-bin, gradle-5.0-bin, gradle-5.6.2-all
    • Hide
      <!-- markdown -->
      Gradle support has been modified to support modern gradle layouts. As such, this is an incompatible change for gradle. Bootstrapping gradle is no longer supported; gradlew is expected to be there. Patches that wish to upgrade gradle are now expected to just be a patch against the gradle components in the repo.

      * checkstyle should now work
      * $HOME/.gradle is no longer imported into docker containers to prevent cache corruption
      * a new <plugin>_postcleanup has been added. In the case of gradle, this now shuts down the gradle daemon
      * two new core routines have been added (escape_html/unescape_html) to cover some basic HTML entity resolution. These are not exhaustive but cover the most common ones seen.
      * GRADLE_ARGS is now GRADLEW_ARGS to match the functionality change.
      Show
      <!-- markdown --> Gradle support has been modified to support modern gradle layouts. As such, this is an incompatible change for gradle. Bootstrapping gradle is no longer supported; gradlew is expected to be there. Patches that wish to upgrade gradle are now expected to just be a patch against the gradle components in the repo. * checkstyle should now work * $HOME/.gradle is no longer imported into docker containers to prevent cache corruption * a new <plugin>_postcleanup has been added. In the case of gradle, this now shuts down the gradle daemon * two new core routines have been added (escape_html/unescape_html) to cover some basic HTML entity resolution. These are not exhaustive but cover the most common ones seen. * GRADLE_ARGS is now GRADLEW_ARGS to match the functionality change.
    • Important

    Description

      While trying to do a 'test-patch' on a PR in the ofbiz-framework repo on Github with following command

      ~/dev/asf/yetus/precommit/src/main/shell/test-patch.sh --basedir=/Users/nl04748/dev/asf/ofbiz/ofbiz-framework --build-tool=gradle --dirty-workspace --docker --patch-dir=/Users/nl04748/dev/test/yetus --plugins=all --project=ofbiz https://github.com/apache/ofbiz-framework/pull/12
      

      I get following error returned:

      ERROR: gradle is not available. 
      

      See also linked mail thread 

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            aw Allen Wittenauer
            pierresmits Pierre Smits
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0h
                0h
                Logged:
                Time Spent - 2.5h
                2.5h

                Slack

                  Issue deployment