Maven 2 & 3

deprecate expression, add system-property

Details

  • Complexity:
    Intermediate
  • Number of attachments :
    0

Description

expression is confusing. It is used for both default-value, and to allow configuration of what system property to accept. However, now that default-value takes expressions, it is redundant for that and can be confusing, and because of its legacy in handling that, it is in the wrong order for system property handling (see MNG-2062).

I propose we deprecate expression in favour of default-value, and add system-property="someProperty" as the way to configure what command line option can be used.

Issue Links

duplicates

Wish - General wishlist item MPLUGIN-196 rename expression parameter to property

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

Wish - General wishlist item MPLUGIN-196 rename expression parameter to property

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

Bug - A problem which impairs or prevents the functions of the product. MNG-2062 -D on CLI does not override <configuration>

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

New Feature - A new feature of the product, which has yet to be developed. MNG-2064 automatic plugin configuration from system properties

  • Major - Major loss of function.
  • Open - The issue is open and ready for the assignee to start work on it.
supercedes

Improvement - An improvement or enhancement to an existing feature or task. MNG-1936 pattern: for mojo parameters which have default values in the POM we need standard usage

  • 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
Jason van Zyl added a comment -

I think cliProperty would be more appropriate as system property use is being eradicated in the core as it makes embedding not work.

Show
Jason van Zyl added a comment - I think cliProperty would be more appropriate as system property use is being eradicated in the core as it makes embedding not work.
Hide
Permalink
Brett Porter added a comment -

it doesn't necessarily need to be a system property, but I also wouldn't limit it to CLI property. It might be a good idea to be able to set some global properties in your IDE configuration too, or some other embedded scenario.

Show
Brett Porter added a comment - it doesn't necessarily need to be a system property, but I also wouldn't limit it to CLI property. It might be a good idea to be able to set some global properties in your IDE configuration too, or some other embedded scenario.

People

  • Assignee:
    Unassigned
    Reporter:
    Brett Porter
Vote (2)
Watch (5)

Dates

  • Created:
    Updated:
    Resolved: