Flume
  1. Flume
  2. FLUME-906

Maven Avro plugin missing an entry in plugin dep management

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: v1.0.0
    • Fix Version/s: v1.0.0
    • Component/s: Build
    • Labels:
      None

      Description

      Looks like I forgot to include the maven avro plugin in the plugin dependency management when I removed the version in flume-ng-core's pom.

        Activity

        Hide
        Hudson added a comment -

        Integrated in flume-728 #87 (See https://builds.apache.org/job/flume-728/87/)
        FLUME-906: Maven Avro plugin missing an entry in plugin dep management

        esammer : http://svn.apache.org/viewvc/?view=rev&rev=1225383
        Files :

        • /incubator/flume/branches/flume-728/pom.xml
        Show
        Hudson added a comment - Integrated in flume-728 #87 (See https://builds.apache.org/job/flume-728/87/ ) FLUME-906 : Maven Avro plugin missing an entry in plugin dep management esammer : http://svn.apache.org/viewvc/?view=rev&rev=1225383 Files : /incubator/flume/branches/flume-728/pom.xml
        Hide
        E. Sammer added a comment -

        Committed to flume-728.

        Show
        E. Sammer added a comment - Committed to flume-728.
        Hide
        jiraposter@reviews.apache.org added a comment -

        -----------------------------------------------------------
        This is an automatically generated e-mail. To reply, visit:
        https://reviews.apache.org/r/3321/#review4139
        -----------------------------------------------------------

        Ship it!

        +1

        • Arvind

        On 2011-12-28 18:52:05, Eric Sammer wrote:

        -----------------------------------------------------------

        This is an automatically generated e-mail. To reply, visit:

        https://reviews.apache.org/r/3321/

        -----------------------------------------------------------

        (Updated 2011-12-28 18:52:05)

        Review request for Flume.

        Summary

        -------

        Maven is being cranky so we're just going to forego the property rather than fight with it.

        This addresses bug FLUME-906.

        https://issues.apache.org/jira/browse/FLUME-906

        Diffs

        -----

        pom.xml bad0cff

        Diff: https://reviews.apache.org/r/3321/diff

        Testing

        -------

        Thanks,

        Eric

        Show
        jiraposter@reviews.apache.org added a comment - ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3321/#review4139 ----------------------------------------------------------- Ship it! +1 Arvind On 2011-12-28 18:52:05, Eric Sammer wrote: ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3321/ ----------------------------------------------------------- (Updated 2011-12-28 18:52:05) Review request for Flume. Summary ------- Maven is being cranky so we're just going to forego the property rather than fight with it. This addresses bug FLUME-906 . https://issues.apache.org/jira/browse/FLUME-906 Diffs ----- pom.xml bad0cff Diff: https://reviews.apache.org/r/3321/diff Testing ------- Thanks, Eric
        Hide
        jiraposter@reviews.apache.org added a comment -

        -----------------------------------------------------------
        This is an automatically generated e-mail. To reply, visit:
        https://reviews.apache.org/r/3321/
        -----------------------------------------------------------

        Review request for Flume.

        Summary
        -------

        Maven is being cranky so we're just going to forego the property rather than fight with it.

        This addresses bug FLUME-906.
        https://issues.apache.org/jira/browse/FLUME-906

        Diffs


        pom.xml bad0cff

        Diff: https://reviews.apache.org/r/3321/diff

        Testing
        -------

        Thanks,

        Eric

        Show
        jiraposter@reviews.apache.org added a comment - ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3321/ ----------------------------------------------------------- Review request for Flume. Summary ------- Maven is being cranky so we're just going to forego the property rather than fight with it. This addresses bug FLUME-906 . https://issues.apache.org/jira/browse/FLUME-906 Diffs pom.xml bad0cff Diff: https://reviews.apache.org/r/3321/diff Testing ------- Thanks, Eric
        Hide
        E. Sammer added a comment -

        I'm wrong. I did include it, but maven is unhappy with the fact that the version is a property. Maybe it doesn't interpolate properties until after it evaluates pluginManagement? That's weird.

        Show
        E. Sammer added a comment - I'm wrong. I did include it, but maven is unhappy with the fact that the version is a property. Maybe it doesn't interpolate properties until after it evaluates pluginManagement? That's weird.

          People

          • Assignee:
            E. Sammer
            Reporter:
            E. Sammer
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development