You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tinkerpop.apache.org by sp...@apache.org on 2016/08/30 16:58:11 UTC

[1/5] tinkerpop git commit: Use pytest-runner to properly build deps on test invocation

Repository: tinkerpop
Updated Branches:
  refs/heads/virtualenv [created] 24eba0ca4


Use pytest-runner to properly build deps on test invocation


Project: http://git-wip-us.apache.org/repos/asf/tinkerpop/repo
Commit: http://git-wip-us.apache.org/repos/asf/tinkerpop/commit/56737a13
Tree: http://git-wip-us.apache.org/repos/asf/tinkerpop/tree/56737a13
Diff: http://git-wip-us.apache.org/repos/asf/tinkerpop/diff/56737a13

Branch: refs/heads/virtualenv
Commit: 56737a134fc08e56848eed8adc61201bdd0802df
Parents: 0462430
Author: davebshow <da...@gmail.com>
Authored: Mon Aug 29 15:27:49 2016 -0400
Committer: davebshow <da...@gmail.com>
Committed: Mon Aug 29 15:27:49 2016 -0400

----------------------------------------------------------------------
 gremlin-python/pom.xml                   | 7 +++----
 gremlin-python/src/main/jython/setup.cfg | 7 +++++--
 gremlin-python/src/main/jython/setup.py  | 8 +++++---
 3 files changed, 13 insertions(+), 9 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/56737a13/gremlin-python/pom.xml
----------------------------------------------------------------------
diff --git a/gremlin-python/pom.xml b/gremlin-python/pom.xml
index c830790..deaf17e 100644
--- a/gremlin-python/pom.xml
+++ b/gremlin-python/pom.xml
@@ -323,9 +323,8 @@
                                 <configuration>
                                     <outputDirectory>${project.build.testOutputDirectory}</outputDirectory>
                                     <libraries>
-                                        <param>aenum</param>
-                                        <param>requests</param>
-                                        <param>tornado</param>
+                                        <param>aenum==1.4.5</param>
+                                        <param>tornado==4.4.1</param>
                                     </libraries>
                                 </configuration>
                             </execution>
@@ -551,4 +550,4 @@ log.info("Gremlin Server shutdown")
             </build>
         </profile>
     </profiles>
-</project>
\ No newline at end of file
+</project>

http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/56737a13/gremlin-python/src/main/jython/setup.cfg
----------------------------------------------------------------------
diff --git a/gremlin-python/src/main/jython/setup.cfg b/gremlin-python/src/main/jython/setup.cfg
index f8bab76..d025b3a 100644
--- a/gremlin-python/src/main/jython/setup.cfg
+++ b/gremlin-python/src/main/jython/setup.cfg
@@ -17,5 +17,8 @@
 [bdist_wheel]
 universal=1
 
-[pytest]
-addopts = --junitxml=../python-reports/TEST-native-python.xml
\ No newline at end of file
+[aliases]
+test=pytest
+
+[tool:pytest]
+addopts = --junitxml=../python-reports/TEST-native-python.xml

http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/56737a13/gremlin-python/src/main/jython/setup.py
----------------------------------------------------------------------
diff --git a/gremlin-python/src/main/jython/setup.py b/gremlin-python/src/main/jython/setup.py
index 52e6fe4..7540654 100644
--- a/gremlin-python/src/main/jython/setup.py
+++ b/gremlin-python/src/main/jython/setup.py
@@ -64,12 +64,14 @@ setup(
     long_description=open("README").read(),
     test_suite="tests",
     cmdclass = {'test': PyTest},
+    setup_requires=[
+        'pytest-runner',
+    ],
     tests_require=[
         'pytest'
     ],
     install_requires=[
-        'aenum',
-        'requests',
-        'tornado'
+        'aenum==1.4.5',
+        'tornado==4.4.1'
     ]
 )


[5/5] tinkerpop git commit: Massive cleanup to gremlin-python pom.xml

Posted by sp...@apache.org.
Massive cleanup to gremlin-python pom.xml

Restructured to use virtualenv in all native python calls.


Project: http://git-wip-us.apache.org/repos/asf/tinkerpop/repo
Commit: http://git-wip-us.apache.org/repos/asf/tinkerpop/commit/24eba0ca
Tree: http://git-wip-us.apache.org/repos/asf/tinkerpop/tree/24eba0ca
Diff: http://git-wip-us.apache.org/repos/asf/tinkerpop/diff/24eba0ca

Branch: refs/heads/virtualenv
Commit: 24eba0ca4d4e767a83ddc23a6b0c22a0504dc985
Parents: 334470f
Author: Stephen Mallette <sp...@genoprime.com>
Authored: Tue Aug 30 12:57:21 2016 -0400
Committer: Stephen Mallette <sp...@genoprime.com>
Committed: Tue Aug 30 12:57:21 2016 -0400

----------------------------------------------------------------------
 gremlin-python/pom.xml | 189 ++++++++++++++++++++++----------------------
 1 file changed, 93 insertions(+), 96 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/24eba0ca/gremlin-python/pom.xml
----------------------------------------------------------------------
diff --git a/gremlin-python/pom.xml b/gremlin-python/pom.xml
index 30bba03..bc41c57 100644
--- a/gremlin-python/pom.xml
+++ b/gremlin-python/pom.xml
@@ -318,49 +318,7 @@
                             </systemPropertyVariables>
                         </configuration>
                     </plugin>
-                    <!-- get a clean copy of the python files for native processing -->
-                    <plugin>
-                        <artifactId>maven-resources-plugin</artifactId>
-                        <executions>
-                            <execution>
-                                <id>copy-py-files-to-target</id>
-                                <phase>process-resources</phase>
-                                <goals>
-                                    <goal>copy-resources</goal>
-                                </goals>
-                                <configuration>
-                                    <outputDirectory>${basedir}/target/python</outputDirectory>
-                                    <resources>
-                                        <resource>
-                                            <directory>${basedir}/src/main/jython</directory>
-                                            <filtering>true</filtering>
-                                        </resource>
-                                    </resources>
-                                </configuration>
-                            </execution>
-                        </executions>
-                    </plugin>
-                    <plugin>
-                        <groupId>org.codehaus.mojo</groupId>
-                        <artifactId>exec-maven-plugin</artifactId>
-                        <version>1.2.1</version>
-                        <executions>
-                            <execution>
-                                <id>build-python</id>
-                                <phase>generate-test-resources</phase>
-                                <goals>
-                                    <goal>exec</goal>
-                                </goals>
-                                <configuration>
-                                    <executable>python</executable>
-                                    <workingDirectory>${basedir}/target/python</workingDirectory>
-                                    <commandlineArgs>setup.py build --build-lib ${project.build.testOutputDirectory}/Lib
-                                    </commandlineArgs>
-                                </configuration>
-                            </execution>
-                        </executions>
-                    </plugin>
-                    <!-- required for the jython tests -->
+                    <!-- required for the jython tests  -->
                     <plugin>
                         <groupId>net.sf.mavenjython</groupId>
                         <artifactId>jython-compile-maven-plugin</artifactId>
@@ -395,7 +353,7 @@
                         <executions>
                             <execution>
                                 <id>create-python-reports-directory</id>
-                                <phase>process-resources</phase>
+                                <phase>initialize</phase>
                                 <goals>
                                     <goal>run</goal>
                                 </goals>
@@ -403,14 +361,101 @@
                                     <tasks>
                                         <delete dir="${build.directory}/python-reports"/>
                                         <mkdir dir="${build.directory}/python-reports"/>
+                                        <delete dir="${build.directory}/python"/>
+                                        <mkdir dir="${project.build.directory}/python/env"/>
+                                        <delete dir="${build.directory}/python-packaged"/>
+                                        <mkdir dir="${project.build.directory}/python-packaged/env"/>
                                     </tasks>
                                 </configuration>
                             </execution>
+                            <!-- copy files in jython directory to target/py and run virtual env to sandbox python.
+                                 there is no need to "activate" the virtualenv because all calls to python occur
+                                 directly from bin/ -->
+                            <execution>
+                                <id>setup-py-env</id>
+                                <phase>process-resources</phase>
+                                <goals>
+                                    <goal>run</goal>
+                                </goals>
+                                <configuration>
+                                    <tasks>
+                                        <!-- seems like we need a few different copies of the same source. all the
+                                             different python/jython stuff doesn't seem to want to share. we use
+                                             /python for basic tests and stuff, /jython for jython compilation with
+                                             the jython plugin, /python-packaged for distribution tasks. some of the
+                                             problem seems to stem from the python lifecycle not binding perfectly well
+                                             to the maven lifecycle (integration tests seems to cause troubles
+                                             specifically) -->
+                                        <copy todir="${project.build.directory}/python">
+                                            <fileset dir="src/main/jython"/>
+                                        </copy>
+                                        <copy todir="${project.build.directory}/python-packaged">
+                                            <fileset dir="src/main/jython"/>
+                                        </copy>
+                                        <exec dir="${project.build.directory}/python" executable="virtualenv"
+                                              failonerror="true">
+                                            <arg value="env"/>
+                                        </exec>
+                                        <exec dir="${project.build.directory}/python-packaged" executable="virtualenv"
+                                              failonerror="true">
+                                            <arg value="env"/>
+                                        </exec>
+                                        <copy todir="${project.build.directory}/jython">
+                                            <fileset dir="src/main/jython"/>
+                                        </copy>
+                                    </tasks>
+                                </configuration>
+                            </execution>
+                            <execution>
+                                <id>native-python-build</id>
+                                <phase>generate-test-resources</phase>
+                                <goals>
+                                    <goal>run</goal>
+                                </goals>
+                                <configuration>
+                                    <skip>${skipTests}</skip>
+                                    <target>
+                                        <exec executable="env/bin/python" dir="${project.build.directory}/python"
+                                              failonerror="true">
+                                            <arg line="setup.py build --build-lib ${project.build.testOutputDirectory}/Lib"/>
+                                        </exec>
+                                    </target>
+                                </configuration>
+                            </execution>
+
+                            <!--
+                            build/package python source distribution and wheel archive. the version is bound to an
+                            environment variable that gets used in setup.py to dynamically construct a module
+                            __version__file
+                             -->
+                            <execution>
+                                <id>package-py</id>
+                                <phase>package</phase>
+                                <goals>
+                                    <goal>run</goal>
+                                </goals>
+                                <configuration>
+                                    <target>
+                                        <exec executable="env/bin/python" dir="${project.build.directory}/python-packaged"
+                                              failonerror="true">
+                                            <env key="VERSION" value="${project.version}"/>
+                                            <arg line="setup.py sdist"/>
+                                        </exec>
+                                        <exec executable="env/bin/python" dir="${project.build.directory}/python-packaged"
+                                              failonerror="true">
+                                            <env key="VERSION" value="${project.version}"/>
+                                            <arg line="setup.py bdist_wheel"/>
+                                        </exec>
+                                    </target>
+                                </configuration>
+                            </execution>
+
                             <!--
                             use pytest to execute native python tests - output of xunit output is configured in setup.cfg.
                             had to use the ant plugin - maven-exec-plugin
                             -->
                             <execution>
+                                <id>native-python-test</id>
                                 <phase>integration-test</phase>
                                 <goals>
                                     <goal>run</goal>
@@ -418,11 +463,9 @@
                                 <configuration>
                                     <skip>${skipTests}</skip>
                                     <target>
-                                        <exec executable="python"
-                                              dir="${basedir}/target/python"
+                                        <exec executable="env/bin/python" dir="${project.build.directory}/python"
                                               failonerror="true">
-                                            <arg value="setup.py"/>
-                                            <arg value="test"/>
+                                            <arg line="setup.py test"/>
                                         </exec>
                                     </target>
                                 </configuration>
@@ -536,52 +579,6 @@ log.info("Gremlin Server shutdown")
                         <groupId>org.apache.maven.plugins</groupId>
                         <artifactId>maven-antrun-plugin</artifactId>
                         <executions>
-                            <!-- copy files in jython directory to target/py and run virtual env to sandbox python -->
-                            <execution>
-                                <id>setup-py-env</id>
-                                <phase>generate-resources</phase>
-                                <goals>
-                                    <goal>run</goal>
-                                </goals>
-                                <configuration>
-                                    <tasks>
-                                        <mkdir dir="${project.build.directory}/py/env"/>
-                                        <copy todir="${project.build.directory}/py">
-                                            <fileset dir="src/main/jython"/>
-                                        </copy>
-                                        <exec dir="${project.build.directory}/py" executable="virtualenv"
-                                              failonerror="true">
-                                            <arg line="env"/>
-                                        </exec>
-                                    </tasks>
-                                </configuration>
-                            </execution>
-                            <!--
-                            build/package python source distribution and wheel archive. the version is bound to an
-                            environment variable that gets used in setup.py to dynamically construct a module
-                            __version__file
-                             -->
-                            <execution>
-                                <id>package-py</id>
-                                <phase>package</phase>
-                                <goals>
-                                    <goal>run</goal>
-                                </goals>
-                                <configuration>
-                                    <tasks>
-                                        <exec dir="${project.build.directory}/py" executable="env/bin/python"
-                                              failonerror="true">
-                                            <env key="VERSION" value="${project.version}"/>
-                                            <arg line="setup.py sdist"/>
-                                        </exec>
-                                        <exec dir="${project.build.directory}/py" executable="env/bin/python"
-                                              failonerror="true">
-                                            <env key="VERSION" value="${project.version}"/>
-                                            <arg line="setup.py bdist_wheel"/>
-                                        </exec>
-                                    </tasks>
-                                </configuration>
-                            </execution>
                             <!-- deploy to pypi. assumes that ~/.pypirc is configured appropriately -->
                             <execution>
                                 <id>deploy-py</id>
@@ -591,11 +588,11 @@ log.info("Gremlin Server shutdown")
                                 </goals>
                                 <configuration>
                                     <tasks>
-                                        <exec dir="${project.build.directory}/py" executable="env/bin/pip"
+                                        <exec dir="${project.build.directory}/python-packaged" executable="env/bin/pip"
                                               failonerror="true">
                                             <arg line="install twine"/>
                                         </exec>
-                                        <exec dir="${project.build.directory}/py" executable="env/bin/twine"
+                                        <exec dir="${project.build.directory}/python-packaged" executable="env/bin/twine"
                                               failonerror="true">
                                             <arg line="upload dist/* -r pypitest"/>
                                         </exec>


[3/5] tinkerpop git commit: add lib/lib64 to norecursedirs in pytest config

Posted by sp...@apache.org.
add lib/lib64 to norecursedirs in pytest config


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

Branch: refs/heads/virtualenv
Commit: d1480314f0994e331e0b6dff2fcebdd998837941
Parents: 5c9d992
Author: davebshow <da...@gmail.com>
Authored: Tue Aug 30 00:06:31 2016 -0400
Committer: davebshow <da...@gmail.com>
Committed: Tue Aug 30 00:06:31 2016 -0400

----------------------------------------------------------------------
 gremlin-python/src/main/jython/setup.cfg | 1 +
 1 file changed, 1 insertion(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/d1480314/gremlin-python/src/main/jython/setup.cfg
----------------------------------------------------------------------
diff --git a/gremlin-python/src/main/jython/setup.cfg b/gremlin-python/src/main/jython/setup.cfg
index d025b3a..b9e5f22 100644
--- a/gremlin-python/src/main/jython/setup.cfg
+++ b/gremlin-python/src/main/jython/setup.cfg
@@ -22,3 +22,4 @@ test=pytest
 
 [tool:pytest]
 addopts = --junitxml=../python-reports/TEST-native-python.xml
+norecursedirs = '.*', 'build', 'dist', 'CVS', '_darcs', '{arch}', '*.egg' lib lib64


[2/5] tinkerpop git commit: remove unneeded custom command

Posted by sp...@apache.org.
remove unneeded custom command


Project: http://git-wip-us.apache.org/repos/asf/tinkerpop/repo
Commit: http://git-wip-us.apache.org/repos/asf/tinkerpop/commit/5c9d9927
Tree: http://git-wip-us.apache.org/repos/asf/tinkerpop/tree/5c9d9927
Diff: http://git-wip-us.apache.org/repos/asf/tinkerpop/diff/5c9d9927

Branch: refs/heads/virtualenv
Commit: 5c9d9927d20c7f56b9a0bc9c5c1ae0fe5a58f4bf
Parents: 56737a1
Author: davebshow <da...@gmail.com>
Authored: Mon Aug 29 15:33:51 2016 -0400
Committer: davebshow <da...@gmail.com>
Committed: Mon Aug 29 15:33:51 2016 -0400

----------------------------------------------------------------------
 gremlin-python/src/main/jython/setup.py | 12 ------------
 1 file changed, 12 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/5c9d9927/gremlin-python/src/main/jython/setup.py
----------------------------------------------------------------------
diff --git a/gremlin-python/src/main/jython/setup.py b/gremlin-python/src/main/jython/setup.py
index 7540654..c37191f 100644
--- a/gremlin-python/src/main/jython/setup.py
+++ b/gremlin-python/src/main/jython/setup.py
@@ -43,17 +43,6 @@ import __version__
 
 version = __version__.version
 
-class PyTest(Command):
-    user_options = []
-    def initialize_options(self):
-        pass
-    def finalize_options(self):
-        pass
-    def run(self):
-        import sys,subprocess
-        errno = subprocess.call([sys.executable, 'runtest.py'])
-        raise SystemExit(errno)
-
 setup(
     name='gremlinpython',
     version=version,
@@ -63,7 +52,6 @@ setup(
     description='Gremlin-Python for Apache TinkerPop',
     long_description=open("README").read(),
     test_suite="tests",
-    cmdclass = {'test': PyTest},
     setup_requires=[
         'pytest-runner',
     ],


[4/5] tinkerpop git commit: Merge branch 'pr-393' into virtualenv

Posted by sp...@apache.org.
Merge branch 'pr-393' into virtualenv


Project: http://git-wip-us.apache.org/repos/asf/tinkerpop/repo
Commit: http://git-wip-us.apache.org/repos/asf/tinkerpop/commit/334470ff
Tree: http://git-wip-us.apache.org/repos/asf/tinkerpop/tree/334470ff
Diff: http://git-wip-us.apache.org/repos/asf/tinkerpop/diff/334470ff

Branch: refs/heads/virtualenv
Commit: 334470ffb07cbb9f105d3c287ac183f123d4c3c6
Parents: d2fc54f d148031
Author: Stephen Mallette <sp...@genoprime.com>
Authored: Tue Aug 30 08:29:39 2016 -0400
Committer: Stephen Mallette <sp...@genoprime.com>
Committed: Tue Aug 30 08:29:39 2016 -0400

----------------------------------------------------------------------
 gremlin-python/pom.xml                   |  7 +++----
 gremlin-python/src/main/jython/setup.cfg |  8 ++++++--
 gremlin-python/src/main/jython/setup.py  | 20 +++++---------------
 3 files changed, 14 insertions(+), 21 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/334470ff/gremlin-python/pom.xml
----------------------------------------------------------------------