Augmenting instructions re setting up qml extensions #909
Conversation
I tried both `;` and `:` as path separators for `QML2_IMPORT_PATH` and had no luck. Change above was the first method that got everything enabled.
INSTALL.md
Outdated
| @@ -46,6 +46,24 @@ Additional Qt plugins can be built to extend Meshroom UI features. They can be f | |||
| though they might get better integration in the future. | |||
| Note that they are optional but highly recommended. | |||
|
|
|||
| For Windows, there may be problems setting `QML2_IMPORT_PATH` to contain multiple paths. The easiest approach may be to duplicate what is done for the binary release. This has a directory structure like this: | |||
There was a problem hiding this comment.
I would like to avoid "may be" in the wording of the documentation. Documentation is there to provide information that people can trust.
From Qt documentation QML2_IMPORT_PATH is supposed to accept multiple paths.
https://doc.qt.io/qt-5/qtqml-syntax-imports.html
Multiple path entries must be separated by ':' under Unix and ';' under Windows.
Could you double check? If it doesn't work, it would be good to report an issue on the Qt project.
There was a problem hiding this comment.
I just tried:
set QML2_IMPORT_PATH="D:\Meshroom_Dev\qmlAlembic\qml;D:\Meshroom_Dev\QtOIIO\qml;D:\Meshroom_Dev\QtAliceVision\qml"
python meshroom/ui
and none of the plugins are available.
I've created https://bugreports.qt.io/browse/QTBUG-84390
There was a problem hiding this comment.
I've reworded the pull request. It's purposely worded more passively as I'm don't regularly build large projects any more, and even less so on Windows. There's a very large chance I've broken something when I used vcpkg built qt :-)
There was a problem hiding this comment.
And added a link to the QT bug.
There was a problem hiding this comment.
After follow up on https://bugreports.qt.io/browse/QTBUG-84390, turns out quotes don't work in path specifications.
The things you learn.
Update to remove most of the changes, add a comment re quotes, and tweak the example path specs to add to, rather than replace.
Just re-read it and it seems odd to be disjoint from the rest of the installation steps.
|
An alternative would be #1169 (placing alicevision and qtplugins in same meshroom root folder instead of setting the path(s)) |
|
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
|
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
|
This issue is closed due to inactivity. Feel free to re-open if new information is available. |
I tried both
;and:as path separators forQML2_IMPORT_PATHand had no luck. Change above was the first method that got everything enabled.