asterixdb-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ian Maxon (Code Review)" <do-not-re...@asterixdb.incubator.apache.org>
Subject Change in asterixdb[master]: ASTERIXDB-1870 - Cannot Build From 0.9.1 RC0 SA
Date Tue, 04 Apr 2017 06:32:38 GMT
Ian Maxon has submitted this change and it was merged.

Change subject: ASTERIXDB-1870 - Cannot Build From 0.9.1 RC0 SA
......................................................................


ASTERIXDB-1870 - Cannot Build From 0.9.1 RC0 SA

Fix RAT failure when building from source assembly, by no longer
including release-time git.properties in the source assembly

Change-Id: Ia3daf97b93ba4ffc5c69823213a455adbdf6e38c
Reviewed-on: https://asterix-gerrit.ics.uci.edu/1658
Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Reviewed-by: Ian Maxon <imaxon@apache.org>
---
M asterixdb/src/main/assembly/source.xml
1 file changed, 0 insertions(+), 10 deletions(-)

Approvals:
  Ian Maxon: Looks good to me, approved
  Jenkins: Verified; No violations found; Verified

Objections:
  Jenkins: Violations found



diff --git a/asterixdb/src/main/assembly/source.xml b/asterixdb/src/main/assembly/source.xml
index 34fc3e9..6552829 100644
--- a/asterixdb/src/main/assembly/source.xml
+++ b/asterixdb/src/main/assembly/source.xml
@@ -30,18 +30,8 @@
       <useDefaultExcludes>true</useDefaultExcludes>
       <excludes>
         <exclude>**/${project.build.directory}/**</exclude>
-        <exclude>**/git.properties</exclude>
         <exclude>release.properties</exclude>
-        <exclude>asterix-app/src/test/resources/externallib/**</exclude>
-        <exclude>asterix-installer/src/test/resources/clusterts/asterix-installer-*-binary-assembly/**</exclude>
       </excludes>
-    </fileSet>
-    <fileSet>
-      <directory>asterix-app/target/classes</directory>
-      <outputDirectory>./asterix-app/src/main/resources/</outputDirectory>
-      <includes>
-        <include>git.properties</include>
-      </includes>
     </fileSet>
   </fileSets>
 </assembly>

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1658
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia3daf97b93ba4ffc5c69823213a455adbdf6e38c
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow <mblow@apache.org>
Gerrit-Reviewer: Ian Maxon <imaxon@apache.org>
Gerrit-Reviewer: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Mime
View raw message