Warnings when creating embedded Menus with Instantiator

[anda_skoa]

I guess the only way to fix this is in Menu.qml

That line needs to be something like

property var __parentContentItem: __parentMenu ? __parentMenu.__contentItem : null

property var __parentContentItem: __parentMenu ? __parentMenu.__contentItem : null

To copy to clipboard, switch view to plain text mode 

Cheers,
_

[ribtoks]

Should I then send my edits to Qt repo? I mean, officially I would patch Qt, so I would need to ship modified Qt and all that stuff with licenses, you know..

[anda_skoa]

If you can verify that this fixes the problem, it would be great is this would end up in Qt proper.
First step for that is to create a ticket at https://bugreports.qt.io/secure/Dashboard.jspa ideally with a test case that shows the problem.

For you locally it depends on the license you are using Qt with.

The commerical license doesn't require you to ship any source, the open source license already require you to make the sources of Qt available to whoever you distribute the software to.

Cheers,
_

[ribtoks]

The OpenSource License doesn't require me to ship Qt if I didn't modify it.
Yes, I'll create a bug. Thank you.

[anda_skoa]

The OpenSource License doesn't require me to ship Qt if I didn't modify it.

Right, you can also offer a download of the Qt code.
In case of a modifcation you would need to offer download of your modifed source instead of just having the official Qt sources mirrored.

Cheers,
_