Skip to content

fix: Resolve version manually #143

New issue

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

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

Already on GitHub? # to your account

Merged
merged 3 commits into from
Jul 13, 2024

Conversation

caffeine-addictt
Copy link
Member

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Breaking Change
  • Documentation Update

Description

Compiling python with thread included do not play nice with resolving version with importlib.

Related Tickets & Documents

QA Instructions, Screenshots, Recordings

Please replace this line with instructions on how to test your changes, a note
on the devices and browsers this has been tested on, as well as any relevant
images for UI changes.

Added/updated tests?

We encourage you to keep the code coverage percentage at 80% and above.

  • Yes
  • No, and this is why: Unneeded
  • I need help with writing tests

[optional] Are there any post deployment tasks we need to perform?

Checklist

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have incremented the version according to semantic versioning.
  • I have updated the changelog accordingly.

caffeine-addictt and others added 3 commits July 14, 2024 00:46
* exceptiongroup 1.2.1 -> 1.2.2
* importlib-metadata 7.1.0 -> 8.0.0
* packaging 24.0 -> 24.1
* zipp 3.19.1 -> 3.19.2

---

Signed-off-by: AlexNg <contact@ngjx.org>
This fixes #141 where compiling python with nuitka with --standalone
flag causes runtime errors with importlib.metadata.

This patch also removes the now unneeded importlib-metadata dependency

Thanks to Abdullah Albanna <albannaa78@gmail.com> who raised this issue
and helped out in debugging.

---

Fixes #141
Co-authored-by: Abdullah Albanna <albannaa78@gmail.com>
Signed-off-by: AlexNg <contact@ngjx.org>
@caffeine-addictt caffeine-addictt added the Type: Bug Something isn't working as expected. label Jul 13, 2024
@caffeine-addictt caffeine-addictt self-assigned this Jul 13, 2024
@github-actions github-actions bot added the Documentation An issue/change with the documentation. label Jul 13, 2024
@caffeine-addictt caffeine-addictt marked this pull request as ready for review July 13, 2024 17:03
@caffeine-addictt caffeine-addictt merged commit 4608b55 into main Jul 13, 2024
41 checks passed
@caffeine-addictt caffeine-addictt deleted the fix/compiling-python-dont-like-importlib branch July 13, 2024 17:08
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Documentation An issue/change with the documentation. Type: Bug Something isn't working as expected.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

importlib.metadata.PackageNotFoundError Nuitka
1 participant