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

Add toltec hooks #39

Merged
merged 28 commits into from
Jan 3, 2024
Merged

Add toltec hooks #39

merged 28 commits into from
Jan 3, 2024

Conversation

Eeems
Copy link
Member

@Eeems Eeems commented Dec 15, 2023

  • Fixed c-quoted string handling with escaped unicode characters
  • Added reload_oxide_apps hook
  • Added patch_rm2fb hook
  • Added install_lib hook
  • Load included hooks automatically
  • Change standalone version to use pyinstaller so that user hooks will work
  • Build standalone with python 3.12
  • Test python 3.11 and 3.12
  • Add build for pip packages
  • Add logic to upload pip packages to pypi
  • Update version number to 0.3.0
  • Update github release with build artifacts automatically

@Eeems Eeems added the enhancement New feature or request label Dec 15, 2023
@Eeems Eeems changed the title Add app reload hook Add toltec hooks Dec 15, 2023
This was linked to issues Dec 15, 2023
@Eeems Eeems marked this pull request as ready for review December 16, 2023 03:39
@Eeems Eeems marked this pull request as draft December 16, 2023 03:41
@Eeems Eeems marked this pull request as ready for review December 16, 2023 04:47
@Eeems
Copy link
Member Author

Eeems commented Dec 27, 2023

One concern, I just realized, is that custom hooks will not have the full stdlib available due to how pyinstaller optimizes things. I've solved this before by generating a stdlib zip file to include that is automatically added to the path. I think for now we'll just leave it out until there is a need.

@Eeems
Copy link
Member Author

Eeems commented Dec 27, 2023

I'll leave this open for comment until the new year, and then I'll merge and release.

@Eeems Eeems merged commit fe4ee54 into main Jan 3, 2024
11 checks passed
@Eeems Eeems deleted the hooks branch January 3, 2024 02:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add hooks for all toltec flags/hooks install-lib missing
1 participant