Cactus
  1. Cactus
  2. CACTUS-137

Building maven plugin should be easier

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.7
    • Fix Version/s: 1.7
    • Component/s: Maven Integration
    • Labels:
      None

      Description

      Building the maven integration plugin should be an automated task - right now, the developer must manually copy jars to his local maven repo.

      Ideally, the whole Cactus should be build with Maven, but while that doesn't happen, we need a workaround. We have at least 2 options:

      1.Modify root's build.xml to copy the JARs to the local repo
      2.Modify maven.xml on integration/maven to do both jobs.

      Each approach has its advantages and disadvantages:

      Ant approach
      ------------

      • D: requires a lot of Ant coding (once you get used to Maven, is hard to bo back to the task of writing Ant targets
      • D: it needs to check if user has maven installed, if $MAVEN_REPO, exists, etc...
      • A: the process would be transparent for the maven integration module

      Maven approach
      --------------

      The main issue with the Maven approach is that we have a chicken-egg p roblem: we would be copying the jars to the repo using maven, but maven wouldn't run because the dependencies check would fail. The solution would be overriding the jars on build.properties and creating a postGoal on plugin:install to copy the jars.

      • A: easier to implement
      • A: the process would be isolated to the maven integration module
      • D: overriding the jars is kind of a hack solution

      If we opt for the maven approach, I can work on a patch (well, I could do the same for the Ant approach too, but I rather not touch those evil scripts

      1. CACTUS-137
        7 kB
        Felipe Leme
      2. CACTUS-137-using-properties.patch
        6 kB
        Felipe Leme
      3. CACTUS-137-using-properties-2.patch
        6 kB
        Felipe Leme

        Activity

        Felipe Leme created issue -
        Felipe Leme made changes -
        Field Original Value New Value
        Attachment CACTUS-137 [ 15052 ]
        Felipe Leme made changes -
        Attachment CACTUS-137-using-properties.patch [ 15057 ]
        Felipe Leme made changes -
        Felipe Leme made changes -
        Assignee Felipe Leme [ felipeal ]
        Felipe Leme made changes -
        Resolution Fixed [ 1 ]
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 1.7 [ 10693 ]
        Vincent Massol made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Felipe Leme
            Reporter:
            Felipe Leme
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development