add feature build vtk[qtopengl]#10060
Conversation
|
@NancyLi1013 - fix for this issue. |
|
/azp run |
ports/vtk/CONTROL
Outdated
|
|
||
| Feature: qtopengl | ||
| Description: OpenGL with Qt for vtk on Windows | ||
| Build-Depends: qt5 |
There was a problem hiding this comment.
don't add all of qt5. You probably only need qt5-base
There was a problem hiding this comment.
also you can depend on vtk[core,qt5]
There was a problem hiding this comment.
Good call, this will reduce the build time substantially, however, it also needs qt5-tools,
(if you only include qt5-base, it fails as below):
CMake Error at C:/src/vcpkg/work/vcpkg/scripts/buildsystems/vcpkg.cmake:288 (_find_package):
Could not find a package configuration file provided by "Qt5UiPlugin" with
any of the following names:
Qt5UiPluginConfig.cmake
qt5uiplugin-config.cmake
Add the installation prefix of "Qt5UiPlugin" to CMAKE_PREFIX_PATH or set
"Qt5UiPlugin_DIR" to a directory containing one of the above files. If
"Qt5UiPlugin" provides a separate development package or SDK, be sure it
has been installed.
Call Stack (most recent call first):
C:/src/vcpkg/work/vcpkg/installed/x86-windows/share/cmake/Qt5/Qt5Config.cmake:28 (find_package)
C:/src/vcpkg/work/vcpkg/scripts/buildsystems/vcpkg.cmake:288 (_find_package)
GUISupport/Qt/CMakeLists.txt:128 (find_package)
Note that this has the effect of also bringing in:
* qt5-activeqt[core]:x64-windows
* qt5-base[core]:x64-windows
* qt5-declarative[core]:x64-windows
* qt5-imageformats[core]:x64-windows
* qt5-svg[core]:x64-windows
Which is still better than the whole Qt5 package!
Merge branch 'master' of https://github.com/microsoft/vcpkg
jafrado
left a comment
There was a problem hiding this comment.
Will resubmit new change set.
ports/vtk/CONTROL
Outdated
|
|
||
| Feature: qtopengl | ||
| Description: OpenGL with Qt for vtk on Windows | ||
| Build-Depends: qt5 |
There was a problem hiding this comment.
Good call, this will reduce the build time substantially, however, it also needs qt5-tools,
(if you only include qt5-base, it fails as below):
CMake Error at C:/src/vcpkg/work/vcpkg/scripts/buildsystems/vcpkg.cmake:288 (_find_package):
Could not find a package configuration file provided by "Qt5UiPlugin" with
any of the following names:
Qt5UiPluginConfig.cmake
qt5uiplugin-config.cmake
Add the installation prefix of "Qt5UiPlugin" to CMAKE_PREFIX_PATH or set
"Qt5UiPlugin_DIR" to a directory containing one of the above files. If
"Qt5UiPlugin" provides a separate development package or SDK, be sure it
has been installed.
Call Stack (most recent call first):
C:/src/vcpkg/work/vcpkg/installed/x86-windows/share/cmake/Qt5/Qt5Config.cmake:28 (find_package)
C:/src/vcpkg/work/vcpkg/scripts/buildsystems/vcpkg.cmake:288 (_find_package)
GUISupport/Qt/CMakeLists.txt:128 (find_package)
Note that this has the effect of also bringing in:
* qt5-activeqt[core]:x64-windows
* qt5-base[core]:x64-windows
* qt5-declarative[core]:x64-windows
* qt5-imageformats[core]:x64-windows
* qt5-svg[core]:x64-windows
Which is still better than the whole Qt5 package!
|
/azp run |
|
@jafrado Thanks for your PR. Current this port build failed with the following error, could you please continue to fix it? |
|
@LilyWangL - thanks, did you run this patch-set: 993267b The error you see should not be present, since the lines: include qt5-tools which has the qt5-config.cmake Let me know and I can repost the original patch. |
|
I cloned your latest commit and reproduced the CI test error. Due to This error appear when cmake vtk/src/v8.2.0-d1dd9405dd/GUISupport/Qt/CMakeLists.txt Line71. |
|
@LilyWangL - thanks, I will check it out. |
|
I try the new port.cmake and isntall use following: Howerver, the "vtkQtLabelRenderStrategy.h" is not installed. Is any solution for this? |
|
/azp run |
|
This is really weird. This works for me and I have it checked in to my repo for some time now. |
|
Rerun test maually, I will give you failure log later. |
|
|
Thanks, will check it out tomorrow. |
|
Yep, broken. Hang on, lets go back to basics ...
…On Thu, Mar 26, 2020 at 11:55 PM Jack·Boos·Yu ***@***.***> wrote:
...
-- Found Eigen3: G:/10060/vcpkg/installed/x86-windows/include/eigen3 (Required is at least version "2.91.0")
-- Found ZLIB: optimized;G:/10060/vcpkg/installed/x86-windows/lib/zlib.lib;debug;G:/10060/vcpkg/installed/x86-windows/debug/lib/zlibd.lib (found version "1.2.11")
-- Found Freetype: optimized;G:/10060/vcpkg/installed/x86-windows/lib/freetype.lib;debug;G:/10060/vcpkg/installed/x86-windows/debug/lib/freetyped.lib (found version "2.10.1")
-- Found double-conversion: G:/10060/vcpkg/installed/x86-windows/include/double-conversion
-- Found LZ4: optimized;G:/10060/vcpkg/installed/x86-windows/lib/lz4.lib;debug;G:/10060/vcpkg/installed/x86-windows/debug/lib/lz4d.lib
-- Found LZMA: optimized;G:/10060/vcpkg/installed/x86-windows/lib/lzma.lib;debug;G:/10060/vcpkg/installed/x86-windows/debug/lib/lzmad.lib
-- Found EXPAT: G:/10060/vcpkg/installed/x86-windows/lib/expat.lib (found version "2.2.7")
-- Found OpenGL: opengl32
CMake Error at G:/10060/vcpkg/scripts/buildsystems/vcpkg.cmake:288 (_find_package):
By not providing "FindQt5.cmake" in CMAKE_MODULE_PATH this project has
asked CMake to find a package configuration file provided by "Qt5", but
CMake did not find one.
Could not find a package configuration file provided by "Qt5" with any of
the following names:
Qt5Config.cmake
qt5-config.cmake
Add the installation prefix of "Qt5" to CMAKE_PREFIX_PATH or set "Qt5_DIR"
to a directory containing one of the above files. If "Qt5" provides a
separate development package or SDK, be sure it has been installed.
Call Stack (most recent call first):
GUISupport/Qt/CMakeLists.txt:71 (find_package)
-- Configuring incomplete, errors occurred!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#10060 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACXSUDFENXJLTZP4KXMU4OTRJREXNANCNFSM4KU4LT7A>
.
|
|
Ok, lets try listing qt5-tools first to pickup Qt5Config.cmake, qt5-config.cmake before qtbase, otherwise we can just list the whole qt5 package as a dependency (which is what we have checked in to make a working software where I am working right now). See if this change works: |
|
Let me know how to check/resolve this one. The patches I provided work here locally for me. I guess I need to know how to run your pre-commit |
|
Any progress? |
|
Thanks, will take a look this week. Can you send a link to your CI
setup/test docs?
…On Thu, May 7, 2020 at 9:54 PM Jack·Boos·Yu ***@***.***> wrote:
Any progress?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#10060 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACXSUDBEN3UGCDAFCLVMCOTRQOGAVANCNFSM4KU4LT7A>
.
|
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
|
|
My core development platform is Linux and I don't have that error. If there
is something I can read to try to debug your Azure pipeline let me know.
Otherwise, I will wait for you to update to VTK 9 and then retest and see
if it works.
…On Mon, May 11, 2020 at 2:26 AM Jack·Boos·Yu ***@***.***> wrote:
x64-linux regression(vtk):
CMake Error at /mnt/1/s/scripts/buildsystems/vcpkg.cmake:329 (_find_package):
Could not find a package configuration file provided by "Qt5X11Extras" with
any of the following names:
Qt5X11ExtrasConfig.cmake
qt5x11extras-config.cmake
Add the installation prefix of "Qt5X11Extras" to CMAKE_PREFIX_PATH or set
"Qt5X11Extras_DIR" to a directory containing one of the above files. If
"Qt5X11Extras" provides a separate development package or SDK, be sure it
has been installed.
Call Stack (most recent call first):
/mnt/1/s/installed/x64-linux/share/cmake/Qt5/Qt5Config.cmake:28 (find_package)
/mnt/1/s/scripts/buildsystems/vcpkg.cmake:329 (_find_package)
GUISupport/Qt/CMakeLists.txt:71 (find_package)
x64-osx regression(pcl):
CMake Error at /Volumes/data/work/1/s/installed/x64-osx/share/cmake/Qt5Core/Qt5CoreConfig.cmake:113 (message):
Library not found: m
Call Stack (most recent call first):
/Volumes/data/work/1/s/installed/x64-osx/share/cmake/Qt5Core/Qt5CoreConfig.cmake:195 (_qt5_Core_parse_prl)
/Volumes/data/work/1/s/installed/x64-osx/share/cmake/Qt5Core/Qt5CoreConfig.cmake:317 (_populate_Core_target_properties)
/Volumes/data/work/1/s/scripts/buildsystems/vcpkg.cmake:329 (_find_package)
/Volumes/data/work/1/s/installed/x64-osx/share/cmake/Qt5Gui/Qt5GuiConfig.cmake:255 (find_package)
/Volumes/data/work/1/s/scripts/buildsystems/vcpkg.cmake:329 (_find_package)
/Volumes/data/work/1/s/installed/x64-osx/share/cmake/Qt5Widgets/Qt5WidgetsConfig.cmake:255 (find_package)
/Volumes/data/work/1/s/scripts/buildsystems/vcpkg.cmake:329 (_find_package)
/Volumes/data/work/1/s/installed/x64-osx/share/cmake/Qt5/Qt5Config.cmake:28 (find_package)
/Volumes/data/work/1/s/scripts/buildsystems/vcpkg.cmake:329 (_find_package)
/Volumes/data/work/1/s/installed/x64-osx/share/vtk/Modules/vtkGUISupportQt.cmake:19 (find_package)
/Volumes/data/work/1/s/installed/x64-osx/share/vtk/vtkModuleAPI.cmake:45 (include)
/Volumes/data/work/1/s/installed/x64-osx/share/vtk/vtkModuleAPI.cmake:15 (vtk_module_load)
/Volumes/data/work/1/s/installed/x64-osx/share/vtk/vtkModuleAPI.cmake:152 (_vtk_module_config_recurse)
/Volumes/data/work/1/s/installed/x64-osx/share/vtk/VTKConfig.cmake:137 (vtk_module_config)
/Volumes/data/work/1/s/scripts/buildsystems/vcpkg.cmake:329 (_find_package)
CMakeLists.txt:364 (find_package)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#10060 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACXSUDBLFI6MQXBY655F2N3RQ7AFTANCNFSM4KU4LT7A>
.
|
|
Thanks, I saw that, I'm trying to recreate the issue. Not just spam the
problem with run after run of commits. Is there a way to recreate or test
changes in the CI environment?
…On Mon, May 11, 2020 at 8:17 PM Jack·Boos·Yu ***@***.***> wrote:
@jafrado <https://github.com/jafrado> Please get full logs here
<https://dev.azure.com/vcpkg/public/_build/results?buildId=36597&view=artifacts&type=publishedArtifacts>
.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#10060 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACXSUDD7BBTWD3GHMGJT67TRRC5WFANCNFSM4KU4LT7A>
.
|
|
Can you try to install this in your CI environment?
CMake Warning at ports/vtk/portfile.cmake:2 (message):
You will need to install Xorg dependencies to build vtk:
apt-get install libxt-dev
Call Stack (most recent call first):
scripts/ports.cmake:90 (include)
…On Mon, May 11, 2020 at 8:19 PM James Dougherty ***@***.***> wrote:
Thanks, I saw that, I'm trying to recreate the issue. Not just spam the
problem with run after run of commits. Is there a way to recreate or test
changes in the CI environment?
On Mon, May 11, 2020 at 8:17 PM Jack·Boos·Yu ***@***.***>
wrote:
> @jafrado <https://github.com/jafrado> Please get full logs here
> <https://dev.azure.com/vcpkg/public/_build/results?buildId=36597&view=artifacts&type=publishedArtifacts>
> .
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <#10060 (comment)>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/ACXSUDD7BBTWD3GHMGJT67TRRC5WFANCNFSM4KU4LT7A>
> .
>
|
# Conflicts: # ports/vtk/CONTROL # ports/vtk/portfile.cmake
|
I just pulled the latest master, it looks like it includes QtOpenGL support
and VTK 9.0
Let me test it out this week - more later.
…On Mon, May 11, 2020 at 11:04 PM Jack·Boos·Yu ***@***.***> wrote:
***@***.**** requested changes on this pull request.
------------------------------
In ports/vtk/CONTROL
<#10060 (comment)>:
> @@ -2,7 +2,7 @@ Source: vtk
Version: 8.2.0-12
Description: Software system for 3D computer graphics, image processing, and visualization
Homepage: https://github.com/Kitware/VTK
-Build-Depends: zlib, libpng, tiff, libxml2, jsoncpp, glew, freetype, expat, hdf5, libjpeg-turbo, proj4, lz4, libtheora, eigen3, double-conversion, pugixml, libharu, sqlite3, netcdf-c
+Build-Depends: zlib, libpng, tiff, libxml2, jsoncpp, glew, freetype, expat, hdf5, libjpeg-turbo, proj4, lz4, libtheora, eigen3, double-conversion, pugixml, libharu, sqlite3, netcdf-c, qt5
⬇️ Suggested change
-Build-Depends: zlib, libpng, tiff, libxml2, jsoncpp, glew, freetype, expat, hdf5, libjpeg-turbo, proj4, lz4, libtheora, eigen3, double-conversion, pugixml, libharu, sqlite3, netcdf-c, qt5
+Build-Depends: zlib, libpng, tiff, libxml2, jsoncpp, glew, freetype, expat, hdf5, libjpeg-turbo, proj4, lz4, libtheora, eigen3, double-conversion, pugixml, libharu, sqlite3, netcdf-c
------------------------------
In ports/vtk/CONTROL
<#10060 (comment)>:
> @@ -2,7 +2,7 @@ Source: vtk
Version: 8.2.0-12
⬇️ Suggested change
-Version: 8.2.0-12
+Version: 8.2.0-13
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#10060 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACXSUDHW47XJNMCQHLELO63RRDRFRANCNFSM4KU4LT7A>
.
|
|
| endif() | ||
|
|
||
| if("paraview" IN_LIST FEATURES) | ||
| if(VTK_WITH_QT_OPENGL) |
There was a problem hiding this comment.
Where this value is not set, I think it is always 0.
|
If this was waiting for a VM change that might have been done now in support of the Qt update change merged recently. |
|
@JackBoosY @BillyONeal: This PR can probably be closed due to the update to VTK 9 which includes opengl feature. |
|
Yes. Please close. Thank you all very much for your support! |
# Updates: # ports/vtk/CONTROL # ports/vtk/portfile.cmake
# Updates: # ports/vtk/portfile.cmake
|
Please disregard last commit. Thanks again! |
|
Thanks for this PR! |

Describe the pull request
What does your PR fix? Fixes issue [<vtk>] build failure with QTOpenGLWidget #10027
[<vtk>] build failure with QTOpenGLWidget #10027
Which triplets are supported/not supported?
None
Have you updated the CI baseline?
Yes
Does your PR follow the maintainer guide?
Yes