Uploaded image for project: 'Commons CLI'
  1. Commons CLI
  2. CLI-134

1.1 is not backwards compatible because it adds methods to the CommandLineParser interface

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.0
    • Fix Version/s: 1.1
    • Component/s: CLI-1.x
    • Labels:
      None

      Description

      General problem - the interface adds methods.

      Solution - remove the interfaces from the methods and people who want to use them will have to use the Parser abstract class instead of the CommandLineParser interface.

      1. CLI-134-2nd.patch
        5 kB
        Henri Yandell
      2. CLI-134.patch
        2 kB
        Henri Yandell

        Activity

        Hide
        bayard Henri Yandell added a comment -

        Patch removing the methods.

        Show
        bayard Henri Yandell added a comment - Patch removing the methods.
        Hide
        bayard Henri Yandell added a comment -

        Applied to SVN - r551811.

        Show
        bayard Henri Yandell added a comment - Applied to SVN - r551811.
        Hide
        bayard Henri Yandell added a comment -

        Damn. Missed and removed a method I shouldn't have Leaving one in that should have stayed.

        Show
        bayard Henri Yandell added a comment - Damn. Missed and removed a method I shouldn't have Leaving one in that should have stayed.
        Hide
        bayard Henri Yandell added a comment -

        Last patch commented out the wrong method. This patch fixes that and fixes a unit test that breaks.

        Show
        bayard Henri Yandell added a comment - Last patch commented out the wrong method. This patch fixes that and fixes a unit test that breaks.
        Hide
        bayard Henri Yandell added a comment -

        Second patch solves it.

        Show
        bayard Henri Yandell added a comment - Second patch solves it.

          People

          • Assignee:
            Unassigned
            Reporter:
            bayard Henri Yandell
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development