Skip to content

Conversation

Copy link

Copilot AI commented Jan 3, 2026

The Windows executable build workflow failed because wallet.py used an absolute import (from trinity_wallet_py.gui.main_window) that broke when PyInstaller ran from within the trinity_wallet_py directory.

Changes

  • wallet.py: Changed to relative import from gui.main_window
  • Updated sys.path to add current directory instead of parent directory
# Before
sys.path.insert(0, os.path.dirname(os.path.dirname(os.path.abspath(__file__))))
from trinity_wallet_py.gui.main_window import TrinityWalletGUI

# After  
sys.path.insert(0, os.path.dirname(os.path.abspath(__file__)))
from gui.main_window import TrinityWalletGUI

The build script executes PyInstaller from within trinity_wallet_py/, so imports must be relative to that directory. All other modules already use correct relative imports (..core.wallet, ..utils.base58, etc.).

Original prompt

Reference: https://github.com/Action-Committee/Trinity_py/actions/runs/20665461000/job/59337105598 I want make this work


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI self-assigned this Jan 3, 2026
Co-authored-by: 5mil <4460242+5mil@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix Action Committee workflow issues Fix PyInstaller build failure due to incorrect import path in wallet.py Jan 3, 2026
Copilot AI requested a review from 5mil January 3, 2026 00:50
Copilot AI and others added 3 commits January 3, 2026 23:31
Co-authored-by: 5mil <4460242+5mil@users.noreply.github.com>
…tion-issue

Fix MSYS2 BerkeleyDB package name in Windows build workflow
@5mil 5mil marked this pull request as ready for review January 13, 2026 11:01
@5mil 5mil merged commit 9e030ad into copilot/implement-blockchain-storage-solution Jan 13, 2026
1 check failed
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.

2 participants