Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow starting daemon with lower priority #7403

Merged
merged 1 commit into from Oct 18, 2018
Merged

Conversation

oehme
Copy link
Contributor

@oehme oehme commented Oct 16, 2018

Fixes #1606

@oehme oehme added this to the 5.0 RC1 milestone Oct 16, 2018
@oehme oehme self-assigned this Oct 16, 2018
public static final String GRADLE_PROPERTY = "org.gradle.daemon.priority";

public PriorityOption() {
super(GRADLE_PROPERTY, CommandLineOptionConfiguration.create("priority", "Specifies the scheduling priority for the Gradle daemon and all processes launched by it. Values are 'normal'(default) or 'low'"));
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't this be .incubating()?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: missing space before (default)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed

@oehme oehme force-pushed the oehme/daemon/priority branch 2 times, most recently from 98d16ea to 662e81d Compare October 16, 2018 15:14
@oehme oehme force-pushed the oehme/daemon/priority branch 9 times, most recently from 21234ee to df25b87 Compare October 18, 2018 10:18
@oehme oehme merged commit deac1c7 into release Oct 18, 2018
@oehme oehme deleted the oehme/daemon/priority branch October 18, 2018 13:57
jjohannes added a commit that referenced this pull request Jan 18, 2021
This was introduced in Gradle 5.0 (#7403) and no more has been done on
it since then.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants