Details

    • Type: New Feature
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.3.1
    • Fix Version/s: 1.4
    • Component/s: Build
    • Labels:
      None
    • Flags:
      Patch

      Description

      For Maven there are several places where one can define the commandline arguments. These should be merged to 1 new commandline, but we need to have control over the order in which these are merged.
      Best solution seems to be to introduce a CommandLineBuilder.

        Issue Links

          Activity

          Hide
          britter Benedikt Ritter added a comment -

          Hello Robert, I'll try to have a look later today. Note, that all ASF committers have write access to the Apache Commons repositories. So if you announce your intend to work on CLI you can simply go ahead and apply the changes yourself.

          Show
          britter Benedikt Ritter added a comment - Hello Robert, I'll try to have a look later today. Note, that all ASF committers have write access to the Apache Commons repositories. So if you announce your intend to work on CLI you can simply go ahead and apply the changes yourself.
          Hide
          pascalschumacher Pascal Schumacher added a comment -

          Hi Robert,

          nice to see somebody doing some work on commons-cli.

          It would be nice if you could remove the formatting changes (looks like you changed the formatting to maven code style) from the patch.

          Show
          pascalschumacher Pascal Schumacher added a comment - Hi Robert, nice to see somebody doing some work on commons-cli. It would be nice if you could remove the formatting changes (looks like you changed the formatting to maven code style) from the patch.
          Hide
          rfscholte Robert Scholte added a comment -

          Yeah, auto-save formatting, will reduce the noice

          Show
          rfscholte Robert Scholte added a comment - Yeah, auto-save formatting, will reduce the noice
          Hide
          rfscholte Robert Scholte added a comment -

          Fixed in 1784363

          Show
          rfscholte Robert Scholte added a comment - Fixed in 1784363
          Hide
          britter Benedikt Ritter added a comment -

          Reopening this issue to set it to "fixed". We only close issues after their fix version has been released.

          Show
          britter Benedikt Ritter added a comment - Reopening this issue to set it to "fixed". We only close issues after their fix version has been released.

            People

            • Assignee:
              Unassigned
              Reporter:
              rfscholte Robert Scholte
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development