Return-Path: <commits-return-2820-archive-asf-public=cust-asf.ponee.io@flume.apache.org>
X-Original-To: archive-asf-public-internal@cust-asf2.ponee.io
Delivered-To: archive-asf-public-internal@cust-asf2.ponee.io
Received: from cust-asf.ponee.io (cust-asf.ponee.io [163.172.22.183])
	by cust-asf2.ponee.io (Postfix) with ESMTP id 7894B200D13
	for <archive-asf-public-internal@cust-asf2.ponee.io>; Fri, 15 Sep 2017 14:34:20 +0200 (CEST)
Received: by cust-asf.ponee.io (Postfix)
	id 774641609CF; Fri, 15 Sep 2017 12:34:20 +0000 (UTC)
Delivered-To: archive-asf-public@cust-asf.ponee.io
Received: from mail.apache.org (hermes.apache.org [140.211.11.3])
	by cust-asf.ponee.io (Postfix) with SMTP id C333D1609D1
	for <archive-asf-public@cust-asf.ponee.io>; Fri, 15 Sep 2017 14:34:19 +0200 (CEST)
Received: (qmail 93331 invoked by uid 500); 15 Sep 2017 12:34:19 -0000
Mailing-List: contact commits-help@flume.apache.org; run by ezmlm
Precedence: bulk
List-Help: <mailto:commits-help@flume.apache.org>
List-Unsubscribe: <mailto:commits-unsubscribe@flume.apache.org>
List-Post: <mailto:commits@flume.apache.org>
List-Id: <commits.flume.apache.org>
Reply-To: dev@flume.apache.org
Delivered-To: mailing list commits@flume.apache.org
Received: (qmail 93315 invoked by uid 99); 15 Sep 2017 12:34:18 -0000
Received: from git1-us-west.apache.org (HELO git1-us-west.apache.org) (140.211.11.23)
    by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 15 Sep 2017 12:34:18 +0000
Received: by git1-us-west.apache.org (ASF Mail Server at git1-us-west.apache.org, from userid 33)
	id D1454F56C9; Fri, 15 Sep 2017 12:34:18 +0000 (UTC)
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
From: denes@apache.org
To: commits@flume.apache.org
Date: Fri, 15 Sep 2017 12:34:19 -0000
Message-Id: <92ce3688107748f7b5ffcadd3813a549@git.apache.org>
In-Reply-To: <d5f92b316642469aa387cd8e1f05e1f3@git.apache.org>
References: <d5f92b316642469aa387cd8e1f05e1f3@git.apache.org>
X-Mailer: ASF-Git Admin Mailer
Subject: [2/2] flume git commit: Fix source assembly
archived-at: Fri, 15 Sep 2017 12:34:20 -0000

Fix source assembly

flume-checkstyle breaks the assembly because its parent is not the flume-parent
Removing the moduleSets definition from the src assembly solved the issue.
Files are added based on fileSets, the resulting tarball's content equals
to the result of the dev-support/generate-source-release.sh in a clean
working directory.

This closes #175

Reviewers: Miklos Csanady, Ferenc Szabo

(Denes Arvay via Denes Arvay)


Project: http://git-wip-us.apache.org/repos/asf/flume/repo
Commit: http://git-wip-us.apache.org/repos/asf/flume/commit/43a3c40d
Tree: http://git-wip-us.apache.org/repos/asf/flume/tree/43a3c40d
Diff: http://git-wip-us.apache.org/repos/asf/flume/diff/43a3c40d

Branch: refs/heads/flume-1.8
Commit: 43a3c40d45f0da025cc9dc00e1d615b6c8366359
Parents: 4ddde20
Author: Denes Arvay <denes@apache.org>
Authored: Fri Sep 15 13:33:20 2017 +0200
Committer: Denes Arvay <denes@apache.org>
Committed: Fri Sep 15 14:26:45 2017 +0200

----------------------------------------------------------------------
 flume-ng-dist/src/main/assembly/src.xml | 77 ++--------------------------
 1 file changed, 3 insertions(+), 74 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/flume/blob/43a3c40d/flume-ng-dist/src/main/assembly/src.xml
----------------------------------------------------------------------
diff --git a/flume-ng-dist/src/main/assembly/src.xml b/flume-ng-dist/src/main/assembly/src.xml
index bf874ca..a79bed5 100644
--- a/flume-ng-dist/src/main/assembly/src.xml
+++ b/flume-ng-dist/src/main/assembly/src.xml
@@ -31,91 +31,20 @@
 
   <baseDirectory>apache-flume-${project.version}-src</baseDirectory>
 
-  <moduleSets>
-
-    <moduleSet>
-      <useAllReactorProjects>true</useAllReactorProjects>
-
-      <includes>
-        <include>org.apache.flume:flume-ng-configuration</include>
-        <include>org.apache.flume:flume-ng-sdk</include>
-        <include>org.apache.flume:flume-ng-core</include>
-        <include>org.apache.flume:flume-ng-node</include>
-        <include>org.apache.flume:flume-ng-dist</include>
-        <include>org.apache.flume:flume-ng-channels</include>
-        <include>org.apache.flume:flume-ng-sinks</include>
-        <include>org.apache.flume:flume-ng-sources</include>
-        <include>org.apache.flume:flume-ng-legacy-sources</include>
-        <include>org.apache.flume:flume-ng-clients</include>
-        <include>org.apache.flume:flume-ng-embedded-agent</include>
-        <include>org.apache.flume:flume-tools</include>
-        <include>org.apache.flume:flume-ng-auth</include>
-        <include>org.apache.flume:flume-shared</include>
-      </includes>
-
-      <sources>
-        <includeModuleDirectory>true</includeModuleDirectory>
-        <excludeSubModuleDirectories>false</excludeSubModuleDirectories>
-
-        <fileSets>
-          <fileSet>
-            <excludes>
-              <exclude>target/**</exclude>
-              <exclude>*/target/**</exclude>
-              <exclude>.classpath</exclude>
-              <exclude>*/.classpath</exclude>
-              <exclude>.project</exclude>
-              <exclude>*/.project</exclude>
-              <exclude>.settings/**</exclude>
-              <exclude>*/.settings/**</exclude>
-            </excludes>
-          </fileSet>
-        </fileSets>
-      </sources>
-
-    </moduleSet>
-
-  </moduleSets>
-
    <fileSets>
     <fileSet>
       <directory>../</directory>
 
       <excludes>
-        <exclude>flume-ng-configuration/**</exclude>
-        <exclude>flume-ng-sdk/**</exclude>
-        <exclude>flume-ng-core/**</exclude>
-        <exclude>flume-ng-node/**</exclude>
-        <exclude>flume-ng-dist/**</exclude>
-        <exclude>flume-ng-channels/**</exclude>
-        <exclude>flume-ng-sinks/**</exclude>
-        <exclude>flume-ng-sources/**</exclude>
-        <exclude>flume-ng-legacy-sources/**</exclude>
-        <exclude>flume-ng-clients/**</exclude>
-        <exclude>flume-ng-embedded-agent/**</exclude>
-        <exclude>flume-tools/**</exclude>
         <exclude>**/target/**</exclude>
         <exclude>**/.classpath</exclude>
         <exclude>**/.project</exclude>
+        <exclude>**/.idea/**</exclude>
+        <exclude>**/*.iml</exclude>
         <exclude>**/.settings/**</exclude>
         <exclude>lib/**</exclude>
+        <exclude>**/.DS_Store</exclude>
       </excludes>
-
-      <includes>
-        <include>.gitignore</include>
-        <include>DEVNOTES</include>
-        <include>README</include>
-        <include>LICENSE</include>
-        <include>NOTICE</include>
-        <include>CHANGELOG</include>
-        <include>RELEASE-NOTES</include>
-        <include>bin/**</include>
-        <include>conf/**</include>
-        <include>pom.xml</include>
-        <include>flume-ng-doc/**</include>
-        <include>flume-ng-tests/**</include>
-        <include>dev-support/**</include>
-      </includes>
     </fileSet>
   </fileSets>