Skip to content

use the current venv to launch therock build#4

Merged
lamikr merged 1 commit into
mainfrom
users/lamikr/wip/therock_build_fix_from_theconfig
Oct 25, 2025
Merged

use the current venv to launch therock build#4
lamikr merged 1 commit into
mainfrom
users/lamikr/wip/therock_build_fix_from_theconfig

Conversation

@lamikr
Copy link
Copy Markdown
Collaborator

@lamikr lamikr commented Oct 25, 2025

  • use the currently used python venv to launch the therock build from the rockbuilder_cfg.py
  • This ensures that rockbuilder_cfg has all the python packages installed that the init_rcb_env.sh/bat installs.
  • This fixes a following error detetected on windows where the build tried to use different python env which did not have the windows-curses installed.
Traceback (most recent call last):
File "c:\rock\rockbuilder\rockbuilder.py", line 11
in <module> import rockbuilder_cfg as rcb_cfg_writer File "c:\rock\rockbuilder\rockbuilder_cfg.py", 
line 7, in <module> import curses File "C:\Python313\Lib\curses\__init__.py", line 13, in 
<module> from _curses import * ModuleNotFoundError: No module named '_curses'

- use the currently used python venv to launch
  the therock build from the rockbuilder_cfg.py
- This ensures that rockbuilder_cfg has all the
  python packages installed that the
  init_rcb_env.sh/bat installs.
- This fixes a following error detetected on windows
  where the build tried to use different python env
  which did not have the windows-curses installed.

Traceback (most recent call last):
  File "c:\rock\rockbuilder\rockbuilder.py", line 11, in <module>
    import rockbuilder_cfg as rcb_cfg_writer
  File "c:\rock\rockbuilder\rockbuilder_cfg.py", line 7, in <module>
    import curses
  File "C:\Python313\Lib\curses\__init__.py", line 13, in <module>
    from _curses import *
ModuleNotFoundError: No module named '_curses'

Signed-off-by: Mika Laitio <mika.laitio@amd.com>
@lamikr lamikr merged commit f3b6413 into main Oct 25, 2025
1 check passed
@lamikr lamikr deleted the users/lamikr/wip/therock_build_fix_from_theconfig branch October 25, 2025 07:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant