Skip to content

cmake: On macOS fix Qt6 hard coded path - #1534

Merged
billmeek merged 1 commit into
MythTV:masterfrom
jhoyt4:master
Nov 7, 2026
Merged

billmeek merged 1 commit into
MythTV:masterfrom
jhoyt4:master

Conversation

@jhoyt4

@jhoyt4 jhoyt4 commented Nov 7, 2026

Copy link
Copy Markdown
Contributor

Resolves a runtime issue where the bundled QtCore framework looks
in the homebrew or macports install path instead of internally within
the app bundle.

  • Add a qt.conf template to explicitly override Qt's hardcoded prefix path and confine plugin discovery to the application bundle.
  • Update darwin bundling CMake configuration to automatically package qt.conf into the app bundle's Contents/Resources/ directory during deployment.
  • Add QT_PLUGIN_PATH and QT_AUTOLOAD_PLUGINS environmental variables to app bundle startup script. QT_AUTOLOAD_PLUGINS is set to 0 to prevent rogue host plugin discovery at runtime.

Thank you for contributing to MythTV!

Please review the checklist below to ensure that your contribution
to MythTV is ready for review by our developers.

It is helpful to regularly rebase your pull request to ensure changes
apply cleanly to the current MythTV development branch.

Checklist

  Resolves a runtime issue where the bundled QtCore framework looks
  in the homebrew or macports install path instead of internally within
  the app bundle.

  - Add a `qt.conf` template to explicitly override Qt's hardcoded
    prefix path and confine plugin discovery to the application bundle.
  - Update darwin bundling CMake configuration to automatically
    package `qt.conf` into the app bundle's Contents/Resources/
    directory during deployment.
  - Add QT_PLUGIN_PATH and QT_AUTOLOAD_PLUGINS environmental variables
    to app bundle startup script. QT_AUTOLOAD_PLUGINS is set to 0 to
    prevent rogue host plugin discovery at runtime.
@jhoyt4

jhoyt4 commented Nov 7, 2026

Copy link
Copy Markdown
Contributor Author

please cherry-pick onto fixes/36

@billmeek
billmeek merged commit 971d8b6 into MythTV:master Nov 7, 2026
6 of 11 checks passed
@jhoyt4 jhoyt4 mentioned this pull request Nov 7, 2026
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