>From Ian Maxon <[email protected]>:

Ian Maxon has submitted this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb-clients/+/20830?usp=email )

Change subject: [NO ISSUE][JDBC] Install TACO SDK deps in venv
......................................................................

[NO ISSUE][JDBC] Install TACO SDK deps in venv

The TACO SDK has some dependencies that may not be
present in the user's Python install, so make sure
they are installed in the venv used to package the
connector.

Change-Id: I5b9b1f4a8ecd82d75a61b8e1ea056b7fd13ac99a
Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb-clients/+/20830
Tested-by: Jenkins <[email protected]>
Reviewed-by: Peeyush Gupta <[email protected]>
---
M asterixdb-jdbc/asterix-jdbc-taco/pom.xml
1 file changed, 23 insertions(+), 0 deletions(-)

Approvals:
  Peeyush Gupta: Looks good to me, approved
  Jenkins: Verified




diff --git a/asterixdb-jdbc/asterix-jdbc-taco/pom.xml 
b/asterixdb-jdbc/asterix-jdbc-taco/pom.xml
index 99930a3..12de8a4 100644
--- a/asterixdb-jdbc/asterix-jdbc-taco/pom.xml
+++ b/asterixdb-jdbc/asterix-jdbc-taco/pom.xml
@@ -72,6 +72,7 @@
     
<build.taco.file>${project.build.directory}/${build.taco.filename}</build.taco.file>
     
<build.venv.directory>${project.build.directory}/venv</build.venv.directory>
     
<build.venv.python.path>${build.venv.directory}/bin/python</build.venv.python.path>
+    <build.venv.pip.path>${build.venv.directory}/bin/pip</build.venv.pip.path>
   </properties>

   <build>
@@ -172,6 +173,28 @@
             </configuration>
           </execution>
           <execution>
+            <id>dep-install</id>
+            <phase>${build.taco.init.sdk.stage}</phase>
+            <goals>
+              <goal>exec</goal>
+            </goals>
+            <configuration>
+              <executable>${build.venv.pip.path}</executable>
+              <workingDirectory>${build.venv.directory}</workingDirectory>
+              <arguments>
+                <argument>install</argument>
+                <argument>setuptools</argument>
+                <argument>packaging==25.0</argument>
+                <argument>xmlschema==4.3.1</argument>
+                <argument>defusedxml==0.7.1</argument>
+              </arguments>
+              <environmentVariables>
+                <VIRTUALENV>${build.venv.directory}</VIRTUALENV>
+                
<PATH>${build.venv.directory}${file.separator}bin:${path.separator}${env.PATH}</PATH>
+              </environmentVariables>
+            </configuration>
+          </execution>
+          <execution>
             <id>taco-packager-install</id>
             <phase>${build.taco.init.sdk.stage}</phase>
             <goals>

--
To view, visit 
https://asterix-gerrit.ics.uci.edu/c/asterixdb-clients/+/20830?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://asterix-gerrit.ics.uci.edu/settings?usp=email

Gerrit-MessageType: merged
Gerrit-Project: asterixdb-clients
Gerrit-Branch: master
Gerrit-Change-Id: I5b9b1f4a8ecd82d75a61b8e1ea056b7fd13ac99a
Gerrit-Change-Number: 20830
Gerrit-PatchSet: 2
Gerrit-Owner: Ian Maxon <[email protected]>
Gerrit-Reviewer: Ian Maxon <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Peeyush Gupta <[email protected]>

Reply via email to