groovy

A Gradle build for Groovy

Details

  • Type: New Feature New Feature
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: None
  • Labels:
    None
  • Patch Submitted:
    Yes
  • Number of attachments :
    1

Description

I have attached a patch that provides a Gradle build to the latest groovy-core trunk sources.

Issue Links

duplicates

New Feature - A new feature of the product, which has yet to be developed. GROOVY-4160 Gradle build

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Roshan Dawrani added a comment -

Hello Hans. A quick question, if you don't mind.

The 12 security tests that were failing until a day or two back - have they been resolved by adding AllPermission to goovy.policy for gradle home/junit codebases?

Show
Roshan Dawrani added a comment - Hello Hans. A quick question, if you don't mind. The 12 security tests that were failing until a day or two back - have they been resolved by adding AllPermission to goovy.policy for gradle home/junit codebases?
Hide
Permalink
Hans Dockter added a comment - - edited

They have been resolved by adding: 

grant codeBase "file:${gradle.junit}" {
    permission java.security.AllPermission;
};

Without adding

grant codeBase "file:${gradle.home}/-" {
    permission java.security.AllPermission;
};

the tests would even run. We have to set the junit jar explicitly, as Gradle does not ship with one.

Show
Hans Dockter added a comment - - edited They have been resolved by adding: grant codeBase "file:${gradle.junit}" { permission java.security.AllPermission; }; Without adding grant codeBase "file:${gradle.home}/-" { permission java.security.AllPermission; }; the tests would even run. We have to set the junit jar explicitly, as Gradle does not ship with one.
Hide
Permalink
Guillaume Laforge added a comment -

Patch applied.
Please, developers, have a go at running the build to see if it's working for you.
Run ./gradlew build or other targets to double check.

Show
Guillaume Laforge added a comment - Patch applied. Please, developers, have a go at running the build to see if it's working for you. Run ./gradlew build or other targets to double check.

People

  • Assignee:
    Guillaume Laforge
    Reporter:
    Hans Dockter
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: