Qpid
  1. Qpid
  2. QPID-4134

Provide Perl language bindings for Qpid

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.21
    • Component/s: None
    • Labels:

      Description

      Provides a new Perl language binding that lives in the qpid:: namespace.

      There are no Sub-Tasks for this issue.

        Activity

        Darryl L. Pierce created issue -
        Darryl L. Pierce made changes -
        Field Original Value New Value
        Attachment 0001-Added-a-new-make-target-to-bundle-the-Perl-bindings-.patch [ 12536263 ]
        Darryl L. Pierce made changes -
        Attachment 0001-Added-a-new-make-target-to-bundle-the-Perl-bindings-.patch [ 12536263 ]
        Darryl L. Pierce made changes -
        Attachment 0001-Adds-a-Cmake-target-to-generate-a-source-tarball-for.patch [ 12536415 ]
        Darryl L. Pierce made changes -
        Description Adds a new target to release.sh (--perl|-p) which bundles together files from the bindings/qpid/perl directory. These sources can then be used as upstreams for Perl packages.

        It also includes changes to the Cmake build environment to do the same.
        This patch adds a new target to the Cmake build system to produce a source tarball for the Perl bindings.

        It replaces Makefile.PL.in with Makefile.PL, and removes Perl from the autotools build.
        Hide
        Andrew Stitcher added a comment -

        I'm not really happy that this is really a source tarball in any useful way. The "source" tarball packages up the code generated by swig. This is not really source in the sense that a developer could use it to change anything.

        I'd expect source to include the swig typemap and whatever other files that swig needs to do the generation, and perhaps the original qpid/messaging.h and other C++ header files .Although for packaging purposes you could have a build dependency on the appropriate -devel package and not include the actual files.

        The extra work I think this needs is to make the Cmake tooling now work both standalone and as part of the full build - I'm not sure how good cmake is at that.

        Show
        Andrew Stitcher added a comment - I'm not really happy that this is really a source tarball in any useful way. The "source" tarball packages up the code generated by swig. This is not really source in the sense that a developer could use it to change anything. I'd expect source to include the swig typemap and whatever other files that swig needs to do the generation, and perhaps the original qpid/messaging.h and other C++ header files .Although for packaging purposes you could have a build dependency on the appropriate -devel package and not include the actual files. The extra work I think this needs is to make the Cmake tooling now work both standalone and as part of the full build - I'm not sure how good cmake is at that.
        Darryl L. Pierce made changes -
        Attachment 0001-Adds-a-Cmake-target-to-generate-a-source-tarball-for.patch [ 12536415 ]
        Darryl L. Pierce made changes -
        Attachment 0001-Statically-linked-the-common-swig-files-into-the-Per.patch [ 12537878 ]
        Attachment 0002-Installs-the-qpid.i-file-into-the-data-directory.patch [ 12537879 ]
        Attachment 0003-Adds-a-Cmake-target-to-generate-a-source-tarball-for.patch [ 12537880 ]
        Hide
        Darryl L. Pierce added a comment -

        The updated patches fix the issues brought up by Andrew. The source tarball created bundles the perl.i file and swig_perl_typemaps.i, which will then be used to generated sources separately.

        Show
        Darryl L. Pierce added a comment - The updated patches fix the issues brought up by Andrew. The source tarball created bundles the perl.i file and swig_perl_typemaps.i, which will then be used to generated sources separately.
        Darryl L. Pierce made changes -
        Attachment 0001-Statically-linked-the-common-swig-files-into-the-Per.patch [ 12537878 ]
        Darryl L. Pierce made changes -
        Attachment 0002-Installs-the-qpid.i-file-into-the-data-directory.patch [ 12537879 ]
        Darryl L. Pierce made changes -
        Attachment 0003-Adds-a-Cmake-target-to-generate-a-source-tarball-for.patch [ 12537880 ]
        Darryl L. Pierce made changes -
        Attachment 0001-Added-I-to-the-swig-command-line-for-other-included-.patch [ 12537892 ]
        Attachment 0002-Installs-the-qpid.i-file-into-the-data-directory.patch [ 12537893 ]
        Attachment 0003-Adds-a-Cmake-target-to-generate-a-source-tarball-for.patch [ 12537894 ]
        Darryl L. Pierce made changes -
        Attachment 0001-Added-I-to-the-swig-command-line-for-other-included-.patch [ 12537892 ]
        Darryl L. Pierce made changes -
        Attachment 0002-Installs-the-qpid.i-file-into-the-data-directory.patch [ 12537893 ]
        Darryl L. Pierce made changes -
        Attachment 0003-Adds-a-Cmake-target-to-generate-a-source-tarball-for.patch [ 12537894 ]
        Darryl L. Pierce made changes -
        Attachment 0001-Added-I-to-the-swig-command-line-for-other-included-.patch [ 12540082 ]
        Attachment 0002-Adds-a-Cmake-target-to-generate-a-source-tarball-for.patch [ 12540083 ]
        Darryl L. Pierce made changes -
        Attachment 0001-Added-I-to-the-swig-command-line-for-other-included-.patch [ 12540082 ]
        Darryl L. Pierce made changes -
        Attachment 0002-Adds-a-Cmake-target-to-generate-a-source-tarball-for.patch [ 12540083 ]
        Darryl L. Pierce made changes -
        Summary Creates a source tarball for use by the Perl distributions. Provide Perl language bindings for Qpid
        Description This patch adds a new target to the Cmake build system to produce a source tarball for the Perl bindings.

        It replaces Makefile.PL.in with Makefile.PL, and removes Perl from the autotools build.
        Provides a new Perl language binding that lives in the qpid:: namespace.
        Component/s Packaging [ 12313483 ]
        Darryl L. Pierce made changes -
        Assignee Darryl L. Pierce [ mcpierce ]
        Darryl L. Pierce made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 0.21 [ 12323549 ]
        Resolution Fixed [ 1 ]
        Jake Farrell made changes -
        Labels bindings perl, bindings perl
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        137d 22h 42m 1 Darryl L. Pierce 27/Nov/12 18:09

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development