Skip to content

Conversation

@0xFEEDC0DE64
Copy link

Fixes my issue #67

@jbagg
Copy link
Owner

jbagg commented Jun 28, 2024

A Qt developer proposed a similar change in this merge request. Qt version policy says that "Minor releases are backwards binary and source compatible." Either the changed line has used the Qt api in an abusive way or Qt has broken their policy by changing the return type of QAndroidApplication::context()

jbagg added a commit that referenced this pull request Jun 28, 2024
QNativeInterface does not provide source compatibility.  context() return
type changed.

https://bugreports.qt.io/browse/QTBUG-123900

Solution contributed by https://github.com/0xFEEDC0DE64 in
#68
@jbagg
Copy link
Owner

jbagg commented Jun 28, 2024

I ended up doing my own commit as I added a comment and fixed a white space issue (tabs vs spaces). I put a note in the commit that it was contributed by this MR .

@jbagg jbagg closed this Jun 28, 2024
t-mon pushed a commit to nymea/QtZeroConf that referenced this pull request Oct 17, 2025
QNativeInterface does not provide source compatibility.  context() return
type changed.

https://bugreports.qt.io/browse/QTBUG-123900

Solution contributed by https://github.com/0xFEEDC0DE64 in
jbagg#68
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