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

Update contruct to v2.10.57 (bugfix) #957

Merged
merged 3 commits into from
Jan 22, 2024
Merged

Update contruct to v2.10.57 (bugfix) #957

merged 3 commits into from
Jan 22, 2024

Conversation

Hook25
Copy link
Collaborator

@Hook25 Hook25 commented Jan 22, 2024

Description

The vendorized version of contruct uses the now removed (as of python3.12) imp module. This PR updates the vendorized module to a more up-to-date version of construct with the following changes:

  • All imports made relative to vendor
  • Fixed the creation of a module to be compatible with python3.5
  • Patched (keeping the old version) py3compact.py to keep the supportshalffloats among other functions (used in python3.5 that doesn't support the struct e formatter)

Resolved issues

Partially solves: https://launchpadlibrarian.net/710488943/buildlog_ubuntu-noble-amd64.checkbox-support_3.3.0~dev96~ubuntu24.04.1_BUILDING.txt.gz

Documentation

This PR documentation and the commit messages should be enough to document what was done

Tests

N/A

- Modify the module get mechanism to be backward compatible with python3.4
- Modify the size calculator to be compatible with python3.4 ('e' not
    supported)
Minor: whitespaces
Copy link

codecov bot commented Jan 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (9fbbf7b) 39.18% compared to head (0e0cc8e) 39.18%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #957   +/-   ##
=======================================
  Coverage   39.18%   39.18%           
=======================================
  Files         315      315           
  Lines       34899    34899           
  Branches     5972     5972           
=======================================
  Hits        13676    13676           
  Misses      20610    20610           
  Partials      613      613           
Flag Coverage Δ
checkbox-support 50.06% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@kissiel kissiel left a comment

Choose a reason for hiding this comment

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

makes sense; looks good; +1

@kissiel kissiel merged commit 6ccc8cd into main Jan 22, 2024
12 checks passed
@kissiel kissiel deleted the update_contruct_py312 branch January 22, 2024 18:07
LiaoU3 pushed a commit to LiaoU3/checkbox that referenced this pull request Mar 20, 2024
* Updated to 2.10.57

- Modify the module get mechanism to be backward compatible with python3.4
- Modify the size calculator to be compatible with python3.4 ('e' not
    supported)

* Updated to 2.10.57

Minor: whitespaces

* Updated to 2.10.57
binli pushed a commit to binli/checkbox that referenced this pull request Mar 22, 2024
* Updated to 2.10.57

- Modify the module get mechanism to be backward compatible with python3.4
- Modify the size calculator to be compatible with python3.4 ('e' not
    supported)

* Updated to 2.10.57

Minor: whitespaces

* Updated to 2.10.57
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