Qpid
  1. Qpid
  2. QPID-4781

cmake build of perl bindings fails in RC2

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.22
    • Fix Version/s: 0.22
    • Component/s: Perl Client
    • Labels:
      None
    • Environment:

      RHEL5

      Description

      Running cmake .. && make && make install

      – Installing: /usr/local/etc/qpid/qpidd.conf
      CMake Error at bindings/qpid/perl/cmake_install.cmake:36 (FILE):
      file INSTALL cannot find file
      "/home/test/qpid-0.22-rc2/cpp/bld/bindings/qpid/perl/libcqpid_perl.so" to
      install.
      Call Stack (most recent call first):
      bindings/cmake_install.cmake:37 (INCLUDE)
      cmake_install.cmake:63 (INCLUDE)

      make: *** [install] Error 1

      The library in that directory is cqpid_perl.so, not libcqpid_perl.so

        Activity

        Hide
        Justin Ross added a comment -

        Reviewed by Alan. Approved for 0.22.

        Show
        Justin Ross added a comment - Reviewed by Alan. Approved for 0.22.
        Hide
        Alan Conway added a comment -

        Reviewed the change, it looks correct to me.

        Show
        Alan Conway added a comment - Reviewed the change, it looks correct to me.
        Hide
        Darryl L. Pierce added a comment -

        There's a fix for a similar issue regarding the Ruby language bindings. For Cmake earlier than 2.8 the produced file is missing the lib prefix. So I copied that fix over to the Perl bindings as well. So now it will be sure that the file installed is named cqpid_perl.so, which is what the bindings expect.

        This commit can be found here:

        http://svn.apache.org/viewvc?view=revision&revision=1482038

        Show
        Darryl L. Pierce added a comment - There's a fix for a similar issue regarding the Ruby language bindings. For Cmake earlier than 2.8 the produced file is missing the lib prefix. So I copied that fix over to the Perl bindings as well. So now it will be sure that the file installed is named cqpid_perl.so, which is what the bindings expect. This commit can be found here: http://svn.apache.org/viewvc?view=revision&revision=1482038
        Hide
        Jimmy Jones added a comment -

        Still an issue in RC3

        Show
        Jimmy Jones added a comment - Still an issue in RC3
        Hide
        Darryl L. Pierce added a comment -

        I'm not sure what issue is surfacing in the original bug report, but the output from the build should be a file named libcqpid_perl.so, not cqpid_perl.so.

        Show
        Darryl L. Pierce added a comment - I'm not sure what issue is surfacing in the original bug report, but the output from the build should be a file named libcqpid_perl.so, not cqpid_perl.so.
        Hide
        Jimmy Jones added a comment -
        Show
        Jimmy Jones added a comment - The commit can be found at: https://svn.apache.org/viewvc?view=revision&revision=1477150
        Hide
        Jimmy Jones added a comment -

        Looks as if this issue was fixed in rev 1477150 - cpp/bindings/qpid/perl/CMakeLists.txt

        Show
        Jimmy Jones added a comment - Looks as if this issue was fixed in rev 1477150 - cpp/bindings/qpid/perl/CMakeLists.txt

          People

          • Assignee:
            Darryl L. Pierce
            Reporter:
            Jimmy Jones
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development