Skip to content

fix: resolve binary tools detection after PyInstaller packaging#317

Open
bob-zebedy wants to merge 2 commits intoRfidResearchGroup:mainfrom
bob-zebedy:fix/pyinstaller-bin-path
Open

fix: resolve binary tools detection after PyInstaller packaging#317
bob-zebedy wants to merge 2 commits intoRfidResearchGroup:mainfrom
bob-zebedy:fix/pyinstaller-bin-path

Conversation

@bob-zebedy
Copy link

This fixes the warning about missing tools after PyInstaller

packaging by correctly detecting the temporary

extraction directory (sys._MEIPASS) in frozen environments.

  • New get_resource_dir() in chameleon_utils.py

  • Import default_cwd in chameleon_cli_unit.py

  • Simplify check_tools() to use imported default_cwd

This fixes the warning about missing tools after PyInstaller

packaging by correctly detecting the temporary

extraction directory (sys._MEIPASS) in frozen environments.

- New get_resource_dir() in chameleon_utils.py

- Import default_cwd in chameleon_cli_unit.py

- Simplify check_tools() to use imported default_cwd
@github-actions
Copy link

github-actions bot commented Nov 1, 2025

You are welcome to add an entry to the CHANGELOG.md as well

@github-actions
Copy link

github-actions bot commented Nov 1, 2025

Built artifacts for commit 301fdb2

Firmware

Client

Copy link
Contributor

@GameTec-live GameTec-live left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@GameTec-live GameTec-live added the testme! This PR is ready but just missing tests. Join in and help testing! label Jan 30, 2026
Copy link
Contributor

@LupusE LupusE left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does not break other components. I have not tested the pyinstaller myself, but looks reasonable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testme! This PR is ready but just missing tests. Join in and help testing!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants