Maven 1.x Eclipse Plugin
  1. Maven 1.x Eclipse Plugin
  2. MPECLIPSE-25

Add the capability of specifying more builders

    Details

    • Type: Improvement Improvement
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.7
    • Labels:
      None
    • Number of attachments :
      3

      Description

      If the project uses any builder besidesd the java buidler the .project file cannot be generated. a property holding different builders and maybe natures would help solving this issue

      1. patch.txt
        2 kB
        Miguel Griffa
      2. patch.txt
        1 kB
        Miguel Griffa
      3. test-patch.txt
        2 kB
        Miguel Griffa

        Activity

        Hide
        Miguel Griffa added a comment -

        This adds a property to allow more build commands on the project description file, the documentation is also updated

        Show
        Miguel Griffa added a comment - This adds a property to allow more build commands on the project description file, the documentation is also updated
        Hide
        Miguel Griffa added a comment -

        This patch solves the bug of previous bui (which did not accept lists) and also adds support for specifying multiple natures

        Show
        Miguel Griffa added a comment - This patch solves the bug of previous bui (which did not accept lists) and also adds support for specifying multiple natures
        Hide
        dion gillard added a comment -

        This patch looks ok to me. It would be nice if we had a test case for it, but I can apply and test if everyone is ok with it

        Show
        dion gillard added a comment - This patch looks ok to me. It would be nice if we had a test case for it, but I can apply and test if everyone is ok with it
        Hide
        Miguel Griffa added a comment -

        Adds tests for the functionallity of the previous patch

        Show
        Miguel Griffa added a comment - Adds tests for the functionallity of the previous patch
        Hide
        dion gillard added a comment -

        Thanks for the tests. I can add some checks on the xml similar to the tests in the ear plugin.

        Show
        dion gillard added a comment - Thanks for the tests. I can add some checks on the xml similar to the tests in the ear plugin.
        Hide
        dion gillard added a comment -

        Patch applied with changes

        Show
        dion gillard added a comment - Patch applied with changes

          People

          • Assignee:
            Unassigned
            Reporter:
            Miguel Griffa
          • Votes:
            1 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: