Skip to content

Commit 539edce

Browse files
authored
Fix gltf (#1213)
* Only run gltf example * Remove superfluous prints * Raise on stderr * Fix path expansion * Final fix for create_sites_for_python_operators.ps1 * Revert "Remove superfluous prints" This reverts commit fb81e4a. * Revert "Only run gltf example" This reverts commit df37386.
1 parent 14db394 commit 539edce

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

docs/source/user_guide/create_sites_for_python_operators.ps1

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ Write-Host "-pythonexe" $pythonexe
66
Write-Host "-tempfolder" $tempfolder
77

88
# Expand shortcut in path
9-
$tempfolder = Resolve-Path -Path $tempfolder
9+
$tempfolder = Resolve-Path -Path $tempfolder -Relative
1010

1111
Write-Host "make venv"
1212
Start-Process $pythonexe -ArgumentList ("-m venv "+ $tempfolder+"\venv") -NoNewWindow -Wait

examples/08-python-operators/02-python_operators_with_dependencies.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -149,7 +149,7 @@
149149
import subprocess
150150

151151
process = subprocess.run(args, stdout=subprocess.PIPE, stderr=subprocess.PIPE)
152-
if process.returncode != 0 and process.stderr:
152+
if process.stderr:
153153
raise RuntimeError(
154154
"Installing pygltf in a virtual environment failed with error:\n"
155155
+ f"return code = {process.returncode}\n"

0 commit comments

Comments
 (0)