Skip to content

Introduce build_support#4

Open
crystall1nedev wants to merge 4 commits into
ThatStella7922:masterfrom
crystall1nedev:build_support
Open

Introduce build_support#4
crystall1nedev wants to merge 4 commits into
ThatStella7922:masterfrom
crystall1nedev:build_support

Conversation

@crystall1nedev
Copy link
Copy Markdown

@crystall1nedev crystall1nedev commented Jun 2, 2026

These are the "other changes". Fixes universal2 builds of #1.

Additionally creates a backbone for future refactoring or modularizing of PyWemoGUI's build.py.

TODO for merge:

  • Add option to disable downloading/creation of universal2 wheels.
  • Fall back to current architecture if universal2 is disabled.
  • Move macOS handling out of build.py to build_support/macosx.py

NOTE: This will be my first and final PR to this repository on GitHub. As described elsewhere and discussed (physically) with you, future work from me on PyWemoGUI will be @ The Enclave.

This is a big commit.
- Work around universal2 issues by downloading or creating universal2 .whls at build time
  - Currently no switch to turn this off, I'll do it soon.
- Re-enable universal2 in the spec (even though it's unused currently)
- Backbone for future build.py refactoring by Stella since build_support exists now?
- Pass --universal2 to build a Universal binary for Intel-based Macs and Macs with Apple silicon.
- Pass --skipmerge to skip the built-in preparation step for universal2. Useful if you don't
  want to repeatedly download the same packages or spend cycles remerging them.
Makes things a bit cleaner for the main build.py script.

- {os}_prep to be used for any pre-build preparations needing to be done
- {os}_build to be used for any building actions and checking
- main to be stubbed to tell the user not to directly call them

WINDOWS REFACTOR CURRENTLY UNTESTED
@crystall1nedev
Copy link
Copy Markdown
Author

All tasks implemented. Making sure the Windows refactor actually works before giving the green.

@crystall1nedev
Copy link
Copy Markdown
Author

Good to merge, tested it.

@ThatStella7922
Copy link
Copy Markdown
Owner

you smell

@crystall1nedev
Copy link
Copy Markdown
Author

i mean probably idk

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