Ticket #11459 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

Paraview 4.3.1 GUI resource files in plugins are no longer supported.

Reported by: Steven Hahn Owned by: Andrei Savici
Priority: critical Milestone: Release 3.4
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Steven Hahn

Description

We have the CMake warnings below when building VATES with Paraview 4.3. These need to be fixed before migrating to this version of Paraview.

CMake Warning at /Users/svh/Documents/MantidProject/ParaView-v4.3.1-source/CMake/ParaViewPlugins.cmake:981 (message):

GUI resource files in plugins are no longer supported. The same functionality can be obtained using Hints in the Server Manager xml files. See the Major API Changes document for details.

Call Stack (most recent call first):

Vates/ParaviewPlugins/ParaViewFilters/SplatterPlot/CMakeLists.txt:3 (ADD_PARAVIEW_PLUGIN)

CMake Warning at /Users/svh/Documents/MantidProject/ParaView-v4.3.1-source/CMake/ParaViewPlugins.cmake:981 (message):

GUI resource files in plugins are no longer supported. The same functionality can be obtained using Hints in the Server Manager xml files. See the Major API Changes document for details.

Call Stack (most recent call first):

Vates/ParaviewPlugins/ParaViewFilters/ScaleWorkspace/CMakeLists.txt:3 (ADD_PARAVIEW_PLUGIN)

CMake Warning at /Users/svh/Documents/MantidProject/ParaView-v4.3.1-source/CMake/ParaViewPlugins.cmake:981 (message):

GUI resource files in plugins are no longer supported. The same functionality can be obtained using Hints in the Server Manager xml files. See the Major API Changes document for details.

Call Stack (most recent call first):

Vates/ParaviewPlugins/ParaViewFilters/PeaksFilter/CMakeLists.txt:3 (ADD_PARAVIEW_PLUGIN)

Change History

comment:1 Changed 6 years ago by Steven Hahn

the Major API Changes document is available at

​https://github.com/Kitware/ParaView/blob/master/Utilities/Doxygen/pages/MajorAPIChanges.md

comment:2 Changed 6 years ago by Stuart Campbell

  • Status changed from new to assigned
  • Owner set to Andrei Savici

Since you're our other Paraview person! :)

comment:3 Changed 6 years ago by Steven Hahn

the paraview 4.3 branch is '7363_vates_paraview_4p3'

comment:4 Changed 6 years ago by Andrei Savici

  • Status changed from assigned to inprogress

Cmake fixes. Refs #11459

Changeset: e94574dbee15933b9d10e485928e223ca32b0f73

comment:5 Changed 6 years ago by Andrei Savici

  • Status changed from inprogress to verify
  • Resolution set to fixed

This is being verified as ​pull request #545.

comment:6 Changed 6 years ago by Steven Hahn

  • Status changed from verify to verifying
  • Tester set to Steven Hahn

comment:7 Changed 6 years ago by Steven Hahn

Merge pull request #545 from mantidproject/11459_Paraview_cmake

Para fixes. Refs #11459

Changeset: 6e8b85a7eab7543040e21fad7dee0867020c057e

comment:8 Changed 6 years ago by Steven Hahn

Currently, the build servers aren't setup to build with paraview 4.3.1. Therefore I'm comfortable merging this into the paraview 4.3 branch (NOT master).

comment:9 Changed 5 years ago by Steven Hahn

  • Status changed from verifying to closed

Merge pull request #545 from mantidproject/11459_Paraview_cmake

Para fixes. Refs #11459

Full changeset: ​6e8b85a7eab7543040e21fad7dee0867020c057e

comment:10 Changed 5 years ago by Nick Draper

Somehow these slipped through without a resolution. Set to Fixed.

comment:11 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​12298

Note: See TracTickets for help on using tickets.