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

update maven to 3.3.9 and refactor toolchain

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.1.0
    • Fix Version/s: 1.2.0
    • Component/s: toolchain
    • Labels:
      None

      Description

      The code for downloading maven is suboptimal. Code is distributed in two files and version numbers are spread all over. refactor it.

      And btw: maven is now at 3.3.9

        Issue Links

          Activity

          Hide
          cos Konstantin Boudnik added a comment -

          Looks good (without actually trying it).

          I believe the same pattern (extracting the version number from the puppet actions) could be applied to the other software packages we pull from Apache for the toolchain. Gradle, groovy, jdk - all of these are polluted with version numbers everywhere. Would be great to have to all fixed as Olaf Flebbe did for Maven.

          Show
          cos Konstantin Boudnik added a comment - Looks good (without actually trying it). I believe the same pattern (extracting the version number from the puppet actions) could be applied to the other software packages we pull from Apache for the toolchain. Gradle, groovy, jdk - all of these are polluted with version numbers everywhere. Would be great to have to all fixed as Olaf Flebbe did for Maven.
          Hide
          oflebbe Olaf Flebbe added a comment -

          I like the idea to give out some "easy" tickets for novice contributors to get into the project. What you suggesting would be one of it.

          Show
          oflebbe Olaf Flebbe added a comment - I like the idea to give out some "easy" tickets for novice contributors to get into the project. What you suggesting would be one of it.
          Hide
          oflebbe Olaf Flebbe added a comment -

          Thanks for looking into it Konstantin Boudnik, will push it in a minute.

          Show
          oflebbe Olaf Flebbe added a comment - Thanks for looking into it Konstantin Boudnik , will push it in a minute.

            People

            • Assignee:
              oflebbe Olaf Flebbe
              Reporter:
              oflebbe Olaf Flebbe
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development