You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@asterixdb.apache.org by AsterixDB Code Review <do...@asterix-gerrit.ics.uci.edu> on 2021/07/07 19:42:06 UTC

Change in asterixdb[master]: [NO ISSUE] Touch 3rdpartylicenses.txt in skip.npm

From Ian Maxon <im...@uci.edu>:

Ian Maxon has uploaded this change for review. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12243 )


Change subject: [NO ISSUE] Touch 3rdpartylicenses.txt in skip.npm
......................................................................

[NO ISSUE] Touch 3rdpartylicenses.txt in skip.npm

- user model changes: no
- storage format changes: no
- interface changes: no

Details:
(Ab)use antrun to make 'skip.npm' work without having to touch
asterixdb/asterix-dashboard/target/dashboard/static/3rdpartylicenses.txt
before the build.

Change-Id: Ic95e592b42139b4750af8bb20297f926b3c973e4
---
M asterixdb/asterix-dashboard/pom.xml
M asterixdb/pom.xml
2 files changed, 30 insertions(+), 0 deletions(-)



  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/43/12243/1

diff --git a/asterixdb/asterix-dashboard/pom.xml b/asterixdb/asterix-dashboard/pom.xml
index 7937f75..07606f2 100644
--- a/asterixdb/asterix-dashboard/pom.xml
+++ b/asterixdb/asterix-dashboard/pom.xml
@@ -175,6 +175,24 @@
               <licenseDirectory>${basedir}/src/main/licenses/content</licenseDirectory>
             </configuration>
           </plugin>
+          <plugin>
+            <groupId>org.apache.maven.plugins</groupId>
+            <artifactId>maven-antrun-plugin</artifactId>
+            <executions>
+              <execution>
+                <id>touch-3rdparty</id>
+                <phase>${skip-npm-touch.stage}</phase>
+                <configuration>
+                  <target>
+                      <touch file="${basedir}/target/dashboard/static/3rdpartylicenses.txt" mkdirs="true"/>
+                  </target>
+                </configuration>
+                <goals>
+                  <goal>run</goal>
+                </goals>
+              </execution>
+            </executions>
+          </plugin>
         </plugins>
       </build>
     </profile>
diff --git a/asterixdb/pom.xml b/asterixdb/pom.xml
index 1cc4ea6..9af167c 100644
--- a/asterixdb/pom.xml
+++ b/asterixdb/pom.xml
@@ -79,6 +79,7 @@
     <prepare-env.stage>none</prepare-env.stage>
     <shim.stage>none</shim.stage>
     <pytestlib.stage>none</pytestlib.stage>
+    <skip-npm-touch.stage>none</skip-npm-touch.stage>
 
     <!-- Versions under dependencymanagement or used in many projects via properties -->
     <algebricks.version>0.3.7-SNAPSHOT</algebricks.version>
@@ -711,6 +712,17 @@
       </properties>
     </profile>
     <profile>
+      <id>skip-npm</id>
+      <activation>
+        <property>
+          <name>skip.npm</name>
+        </property>
+      </activation>
+      <properties>
+          <skip-npm-touch.stage>process-sources</skip-npm-touch.stage>
+      </properties>
+    </profile>
+    <profile>
       <id>invalid-tests</id>
       <properties>
         <invalid.tests />

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12243
To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Change-Id: Ic95e592b42139b4750af8bb20297f926b3c973e4
Gerrit-Change-Number: 12243
Gerrit-PatchSet: 1
Gerrit-Owner: Ian Maxon <im...@uci.edu>
Gerrit-MessageType: newchange

Change in asterixdb[cheshire-cat]: [NO ISSUE] Touch 3rdpartylicenses.txt in skip.npm

Posted by AsterixDB Code Review <do...@asterix-gerrit.ics.uci.edu>.
From Jenkins <je...@fulliautomatix.ics.uci.edu>:

Jenkins has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12243 )

Change subject: [NO ISSUE] Touch 3rdpartylicenses.txt in skip.npm
......................................................................


Patch Set 2: Integration-Tests+1

Integration Tests Successful

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/12187/ : SUCCESS


-- 
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12243
To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-Project: asterixdb
Gerrit-Branch: cheshire-cat
Gerrit-Change-Id: Ic95e592b42139b4750af8bb20297f926b3c973e4
Gerrit-Change-Number: 12243
Gerrit-PatchSet: 2
Gerrit-Owner: Ian Maxon <im...@uci.edu>
Gerrit-Reviewer: Hussain Towaileb <hu...@couchbase.com>
Gerrit-Reviewer: Ian Maxon <im...@uci.edu>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Blow <mb...@apache.org>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-CC: Anon. E. Moose #1000171
Gerrit-Comment-Date: Wed, 07 Jul 2021 22:56:38 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment

Change in asterixdb[master]: [NO ISSUE] Touch 3rdpartylicenses.txt in skip.npm

Posted by AsterixDB Code Review <do...@asterix-gerrit.ics.uci.edu>.
From Ian Maxon <im...@uci.edu>:

Ian Maxon has uploaded this change for review. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12243 )


Change subject: [NO ISSUE] Touch 3rdpartylicenses.txt in skip.npm
......................................................................

[NO ISSUE] Touch 3rdpartylicenses.txt in skip.npm

- user model changes: no
- storage format changes: no
- interface changes: no

Details:
(Ab)use antrun to make 'skip.npm' work without having to touch
asterixdb/asterix-dashboard/target/dashboard/static/3rdpartylicenses.txt
before the build.

Change-Id: Ic95e592b42139b4750af8bb20297f926b3c973e4
---
M asterixdb/asterix-dashboard/pom.xml
M asterixdb/pom.xml
2 files changed, 30 insertions(+), 0 deletions(-)



  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/43/12243/1

diff --git a/asterixdb/asterix-dashboard/pom.xml b/asterixdb/asterix-dashboard/pom.xml
index 7937f75..07606f2 100644
--- a/asterixdb/asterix-dashboard/pom.xml
+++ b/asterixdb/asterix-dashboard/pom.xml
@@ -175,6 +175,24 @@
               <licenseDirectory>${basedir}/src/main/licenses/content</licenseDirectory>
             </configuration>
           </plugin>
+          <plugin>
+            <groupId>org.apache.maven.plugins</groupId>
+            <artifactId>maven-antrun-plugin</artifactId>
+            <executions>
+              <execution>
+                <id>touch-3rdparty</id>
+                <phase>${skip-npm-touch.stage}</phase>
+                <configuration>
+                  <target>
+                      <touch file="${basedir}/target/dashboard/static/3rdpartylicenses.txt" mkdirs="true"/>
+                  </target>
+                </configuration>
+                <goals>
+                  <goal>run</goal>
+                </goals>
+              </execution>
+            </executions>
+          </plugin>
         </plugins>
       </build>
     </profile>
diff --git a/asterixdb/pom.xml b/asterixdb/pom.xml
index 1cc4ea6..9af167c 100644
--- a/asterixdb/pom.xml
+++ b/asterixdb/pom.xml
@@ -79,6 +79,7 @@
     <prepare-env.stage>none</prepare-env.stage>
     <shim.stage>none</shim.stage>
     <pytestlib.stage>none</pytestlib.stage>
+    <skip-npm-touch.stage>none</skip-npm-touch.stage>
 
     <!-- Versions under dependencymanagement or used in many projects via properties -->
     <algebricks.version>0.3.7-SNAPSHOT</algebricks.version>
@@ -711,6 +712,17 @@
       </properties>
     </profile>
     <profile>
+      <id>skip-npm</id>
+      <activation>
+        <property>
+          <name>skip.npm</name>
+        </property>
+      </activation>
+      <properties>
+          <skip-npm-touch.stage>process-sources</skip-npm-touch.stage>
+      </properties>
+    </profile>
+    <profile>
       <id>invalid-tests</id>
       <properties>
         <invalid.tests />

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12243
To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Change-Id: Ic95e592b42139b4750af8bb20297f926b3c973e4
Gerrit-Change-Number: 12243
Gerrit-PatchSet: 1
Gerrit-Owner: Ian Maxon <im...@uci.edu>
Gerrit-MessageType: newchange

Change in asterixdb[cheshire-cat]: [NO ISSUE] Touch 3rdpartylicenses.txt in skip.npm

Posted by AsterixDB Code Review <do...@asterix-gerrit.ics.uci.edu>.
From Ian Maxon <im...@uci.edu>:

Ian Maxon has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12243 )

Change subject: [NO ISSUE] Touch 3rdpartylicenses.txt in skip.npm
......................................................................


Patch Set 3: Contrib+1

contrib tests are no worse


-- 
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12243
To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-Project: asterixdb
Gerrit-Branch: cheshire-cat
Gerrit-Change-Id: Ic95e592b42139b4750af8bb20297f926b3c973e4
Gerrit-Change-Number: 12243
Gerrit-PatchSet: 3
Gerrit-Owner: Ian Maxon <im...@uci.edu>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Hussain Towaileb <hu...@couchbase.com>
Gerrit-Reviewer: Hussain Towaileb <hu...@gmail.com>
Gerrit-Reviewer: Ian Maxon <im...@uci.edu>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Blow <mb...@apache.org>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-Comment-Date: Tue, 20 Jul 2021 00:32:23 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment

Change in asterixdb[cheshire-cat]: [NO ISSUE] Touch 3rdpartylicenses.txt in skip.npm

Posted by AsterixDB Code Review <do...@asterix-gerrit.ics.uci.edu>.
From Ian Maxon <im...@uci.edu>:

Hello Anon. E. Moose #1000171, Hussain Towaileb, Hussain Towaileb, Till Westmann, Jenkins, Michael Blow, 

I'd like you to reexamine a change. Please visit

    https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12243

to look at the new patch set (#3).

Change subject: [NO ISSUE] Touch 3rdpartylicenses.txt in skip.npm
......................................................................

[NO ISSUE] Touch 3rdpartylicenses.txt in skip.npm

- user model changes: no
- storage format changes: no
- interface changes: no

Details:
(Ab)use antrun to make 'skip.npm' work without having to touch
asterixdb/asterix-dashboard/target/dashboard/static/3rdpartylicenses.txt
before the build.

Change-Id: Ic95e592b42139b4750af8bb20297f926b3c973e4
---
M asterixdb/asterix-dashboard/pom.xml
M asterixdb/pom.xml
2 files changed, 35 insertions(+), 0 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/43/12243/3
-- 
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12243
To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-Project: asterixdb
Gerrit-Branch: cheshire-cat
Gerrit-Change-Id: Ic95e592b42139b4750af8bb20297f926b3c973e4
Gerrit-Change-Number: 12243
Gerrit-PatchSet: 3
Gerrit-Owner: Ian Maxon <im...@uci.edu>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Hussain Towaileb <hu...@couchbase.com>
Gerrit-Reviewer: Hussain Towaileb <hu...@gmail.com>
Gerrit-Reviewer: Ian Maxon <im...@uci.edu>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Blow <mb...@apache.org>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-MessageType: newpatchset

Change in asterixdb[cheshire-cat]: [NO ISSUE] Touch 3rdpartylicenses.txt in skip.npm

Posted by AsterixDB Code Review <do...@asterix-gerrit.ics.uci.edu>.
From Ian Maxon <im...@uci.edu>:

Ian Maxon has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12243 )

Change subject: [NO ISSUE] Touch 3rdpartylicenses.txt in skip.npm
......................................................................


Patch Set 3: Code-Review+1

(1 comment)

https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12243/2/asterixdb/pom.xml 
File asterixdb/pom.xml:

https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12243/2/asterixdb/pom.xml@719 
PS2, Line 719: 		 
> WS
a pox upon the house of \t, fixed



-- 
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12243
To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-Project: asterixdb
Gerrit-Branch: cheshire-cat
Gerrit-Change-Id: Ic95e592b42139b4750af8bb20297f926b3c973e4
Gerrit-Change-Number: 12243
Gerrit-PatchSet: 3
Gerrit-Owner: Ian Maxon <im...@uci.edu>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Hussain Towaileb <hu...@couchbase.com>
Gerrit-Reviewer: Hussain Towaileb <hu...@gmail.com>
Gerrit-Reviewer: Ian Maxon <im...@uci.edu>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Blow <mb...@apache.org>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-Comment-Date: Thu, 15 Jul 2021 19:43:55 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: Till Westmann <ti...@apache.org>
Gerrit-MessageType: comment

Change in asterixdb[cheshire-cat]: [NO ISSUE] Touch 3rdpartylicenses.txt in skip.npm

Posted by AsterixDB Code Review <do...@asterix-gerrit.ics.uci.edu>.
Anon. E. Moose #1000171 has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12243 )

Change subject: [NO ISSUE] Touch 3rdpartylicenses.txt in skip.npm
......................................................................


Patch Set 2: Contrib-2

Analytics Compatibility Tests Failed
https://cbjenkins.page.link/FXawtkeYC1akWr4X6 : UNSTABLE


-- 
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12243
To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-Project: asterixdb
Gerrit-Branch: cheshire-cat
Gerrit-Change-Id: Ic95e592b42139b4750af8bb20297f926b3c973e4
Gerrit-Change-Number: 12243
Gerrit-PatchSet: 2
Gerrit-Owner: Ian Maxon <im...@uci.edu>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Hussain Towaileb <hu...@couchbase.com>
Gerrit-Reviewer: Hussain Towaileb <hu...@gmail.com>
Gerrit-Reviewer: Ian Maxon <im...@uci.edu>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Blow <mb...@apache.org>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-Comment-Date: Thu, 08 Jul 2021 00:36:14 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment

Change in asterixdb[cheshire-cat]: [NO ISSUE] Touch 3rdpartylicenses.txt in skip.npm

Posted by AsterixDB Code Review <do...@asterix-gerrit.ics.uci.edu>.
Anon. E. Moose #1000171 has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12243 )

Change subject: [NO ISSUE] Touch 3rdpartylicenses.txt in skip.npm
......................................................................


Patch Set 3:

Analytics Compatibility Compilation Successful
https://cbjenkins.page.link/QKpDfPWyfVtu4aHdA : SUCCESS


-- 
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12243
To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-Project: asterixdb
Gerrit-Branch: cheshire-cat
Gerrit-Change-Id: Ic95e592b42139b4750af8bb20297f926b3c973e4
Gerrit-Change-Number: 12243
Gerrit-PatchSet: 3
Gerrit-Owner: Ian Maxon <im...@uci.edu>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Hussain Towaileb <hu...@couchbase.com>
Gerrit-Reviewer: Hussain Towaileb <hu...@gmail.com>
Gerrit-Reviewer: Ian Maxon <im...@uci.edu>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Blow <mb...@apache.org>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-Comment-Date: Thu, 15 Jul 2021 19:49:31 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment

Change in asterixdb[cheshire-cat]: [NO ISSUE] Touch 3rdpartylicenses.txt in skip.npm

Posted by AsterixDB Code Review <do...@asterix-gerrit.ics.uci.edu>.
Anon. E. Moose #1000171 has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12243 )

Change subject: [NO ISSUE] Touch 3rdpartylicenses.txt in skip.npm
......................................................................


Patch Set 3: Contrib-2

Analytics Compatibility Tests Failed
https://cbjenkins.page.link/j6UuDyJpDVqy8bJv7 : UNSTABLE


-- 
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12243
To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-Project: asterixdb
Gerrit-Branch: cheshire-cat
Gerrit-Change-Id: Ic95e592b42139b4750af8bb20297f926b3c973e4
Gerrit-Change-Number: 12243
Gerrit-PatchSet: 3
Gerrit-Owner: Ian Maxon <im...@uci.edu>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Hussain Towaileb <hu...@couchbase.com>
Gerrit-Reviewer: Hussain Towaileb <hu...@gmail.com>
Gerrit-Reviewer: Ian Maxon <im...@uci.edu>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Blow <mb...@apache.org>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-Comment-Date: Thu, 15 Jul 2021 23:12:10 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment

Change in asterixdb[cheshire-cat]: [NO ISSUE] Touch 3rdpartylicenses.txt in skip.npm

Posted by AsterixDB Code Review <do...@asterix-gerrit.ics.uci.edu>.
Anon. E. Moose #1000171 has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12243 )

Change subject: [NO ISSUE] Touch 3rdpartylicenses.txt in skip.npm
......................................................................


Patch Set 3:

Analytics Compatibility Tests Failed
https://cbjenkins.page.link/s4ZGDzfw97ywQVWb9 : UNSTABLE


-- 
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/12243
To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-Project: asterixdb
Gerrit-Branch: cheshire-cat
Gerrit-Change-Id: Ic95e592b42139b4750af8bb20297f926b3c973e4
Gerrit-Change-Number: 12243
Gerrit-PatchSet: 3
Gerrit-Owner: Ian Maxon <im...@uci.edu>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Hussain Towaileb <hu...@couchbase.com>
Gerrit-Reviewer: Hussain Towaileb <hu...@gmail.com>
Gerrit-Reviewer: Ian Maxon <im...@uci.edu>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Blow <mb...@apache.org>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-Comment-Date: Mon, 19 Jul 2021 23:12:53 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment