Skip to content

Set default JVM memory settings for the build#59

Merged
asfgit merged 1 commit intoapache:masterfrom
neykov:maven-opts
Dec 2, 2016
Merged

Set default JVM memory settings for the build#59
asfgit merged 1 commit intoapache:masterfrom
neykov:maven-opts

Conversation

@neykov
Copy link
Copy Markdown
Member

@neykov neykov commented Nov 15, 2016

  • Will be picked up when building a sub-module
  • Doesn't work with --file option (MNG-5889)

@sjcorbett
Copy link
Copy Markdown
Contributor

sjcorbett commented Nov 15, 2016

I think this has triggered the rat plugin:

[ERROR] Failed to execute goal org.apache.rat:apache-rat-plugin:0.11:check (default) on project brooklyn-dist-root: Too many files with unapproved license: 1 See RAT report in: /home/jenkins/jenkins-slave/workspace/brooklyn-dist-pull-requests/target/rat.txt -> [Help 1]

@neykov
Copy link
Copy Markdown
Member Author

neykov commented Nov 15, 2016

Rat check fixed in apache/brooklyn-server#436.

@geomacy
Copy link
Copy Markdown
Contributor

geomacy commented Dec 2, 2016

LGTM, merging.

@asfgit asfgit merged commit 107faf7 into apache:master Dec 2, 2016
asfgit pushed a commit that referenced this pull request Dec 2, 2016
@geomacy
This closes #59
5482c58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@neykov @sjcorbett @geomacy @asfgit