[Paraview] Install path issue

[Utkarsh Ayachit]

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.ayachit at 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 <orion at 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                  orion at 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
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: Fix.install.patch
Type: text/x-patch
Size: 2214 bytes
Desc: not available
URL: <http://www.paraview.org/pipermail/paraview/attachments/20100603/0647531e/attachment.bin>