Maven Invoker Plugin
  1. Maven Invoker Plugin
  2. MINVOKER-97

Add possibility to inherit settings.xml from calling process

    Details

    • Type: New Feature New Feature
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.4
    • Fix Version/s: 1.6
    • Labels:
      None
    • Number of attachments :
      2

      Description

      Invoker will use default settings xml even if differetn settings xml is specified in calling maven process. It would be very usefull to add posibility to "inherit" settings.xml to invoker:run goal (settingsFile parameter is not enough, since it seems imposible to determine settings file from in calling maven process)

      1. MINVOKER-97_rev2.patch
        18 kB
        Anders Hammar
      2. MINVOKER-97.patch
        8 kB
        Anders Hammar

        Issue Links

          Activity

          Hide
          Anders Hammar added a comment -

          Could you please change the type of this ticket to bug then?

          Show
          Anders Hammar added a comment - Could you please change the type of this ticket to bug then?
          Hide
          Olivier Lamy added a comment -

          @Anders why ?
          I'm usually not too much bureaucratic but IMHO looks more a "new feature".

          Show
          Olivier Lamy added a comment - @Anders why ? I'm usually not too much bureaucratic but IMHO looks more a "new feature".
          Hide
          Anders Hammar added a comment -

          The way I'm thinking is that if it's a new feature we ought to preserve the current behavior. Thus we need a new param that the used needs to configure to enable the settings to be inherit. But if we say that the current behavior is wrong, this is a bug and we should fix it and don't have to keep the current behavior. Thus, no new param.

          Show
          Anders Hammar added a comment - The way I'm thinking is that if it's a new feature we ought to preserve the current behavior. Thus we need a new param that the used needs to configure to enable the settings to be inherit. But if we say that the current behavior is wrong, this is a bug and we should fix it and don't have to keep the current behavior. Thus, no new param.
          Hide
          Anders Hammar added a comment -

          Attached patch (rev2) implementing proposed solution. ITs included.

          Show
          Anders Hammar added a comment - Attached patch (rev2) implementing proposed solution. ITs included.
          Hide
          Olivier Lamy added a comment -

          patch committed.
          Thanks!

          Show
          Olivier Lamy added a comment - patch committed. Thanks!

            People

            • Assignee:
              Olivier Lamy
              Reporter:
              Konstantin Titorenko
            • Votes:
              4 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: