Skip to content
This repository has been archived by the owner on Jul 8, 2022. It is now read-only.

Improve pythonPath config for Open-Needs #40

Merged
merged 12 commits into from Feb 23, 2022
Merged

Improve pythonPath config for Open-Needs #40

merged 12 commits into from Feb 23, 2022

Conversation

haiyangToAI
Copy link
Contributor

#38

@danwos
Copy link
Contributor

danwos commented Feb 22, 2022

When I press "ESC" during the question for the Python Path, I get this error:

Activating Open-Needs IDE
vscode_ext/src/extension.ts:235
Extension version: 0.0.14
vscode_ext/src/extension.ts:241
Error
Command failed: SystemDefaultPython --version
/bin/sh: line 1: SystemDefaultPython: command not found

Error: Command failed: SystemDefaultPython --version
/bin/sh: line 1: SystemDefaultPython: command not found

	at ChildProcess.exithandler (child_process.js:317:12)
	at ChildProcess.emit (events.js:315:20)
	at maybeClose (internal/child_process.js:1048:16)
	at Socket.<anonymous> (internal/child_process.js:439:11)
	at Socket.emit (events.js:315:20)
	at Pipe.<anonymous> (net.js:673:12)
	at Pipe.callbackTrampoline (internal/async_hooks.js:131:14)

/bin/sh: line 1: SystemDefaultPython: command not found

vscode_ext/src/extension.ts:43
Extension Open-Needs:  Python path not configured yet!
vscode_ext/src/extension.ts:57
Extension Open-Needs:  Python path configured!
vscode_ext/src/extension.ts:85

@danwos
Copy link
Contributor

danwos commented Feb 22, 2022

But even if aborted, a Python venv gets used:

Running cmd: python -c "import sys; print(sys.executable)"
Successful exec
Running cmd: /home/daniel/workspace/open-needs/open-needs-ide/.venv/bin/python --version
Successful exec
Python path: /home/daniel/workspace/open-needs/open-needs-ide/.venv/bin/python
Running cmd: /home/daniel/workspace/open-needs/open-needs-ide/.venv/bin/python -c "from needls.version import __version__; print(__version__)"
Successful exec
Saved text doc of type: jsonc

But that's the wrong one. It the one from that main open-needs-ide project and not the one from our test-env project.

@danwos danwos merged commit cb04c4e into main Feb 23, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants