Skip to content

CustomPlugin: Fix QApplicationStatic for Qt 6.6.3#12958

Merged
DonLakeFlyer merged 1 commit into
mavlink:masterfrom
Ventor-Innovations:bugfix/fix-custom-plugin-for-qt-6.6.3
Jun 6, 2025
Merged

CustomPlugin: Fix QApplicationStatic for Qt 6.6.3#12958
DonLakeFlyer merged 1 commit into
mavlink:masterfrom
Ventor-Innovations:bugfix/fix-custom-plugin-for-qt-6.6.3

Conversation

@rubenp02

@rubenp02 rubenp02 commented Jun 3, 2025

Copy link
Copy Markdown
Contributor

CustomPlugin: Fix QApplicationStatic for Qt 6.6.3

Description

Updated the example implementation of the CustomPlugin to include the QtCore/QApplicationStatic header only for Qt versions over 6.8.0, where it was introduced.

Checklist:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Updated the example implementation of the CustomPlugin to include the
QtCore/QApplicationStatic header only for Qt versions over 6.8.0, where
it was introduced.
@rubenp02

rubenp02 commented Jun 3, 2025

Copy link
Copy Markdown
Contributor Author

This was breaking custom builds on Qt 6.6.3.

@HTRamsey

HTRamsey commented Jun 3, 2025

Copy link
Copy Markdown
Member

Oh I think it exists but you have to include via the actual header qapplicationstatic.h

@rubenp02

rubenp02 commented Jun 3, 2025

Copy link
Copy Markdown
Contributor Author

I did know that but it appeared to be included indirectly so I just left it that way. Happy to explicitly include it if needed.

@DonLakeFlyer DonLakeFlyer merged commit 0908ee5 into mavlink:master Jun 6, 2025
2 checks passed
@rubenp02 rubenp02 deleted the bugfix/fix-custom-plugin-for-qt-6.6.3 branch June 6, 2025 20:04
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.

3 participants