Uploaded image for project: 'Bigtop'
  1. Bigtop
  2. BIGTOP-1585

test artifacts tasks include extra tasks for nonexisting components

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 0.8.0
    • Fix Version/s: 1.0.0
    • Component/s: build
    • Labels:
      None

      Description

      Among the install- tasks I see the following

      install-pom.xml - Installs pom.xml artifact with Maven
      install-README - Installs README artifact with Maven
      ...
      install-target - Installs target artifact with Maven
      

      Clearly, those shouldn't be present.

        Activity

        Hide
        cos Konstantin Boudnik added a comment -

        Instead of doing a directory operation, I'd rather read the list of modules from the pom.xml and use them for the iteration.

        Show
        cos Konstantin Boudnik added a comment - Instead of doing a directory operation, I'd rather read the list of modules from the pom.xml and use them for the iteration.
        Hide
        rvs Roman Shaposhnik added a comment -

        While I would love to master a more direct integration between gradle and Maven, I think your patch definitely makes the situation better. Hence +1

        Show
        rvs Roman Shaposhnik added a comment - While I would love to master a more direct integration between gradle and Maven, I think your patch definitely makes the situation better. Hence +1
        Hide
        cos Konstantin Boudnik added a comment - - edited

        While I'd love to see a better integration too I don't see an immediate solution without developing a plugin of some kind. This one provides a not-very-ugly fix for the bug. Will commit it in a bit a open a ticket to investigate a better gradle-maven integration

        Show
        cos Konstantin Boudnik added a comment - - edited While I'd love to see a better integration too I don't see an immediate solution without developing a plugin of some kind. This one provides a not-very-ugly fix for the bug. Will commit it in a bit a open a ticket to investigate a better gradle-maven integration
        Hide
        cos Konstantin Boudnik added a comment -

        Committed and pushed as
        a0035a3..40b6b2f HEAD -> master

        Show
        cos Konstantin Boudnik added a comment - Committed and pushed as a0035a3..40b6b2f HEAD -> master

          People

          • Assignee:
            cos Konstantin Boudnik
            Reporter:
            cos Konstantin Boudnik
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development