emqx / qmqtt

MQTT client for Qt
https://www.emqx.com
Other
698 stars 337 forks source link

Changes to be able to compile the library with Qt 6.2.0. #231

Closed alex-spataru closed 2 years ago

alex-spataru commented 3 years ago

Trying to compile the library with Qt 6 results with the following error:

Screen Shot 2021-09-28 at 14 51 08

The fix that I came up with was simply changing the QString in the Q_PROPERTY to an int or a ConnectionState type.

alex-spataru commented 3 years ago

Please let me know if I should close this PR, for the moment I am using my fork for Qt 6 applications.

mwallnoefer commented 2 years ago

@ejvr is this fine to merge in?

ejvr commented 2 years ago

Yep, I see no problems

mwallnoefer commented 2 years ago

Unfortunately I had to revert this, it breaks the build on Qt5 (also when disabling the new definition MQTT_INCLUDE_MOC). Very strange...

See PR #237.