Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UnboundLocalError: cannot access local variable 'exclude_names' where it is not associated with a value #2362

Closed
gbtami opened this issue Apr 25, 2024 · 7 comments · Fixed by #2363

Comments

@gbtami
Copy link

gbtami commented Apr 25, 2024

Follow up to #2356
After the new MSYS2 cx-Freeze package arrived we triggered a new build for pychess bdist_msi
https://github.com/pychess/pychess/actions/runs/8837952115/job/24269587548

@marcelotduarte
Copy link
Owner

#2363 (comment)

@gbtami
Copy link
Author

gbtami commented Apr 27, 2024

Seems it produces the same error with 7.0.0-3
https://github.com/pychess/pychess/actions/runs/8855111639/job/24329191820

@marcelotduarte
Copy link
Owner

I'm sorry, I didn't patch msys2 correctly. But I'm also working on tests for the msys2 version, so it avoids these errors. I ended up discovering other errors with this #2367.
I'm waiting for the pytest-datafiles package to become available and then I'll send a new patch.

@hartwork
Copy link

I'm sorry, I didn't patch msys2 correctly. But I'm also working on tests for the msys2 version, so it avoids these errors. I ended up discovering other errors with this #2367.

@marcelotduarte thanks for your help!

I'm waiting for the pytest-datafiles package to become available and then I'll send a new patch.

For anyone else wondering how or when pytest-datafiles will be available:

@marcelotduarte
Copy link
Owner

msys2/MINGW-packages#20760 merged.

@hartwork
Copy link

@marcelotduarte thank you!

@marcelotduarte
Copy link
Owner

You can test the patch in the latest development build (7.1.0.dev9):
pip install --force --no-cache --pre --extra-index-url https://marcelotduarte.github.io/packages/ cx_Freeze
For conda-forge the command is:
conda install -y --no-channel-priority -S -c https://marcelotduarte.github.io/packages/conda cx_Freeze
For msys2, download the file at: https://marcelotduarte.github.io/packages/msys2/ and use it w/ pacman -U

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 a pull request may close this issue.

3 participants