Merge pull request #331 from compas-dev/no-binary-bundling

Do not bundle V-REP's remoteApi binaries

Author: gonzalocasas

.github/workflows/integration.yml

@@ -30,6 +30,8 @@ jobs:
       - name: Install
         run: |
           python -m pip install --no-cache-dir -r requirements-dev.txt
+          # Get V-REP remote API for integration testing
+          curl --output $(python -c "import os;import compas_fab.backends.vrep.remote_api as v; print(os.path.dirname(v.__file__))")/remoteApi.so https://wolke.ethz.ch/s/idbWPRKzJ8pFkAn/download/remoteApi.so
       - name: Run integration tests
         run: |
           pytest --doctest-modules --ignore-glob="**ghpython/components/**.py"

CHANGELOG.rst

@@ -20,6 +20,8 @@ Unreleased
 
 **Removed**
 
+* Removed the bundled binary files for the `VrepClient` remote API. To use V-REP, use the `remoteApi` binaries provided with the software.
+
 0.20.0
 ----------
 

docs/examples/conftest.py

@@ -5,7 +5,7 @@
 
 from sybil import Region
 from sybil import Sybil
-from sybil.parsers.codeblock import CodeBlockParser
+from sybil.parsers.codeblock import PythonCodeBlockParser
 from sybil.parsers.doctest import DocTestParser
 
 LITERALPYTHON_START = re.compile(r'\.\.\s*literalinclude\s*::(.*\.py)')
@@ -42,7 +42,7 @@ def parse_literalinclude_python_blocks(document):
 pytest_collect_file = Sybil(
     parsers=[
         DocTestParser(),
-        CodeBlockParser(),
+        PythonCodeBlockParser(),
         parse_literalinclude_python_blocks
     ],
     pattern='*.rst',

src/compas_fab/backends/vrep/remote_api/remoteApi.dll

@@ -133,7 +133,7 @@ def check(ctx):
         lint(ctx)
 
         log.write('Checking MANIFEST.in...')
-        ctx.run('check-manifest --ignore-bad-ideas=remoteApi.so')
+        ctx.run('check-manifest')
 
         log.write('Checking ReStructuredText formatting...')
         ctx.run('python setup.py check --strict --metadata --restructuredtext')