Flume
  1. Flume
  2. FLUME-2567

Remove unneeded repository declarations in pom.xml

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: v1.7.0
    • Component/s: None
    • Labels:
      None

      Description

      pom.xml contains multiple repository declarations that do not seem to be needed (jboss, maven central, cloudera). Actually, it seems to me that none of these is needed.

      Can we remove all repository declarations and let it use just the default (Maven Central)?

      1. FLUME-2567-0.patch
        4 kB
        Santiago M. Mola

        Issue Links

          Activity

          Hide
          Joey Echeverria added a comment -

          Hey Santiago M. Mola, did you test this patch with a clean environment? I plan on testing the patch later today, but I wanted to see what you'd already done.

          Show
          Joey Echeverria added a comment - Hey Santiago M. Mola , did you test this patch with a clean environment? I plan on testing the patch later today, but I wanted to see what you'd already done.
          Hide
          Santiago M. Mola added a comment - - edited

          Joey Echeverria Yes. I've tested it on a clean environment (https://travis-ci.org/Stratio/flume/builds/42887413). Local Maven repository should be empty before every build.

          Show
          Santiago M. Mola added a comment - - edited Joey Echeverria Yes. I've tested it on a clean environment ( https://travis-ci.org/Stratio/flume/builds/42887413 ). Local Maven repository should be empty before every build.
          Hide
          Lior Zeno added a comment -

          Why hasn't this patch been committed?

          Show
          Lior Zeno added a comment - Why hasn't this patch been committed?
          Hide
          Mike Percy added a comment -

          The squeaky wheel gets the grease.

          +1, LGTM. I am about to commit this.

          Show
          Mike Percy added a comment - The squeaky wheel gets the grease. +1, LGTM. I am about to commit this.
          Hide
          ASF subversion and git services added a comment -

          Commit 4eacba1930c3abfddf4fb50c5a272798708df811 in flume's branch refs/heads/trunk from Santiago M. Mola
          [ https://git-wip-us.apache.org/repos/asf?p=flume.git;h=4eacba1 ]

          FLUME-2567. Remove external Maven repositories

          (Santiago M. Mola via Mike Percy)

          Show
          ASF subversion and git services added a comment - Commit 4eacba1930c3abfddf4fb50c5a272798708df811 in flume's branch refs/heads/trunk from Santiago M. Mola [ https://git-wip-us.apache.org/repos/asf?p=flume.git;h=4eacba1 ] FLUME-2567 . Remove external Maven repositories (Santiago M. Mola via Mike Percy)
          Hide
          Mike Percy added a comment -

          Pushed to trunk. Thanks for the patch, Santiago!

          Show
          Mike Percy added a comment - Pushed to trunk. Thanks for the patch, Santiago!
          Hide
          Hudson added a comment -

          FAILURE: Integrated in Flume-trunk-hbase-1 #168 (See https://builds.apache.org/job/Flume-trunk-hbase-1/168/)
          FLUME-2567. Remove external Maven repositories (mpercy: http://git-wip-us.apache.org/repos/asf/flume/repo?p=flume.git&a=commit&h=4eacba1930c3abfddf4fb50c5a272798708df811)

          • flume-ng-sinks/flume-ng-morphline-solr-sink/pom.xml
          • pom.xml
          • flume-ng-sinks/flume-dataset-sink/pom.xml
          Show
          Hudson added a comment - FAILURE: Integrated in Flume-trunk-hbase-1 #168 (See https://builds.apache.org/job/Flume-trunk-hbase-1/168/ ) FLUME-2567 . Remove external Maven repositories (mpercy: http://git-wip-us.apache.org/repos/asf/flume/repo?p=flume.git&a=commit&h=4eacba1930c3abfddf4fb50c5a272798708df811 ) flume-ng-sinks/flume-ng-morphline-solr-sink/pom.xml pom.xml flume-ng-sinks/flume-dataset-sink/pom.xml

            People

            • Assignee:
              Santiago M. Mola
              Reporter:
              Santiago M. Mola
            • Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development