Setting the Jetty HTTP port via system property and Gradle property

Details

  • Type: Improvement Improvement
  • Status: Open Open
  • Resolution: Unresolved
  • Affects Version/s: 0.9
  • Fix Version/s: None

Description

Hi,

it would be great to be able to set the Jetty httpPort property via a system property and/or Gradle property. Right now you have to make a change to your Gradle build script (like adding a configuration task) to make this work. The plugin could predefine a property name (e.g. jettyHttpPort) that replaces the default port value.

Examples:

  • System property: gradle jettyRun -DjettyHttpPort=9090
  • Gradle property (gradle.properties): jettyHttpPort=9090

Issue Links

dependent on

Improvement - An improvement or enhancement to an existing feature or task. GRADLE-1365 Allow override of plugin convention property with project property

  • Open - The issue is open and ready for the assignee to start work on it.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Benjamin Muschko added a comment -

I sent a pull request for this on GitHub.

Show
Benjamin Muschko added a comment - I sent a pull request for this on GitHub.
Hide
Permalink
Peter Niederwieser added a comment - - edited

I understand the need for this feature. But I wonder if instead of manually checking system properties, we could improve the way convention objects work so that "gradle jettyRun -PhttpPort=7070" works out of the box.

Show
Peter Niederwieser added a comment - - edited I understand the need for this feature. But I wonder if instead of manually checking system properties, we could improve the way convention objects work so that "gradle jettyRun -PhttpPort=7070" works out of the box.
Hide
Permalink
Benjamin Muschko added a comment -

I would be OK with enriching convention objects also. Actually, that's better. Do you want me to open another ticket for that?

Show
Benjamin Muschko added a comment - I would be OK with enriching convention objects also. Actually, that's better. Do you want me to open another ticket for that?
Hide
Permalink
Peter Niederwieser added a comment -

Someone already created GRADLE-1365.

Show
Peter Niederwieser added a comment - Someone already created GRADLE-1365.
Hide
Permalink
Benjamin Muschko added a comment -

Great, thanks. If you want you can mark this ticket as a duplicate.

Show
Benjamin Muschko added a comment - Great, thanks. If you want you can mark this ticket as a duplicate.
Hide
Permalink
Peter Niederwieser added a comment -

I've already marked it as dependent.

Show
Peter Niederwieser added a comment - I've already marked it as dependent.

People

  • Assignee:
    Unassigned
    Reporter:
    Benjamin Muschko
Vote (1)
Watch (1)

Dates

  • Created:
    Updated: