Good catch. I've committed a fix to the master. Dave can you move it
to the release branch as well.
Attached is the patch.
commit 07ba5364f3ab16d33e7ae7c67f64c4b25e2de11f
Author: Utkarsh Ayachit <utkarsh.ayac...@kitware.com>
Date: Thu Jun 3 09:49:23 2010 -0400
Paraview clients were ignoring set install path.
The ParaView custom clients were not passing the PV_INSTALL_LIB_DIR to the
build_paraview_client macro. As a consequence it was doing the
default thing.
Fixed that.
Utkarsh
On Wed, Jun 2, 2010 at 6:02 PM, Orion Poplawski <or...@cora.nwra.com> wrote:
> On 06/02/2010 03:42 PM, Orion Poplawski wrote:
>>
>> On 06/02/2010 10:59 AM, Orion Poplawski wrote:
>>>
>>> I'm working on an updated ParaView 3.8.0 rpm for Fedora and running into
>>> the following issue. The paraview-real and streaming_paraview-real are
>>> getting installed as follows:
>>>
>>> -- Installing:
>>>
>>> /builddir/build/BUILDROOT/paraview-3.8.0-1.fc14.x86_64/usr/lib/paraview-3.8/paraview-real
>>>
>>>
>>> -- Installing:
>>>
>>> /builddir/build/BUILDROOT/paraview-3.8.0-1.fc14.x86_64/usr/lib/streaming_paraview-1.1/streaming_paraview-real
>>>
>>>
>>>
>>> despite setting -DPV_INSTALL_LIB_DIR:PATH=lib64/paraview.
>>>
>>>
>>
>> Another one:
>>
>> /usr/lib/python2.6/site-packages/Xdmf/libvtkXdmfPython.so
>>
>> is installed on a 64-bit machine instead of in /usr/lib64/...
>
> Sorry, that one is okay. Not that pvbatch-real (and others) is installed in
> /usr/lib64/paraview/ as requested, just not paraview-real. Looks like an
> issue of some kind with add_executable_with_forwarding2?
>
> ParaView-3.8.0/CMake/ParaViewBranding.cmake:
> add_executable_with_forwarding2(pv_exe_suffix
> "${PARAVIEW_LIBRARY_DIRS}"
> "../${PARAVIEW_INSTALL_LIB_DIR}"
> "${BPC_INSTALL_LIB_DIR}"
> ${BPC_NAME} WIN32 ${MAKE_BUNDLE}
> ${BPC_NAME}_main.cxx
> ${exe_icon}
> ${apple_bundle_sources}
> ${qt_menu_nib_sources}
> ${EXE_SRCS}
> )
>
> ParaView-3.8.0/ParaViewConfig.cmake.in:
> # The locations/prefixes where this version of ParaView would be installed.
> SET (PARAVIEW_INSTALL_LIB_DIR "@PV_INSTALL_LIB_DIR@")
>
> Perhaps this isn't getting evaluated in the correct order?
>
> --
> Orion Poplawski
> Technical Manager 303-415-9701 x222
> NWRA/CoRA Division FAX: 303-415-9702
> 3380 Mitchell Lane or...@cora.nwra.com
> Boulder, CO 80301 http://www.cora.nwra.com
> _______________________________________________
> Powered by www.kitware.com
>
> Visit other Kitware open-source projects at
> http://www.kitware.com/opensource/opensource.html
>
> Please keep messages on-topic and check the ParaView Wiki at:
> http://paraview.org/Wiki/ParaView
>
> Follow this link to subscribe/unsubscribe:
> http://www.paraview.org/mailman/listinfo/paraview
>
commit 07ba5364f3ab16d33e7ae7c67f64c4b25e2de11f
Author: Utkarsh Ayachit <utkarsh.ayac...@kitware.com>
Date: Thu Jun 3 09:49:23 2010 -0400
Paraview clients were ignoring set install path.
The ParaView custom clients were not passing the PV_INSTALL_LIB_DIR to the
build_paraview_client macro. As a consequence it was doing the default thing.
Fixed that.
diff --git a/Applications/AdaptiveParaView/CMakeLists.txt b/Applications/AdaptiveParaView/CMakeLists.txt
index f93cd01..a951229 100644
--- a/Applications/AdaptiveParaView/CMakeLists.txt
+++ b/Applications/AdaptiveParaView/CMakeLists.txt
@@ -45,6 +45,8 @@ build_paraview_client(adaptive_paraview
${CMAKE_CURRENT_SOURCE_DIR}/CustomResources/CustomFilters.xml
${CMAKE_CURRENT_SOURCE_DIR}/CustomResources/CustomSources.xml
SOURCES ${Application_SOURCE_FILES}
+ INSTALL_BIN_DIR "${PV_INSTALL_BIN_DIR}"
+ INSTALL_LIB_DIR "${PV_INSTALL_LIB_DIR}"
)
ADD_EXECUTABLE(ppRawStridedReader2 ppRawStridedReader2.cxx)
diff --git a/Applications/ParaView/CMakeLists.txt b/Applications/ParaView/CMakeLists.txt
index 6ec9d99..75de3ed 100644
--- a/Applications/ParaView/CMakeLists.txt
+++ b/Applications/ParaView/CMakeLists.txt
@@ -63,6 +63,8 @@ build_paraview_client(paraview
${PARAVIEW_EXTRA_GUI_RESOURCES} # Add any compiled in GUI configuration xmls
COMPRESSED_HELP_FILE "${ParaView_BINARY_DIR}/Documentation/paraview.qch"
SOURCES ${ParaView_SOURCE_FILES}
+ INSTALL_BIN_DIR "${PV_INSTALL_BIN_DIR}"
+ INSTALL_LIB_DIR "${PV_INSTALL_LIB_DIR}"
)
# Configure the ".plugins" configuration xml for making paraview aware of the
diff --git a/Applications/StreamingParaView/CMakeLists.txt b/Applications/StreamingParaView/CMakeLists.txt
index e113172..c39f768 100644
--- a/Applications/StreamingParaView/CMakeLists.txt
+++ b/Applications/StreamingParaView/CMakeLists.txt
@@ -45,6 +45,8 @@ build_paraview_client(streaming_paraview
${CMAKE_CURRENT_SOURCE_DIR}/CustomResources/CustomFilters.xml
${CMAKE_CURRENT_SOURCE_DIR}/CustomResources/CustomSources.xml
SOURCES ${Application_SOURCE_FILES}
+ INSTALL_BIN_DIR "${PV_INSTALL_BIN_DIR}"
+ INSTALL_LIB_DIR "${PV_INSTALL_LIB_DIR}"
)
IF(FALSE)#BUILD_TESTING)
_______________________________________________
Powered by www.kitware.com
Visit other Kitware open-source projects at
http://www.kitware.com/opensource/opensource.html
Please keep messages on-topic and check the ParaView Wiki at:
http://paraview.org/Wiki/ParaView
Follow this link to subscribe/unsubscribe:
http://www.paraview.org/mailman/listinfo/paraview
