Uploaded image for project: 'Flink'
  1. Flink
  2. FLINK-6774

build-helper-maven-plugin version not set

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 1.3.0
    • Fix Version/s: 1.4.0, 1.3.2
    • Component/s: Build System
    • Labels:
      None

      Description

      some modules forgot to specify the version of their build-helper-maven-plugin which causes the following warning in a maven build:

      [WARNING] 'build.plugins.plugin.version' for org.codehaus.mojo:build-helper-maven-plugin is missing. @ org.apache.flink:flink-connector-kafka-base_${scala.binary.version}:[unknown-version], /home/nico/Projects/flink/flink-connectors/flink-connector-kafka-base/pom.xml, line 216, column 12
      [WARNING] It is highly recommended to fix these problems because they threaten the stability of your build.
      

        Issue Links

          Activity

          Hide
          Zentol Chesnay Schepler added a comment -

          1.3: b20ae53a9957683f3b7fe44c0a799422021fbb69
          1.4: 1ae4ac28b102c1a33d071dd1184782eb12625bbc

          Show
          Zentol Chesnay Schepler added a comment - 1.3: b20ae53a9957683f3b7fe44c0a799422021fbb69 1.4: 1ae4ac28b102c1a33d071dd1184782eb12625bbc
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user asfgit closed the pull request at:

          https://github.com/apache/flink/pull/4017

          Show
          githubbot ASF GitHub Bot added a comment - Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/4017
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user rmetzger commented on the issue:

          https://github.com/apache/flink/pull/4017

          That would be nice, yes. Thx

          Sent from my iPhone

          > On 23. Jun 2017, at 11:52, zentol <notifications@github.com> wrote:
          >
          > I could include this in my merge PR #4124 if you want @rmetzger
          >
          > —
          > You are receiving this because you were mentioned.
          > Reply to this email directly, view it on GitHub, or mute the thread.
          >

          Show
          githubbot ASF GitHub Bot added a comment - Github user rmetzger commented on the issue: https://github.com/apache/flink/pull/4017 That would be nice, yes. Thx Sent from my iPhone > On 23. Jun 2017, at 11:52, zentol <notifications@github.com> wrote: > > I could include this in my merge PR #4124 if you want @rmetzger > > — > You are receiving this because you were mentioned. > Reply to this email directly, view it on GitHub, or mute the thread. >
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user zentol commented on the issue:

          https://github.com/apache/flink/pull/4017

          I could include this in my merge PR https://github.com/apache/flink/pull/4124 if you want @rmetzger

          Show
          githubbot ASF GitHub Bot added a comment - Github user zentol commented on the issue: https://github.com/apache/flink/pull/4017 I could include this in my merge PR https://github.com/apache/flink/pull/4124 if you want @rmetzger
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user rmetzger commented on the issue:

          https://github.com/apache/flink/pull/4017

          With the current travis situation, I can not validate this branch.

          Show
          githubbot ASF GitHub Bot added a comment - Github user rmetzger commented on the issue: https://github.com/apache/flink/pull/4017 With the current travis situation, I can not validate this branch.
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user rmetzger commented on the issue:

          https://github.com/apache/flink/pull/4017

          Change looks good.

          I'll merge it once travis gives green light on my fork.

          Show
          githubbot ASF GitHub Bot added a comment - Github user rmetzger commented on the issue: https://github.com/apache/flink/pull/4017 Change looks good. I'll merge it once travis gives green light on my fork.
          Hide
          githubbot ASF GitHub Bot added a comment -

          GitHub user NicoK opened a pull request:

          https://github.com/apache/flink/pull/4017

          FLINK-6774[build] set missing build-helper-maven-plugin version

          this PR sets the version number on those sub-modules which did not set it yet

          You can merge this pull request into a Git repository by running:

          $ git pull https://github.com/NicoK/flink flink-6774

          Alternatively you can review and apply these changes as the patch at:

          https://github.com/apache/flink/pull/4017.patch

          To close this pull request, make a commit to your master/trunk branch
          with (at least) the following in the commit message:

          This closes #4017


          commit 7e1a0e2000c33f9c58c7fae533211087fcb153a6
          Author: Nico Kruber <nico@data-artisans.com>
          Date: 2017-05-30T12:52:23Z

          FLINK-6774[build] set missing build-helper-maven-plugin version


          Show
          githubbot ASF GitHub Bot added a comment - GitHub user NicoK opened a pull request: https://github.com/apache/flink/pull/4017 FLINK-6774 [build] set missing build-helper-maven-plugin version this PR sets the version number on those sub-modules which did not set it yet You can merge this pull request into a Git repository by running: $ git pull https://github.com/NicoK/flink flink-6774 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/flink/pull/4017.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #4017 commit 7e1a0e2000c33f9c58c7fae533211087fcb153a6 Author: Nico Kruber <nico@data-artisans.com> Date: 2017-05-30T12:52:23Z FLINK-6774 [build] set missing build-helper-maven-plugin version

            People

            • Assignee:
              NicoK Nico Kruber
              Reporter:
              NicoK Nico Kruber
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development